tempo-plotter Changeset - a402a433400e · protofusion repositories

Changeset - a402a433400e

Parent rev.

Child rev.

[Not reviewed]

default

0 4 0

Hasan Yavuz ÖZDERYA - 7 years ago 2018-07-26 14:54:32
hy@ozderya.net

fix tests

4 files changed with 6 insertions and 18 deletions:

src/demoreadersettings.ui

tests/CMakeLists.txt

tests/test_readers.cpp

tests/test_recorder.cpp

0 comments (0 inline, 0 general)

src/demoreadersettings.ui

➞

Show inline comments

@@ @@ -41,25 +41,25 @@ @@
         <string/>
        </property>
        <property name="keyboardTracking">
         <bool>false</bool>
        </property>
        <property name="minimum">
         <number>1</number>
        </property>
        <property name="maximum">
         <number>32</number>
        </property>
        <property name="value">
-        <number>5</number>
+        <number>1</number>
        </property>
       </widget>
      </item>
     </layout>
    </item>
    <item>
     <spacer name="verticalSpacer">
      <property name="orientation">
       <enum>Qt::Vertical</enum>
      </property>
      <property name="sizeHint" stdset="0">
       <size>

tests/CMakeLists.txt

➞

Show inline comments

@@ @@ -35,42 +35,44 @@ add_executable(Test EXCLUDE_FROM_ALL @@
   ../src/readonlybuffer.cpp
   ../src/stream.cpp
   ../src/streamchannel.cpp
   ../src/channelinfomodel.cpp
+  )
 add_test(NAME test1 COMMAND Test)
 qt5_use_modules(Test Widgets)
 qt5_wrap_ui(UI_FILES_T
   ../src/binarystreamreadersettings.ui
   ../src/asciireadersettings.ui
   ../src/framedreadersettings.ui
   ../src/demoreadersettings.ui
   ../src/numberformatbox.ui
   ../src/endiannessbox.ui
+  )
 # test for readers
 add_executable(TestReaders EXCLUDE_FROM_ALL
   test_readers.cpp
   ../src/samplepack.cpp
   ../src/sink.cpp
   ../src/source.cpp
   ../src/abstractreader.cpp
   ../src/binarystreamreader.cpp
   ../src/binarystreamreadersettings.cpp
   ../src/asciireader.cpp
   ../src/asciireadersettings.cpp
   ../src/framedreader.cpp
   ../src/framedreadersettings.cpp
   ../src/demoreader.cpp
   ../src/demoreadersettings.cpp
   ../src/commandedit.cpp
   ../src/endiannessbox.cpp
   ../src/numberformatbox.cpp
   ../src/numberformat.cpp
   ${UI_FILES_T}
+  )
 qt5_use_modules(TestReaders Widgets Test)
 add_test(NAME test_readers COMMAND TestReaders)
 # test for recroder
 add_executable(TestRecorder EXCLUDE_FROM_ALL
   test_recorder.cpp

tests/test_readers.cpp

➞

Show inline comments

@@ @@ -157,68 +157,54 @@ TEST_CASE("FramedReader shouldn't read w @@
     REQUIRE(sink._hasX == false);
     bufferDev.open(QIODevice::ReadWrite);
     const uint8_t data[] = {0xAA, 0xBB, 4, 0x01, 0x02, 0x03, 0x04};
     bufferDev.write((const char*) data, 7);
     bufferDev.seek(0);
     QSignalSpy spy(&bufferDev, SIGNAL(readyRead()));
     REQUIRE_FALSE(spy.wait(READYREAD_TIMEOUT));
     REQUIRE(sink.totalFed == 0);
+}
 TEST_CASE("DemoReader doesn't have a settings widget", "[reader, demo]")
+{
     QBuffer bufferDev;          // not actually used
     DemoReader reader(&bufferDev);
     REQUIRE(reader.settingsWidget() == NULL);
+}
 TEST_CASE("Generating data with DemoReader", "[reader, demo]")
+{
     QBuffer bufferDev;          // not actually used
     DemoReader demoReader(&bufferDev);
     demoReader.enable(true);
     TestSink sink;
     demoReader.connectSink(&sink);
     REQUIRE(sink._numChannels == 1);
     demoReader.setNumOfChannels(3);
     REQUIRE(sink._numChannels == 3);
     REQUIRE(sink._hasX == false);
     // we need to wait somehow, we are not actually looking for signals
     QSignalSpy spy(&bufferDev, SIGNAL(readyRead()));
     REQUIRE_FALSE(spy.wait(1000)); // we need some time for demoreader to produce data
     REQUIRE(sink.totalFed >= 9);
+}
 TEST_CASE("DemoReader shouldn't generate data when paused", "[reader, demo]")
+{
     QBuffer bufferDev;          // not actually used
     DemoReader demoReader(&bufferDev); // paused by default
     TestSink sink;
     demoReader.connectSink(&sink);
     REQUIRE(sink._numChannels == 1);
     demoReader.setNumOfChannels(3);
     REQUIRE(sink._numChannels == 3);
     // we need to wait somehow, we are not actually looking for signals
     QSignalSpy spy(&bufferDev, SIGNAL(readyRead()));
     REQUIRE_FALSE(spy.wait(1000)); // we need some time for demoreader to produce data
     REQUIRE(sink.totalFed == 0);
+}
 // Note: this is added because `QApplication` must be created for widgets
 #include <QApplication>
 int main(int argc, char* argv[])
+{
     QApplication a(argc, argv);
     int result = Catch::Session().run( argc, argv );
     return result;
+}

tests/test_recorder.cpp

➞

Show inline comments

@@ @@ -38,25 +38,25 @@ TEST_CASE("test recording single channel @@
     // connect source → sink
     source.connectSink(&rec);
     // prepare data
     QStringList channelNames({"Channel 1"});
     SamplePack samples(5, 1);
     for (int i = 0; i < 5; i++)
+    {
         samples.data(0)[i] = i+1;
+    }
     // test
     rec.startRecording(fileName, ",", channelNames);
+    rec.startRecording(fileName, ",", channelNames, false);
     source._feed(samples);
     rec.stopRecording();
     // read file contents back
     QFile recordFile(fileName);
     REQUIRE(recordFile.open(QIODevice::ReadOnly | QIODevice::Text));
     // NOTE: mind the extra parantheses, otherwise 'catch' macros fail to compile
     REQUIRE((recordFile.readLine() == "Channel 1\n"));
     for (int i = 0; i < 5; i++)
         REQUIRE((recordFile.readLine() == QString("%1\n").arg(i+1)));
     // cleanup
@@ @@ -78,25 +78,25 @@ TEST_CASE("test recording multiple chann @@
     // prepare data
     QStringList channelNames({"Channel 1", "Channel 2", "Channel 3"});
     SamplePack samples(5, 3);
     for (int ci = 0; ci < 3; ci++)
+    {
         for (int i = 0; i < 5; i++)
+        {
             samples.data(ci)[i] = (ci+1)*(i+1);
+        }
+    }
     // test
     rec.startRecording(fileName, ",", channelNames);
+    rec.startRecording(fileName, ",", channelNames, false);
     source._feed(samples);
     rec.stopRecording();
     // read file contents back
     QFile recordFile(fileName);
     REQUIRE(recordFile.open(QIODevice::ReadOnly | QIODevice::Text));
     // NOTE: mind the extra parantheses, otherwise 'catch' macros fail to compile
     REQUIRE((recordFile.readLine() == "Channel 1,Channel 2,Channel 3\n"));
     REQUIRE((recordFile.readLine() == "1,2,3\n"));
     REQUIRE((recordFile.readLine() == "2,4,6\n"));
     REQUIRE((recordFile.readLine() == "3,6,9\n"));
     REQUIRE((recordFile.readLine() == "4,8,12\n"));

0 comments (0 inline, 0 general)