Fix Octave's broken unit test
AbandonedPublic

Authored by shubham on Mar 23 2020, 8:33 AM.

Details

Reviewers
asemke
sirgienko
Test Plan

ctest -R testoctave passes!

Diff Detail

Repository
R55 Cantor
Branch
fixtestoctave
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24092
Build 24110: arc lint + arc unit
shubham created this revision.Mar 23 2020, 8:33 AM
Restricted Application added a project: KDE Edu. · View Herald TranscriptMar 23 2020, 8:33 AM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
shubham requested review of this revision.Mar 23 2020, 8:33 AM
sirgienko requested changes to this revision.EditedMar 23 2020, 2:45 PM
sirgienko added a subscriber: sirgienko.

This is actually wrong changes. This test is almost fine - the command should produce image result. Testing as "not text result" not actually correct, yes, and this is already fixed in the commit d700f7be39d204b370a351330ebd4fdadd6ff9dd. So, I think, this changes should be canceled.
Also, if you have a problems with Octave tests, you should enable plot integration in Octave settings.

This revision now requires changes to proceed.Mar 23 2020, 2:45 PM
shubham abandoned this revision.Mar 23 2020, 2:49 PM