Bug 454 - Unit tests need update to new sr C++ bindings based code
Summary: Unit tests need update to new sr C++ bindings based code
Status: CONFIRMED
Alias: None
Product: PulseView
Classification: Unclassified
Component: Unit tests (show other bugs)
Version: unreleased development snapshot
Hardware: All All
: Normal normal
Target Milestone: ---
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 366
  Show dependency treegraph
 
Reported: 2014-10-19 19:29 CEST by Uwe Hermann
Modified: 2015-09-23 15:43 CEST (History)
2 users (show)



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Uwe Hermann 2014-10-19 19:29:43 CEST
The PulseView unit tests (-DENABLE_TESTS=y) need an update:

[ 57%] Built target pulseview
[ 58%] Building CXX object test/CMakeFiles/pulseview-test.dir/data/logicsnapshot.cpp.o
[...]/pulseview/test/data/logicsnapshot.cpp: In function ‘void LogicSnapshotTest::push_logic(pv::data::LogicSnapshot&, unsigned int, uint8_t)’: 
[...]/pulseview/test/data/logicsnapshot.cpp:36:2: error: ‘sr_datafeed_logic’ was not declared in this scope
  sr_datafeed_logic logic;
  ^
[...]/pulseview/test/data/logicsnapshot.cpp:37:2: error: ‘logic’ was not declared in this scope
  logic.unitsize = 1;
  ^
[...]/pulseview/test/data/logicsnapshot.cpp: In member function ‘void LogicSnapshotTest::Basic::test_method()’:
[...]/pulseview/test/data/logicsnapshot.cpp:65:2: error: ‘sr_datafeed_logic’ was not declared in this scope
  sr_datafeed_logic logic;
  ^
[...]/pulseview/test/data/logicsnapshot.cpp:66:2: error: ‘logic’ was not declared in this scope
  logic.length = 0;
  ^
[...]/pulseview/test/data/logicsnapshot.cpp: In member function ‘void LogicSnapshotTest::LargeData::test_method()’:
[...]/pulseview/test/data/logicsnapshot.cpp:161:2: error: ‘sr_datafeed_logic’ was not declared in this scope
  sr_datafeed_logic logic;
  ^
Comment 1 Uwe Hermann 2015-09-23 15:43:17 CEST
Drop 0.3.0 milestone. This is nice to have fixed, but not a hard requirement for the 0.3.0 release.