Add unit test for Scilab backend
ClosedPublic

Authored by shubham on Mar 22 2020, 5:31 PM.

Details

Summary

Added unit test for Scilab backend.

Test Plan

ctest passes!

Diff Detail

Repository
R55 Cantor
Branch
tests
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24081
Build 24099: arc lint + arc unit
shubham created this revision.Mar 22 2020, 5:31 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptMar 22 2020, 5:31 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
shubham requested review of this revision.Mar 22 2020, 5:31 PM
asemke added inline comments.Mar 22 2020, 5:38 PM
src/backends/scilab/testscilab.cpp
25

where do you need this header file?

29

shouldn't be required here.

40

the usage of spaces is not consistent in Cantor's code. For new code we try to be more consistent and use spaces around operators. We don't use spaces next to the bracket. So, it should be

QVERIFY(e != nullptr);
shubham updated this revision to Diff 78243.Mar 22 2020, 5:53 PM

Fix coding style

shubham updated this revision to Diff 78244.Mar 22 2020, 6:21 PM
shubham marked an inline comment as done.

Add plot test

shubham marked 2 inline comments as done.Mar 22 2020, 6:21 PM
shubham edited the test plan for this revision. (Show Details)
shubham updated this revision to Diff 78245.Mar 22 2020, 6:26 PM

Remove unnecessary headers

shubham updated this revision to Diff 78247.Mar 22 2020, 6:43 PM

Fix coding style

asemke accepted this revision.Mar 22 2020, 6:44 PM
This revision is now accepted and ready to land.Mar 22 2020, 6:44 PM
This revision was automatically updated to reflect the committed changes.