move parsing tests to test_parseMessage.cpp

Author: Murmele

test/test_dbc.cpp

@@ -155,47 +155,3 @@ TEST_CASE("Special characters in unit") {
       REQUIRE(signal.unit.compare("Km/h") == 0);
     }
 }
-
-TEST_CASE("Parse Message little endian") {
-    const auto* filename = std::tmpnam(NULL);
-
-    create_tmp_dbc_with(filename, R"(BO_ 541 STATUS: 8 DEVICE1
- SG_ Temperature : 48|16@1+ (0.01,-40) [-40|125] "C"  DEVICE1
- SG_ SOH : 0|16@1+ (0.01,0) [0|100] "%"  DEVICE1
- SG_ SOE : 32|16@1+ (0.01,0) [0|100] "%"  DEVICE1
- SG_ SOC : 16|16@1+ (0.01,0) [0|100] "%"  DEVICE1)");
-
-    libdbc::DbcParser p;
-    p.parse_file(filename);
-
-    std::vector<uint8_t> data{0x08, 0x27, 0xa3, 0x22, 0xe5, 0x1f, 0x45, 0x14}; // little endian
-    std::vector<double> result_values;
-    REQUIRE(p.parseMessage(0x21d, data, result_values) == libdbc::Message::ParseSignalsStatus::Success);
-    REQUIRE(result_values.size() == 4);
-    REQUIRE(Catch::Approx(result_values.at(0)) == 99.92);
-    REQUIRE(Catch::Approx(result_values.at(1)) == 88.67);
-    REQUIRE(Catch::Approx(result_values.at(2)) == 81.65);
-    REQUIRE(Catch::Approx(result_values.at(3)) == 11.89);
-}
-
-TEST_CASE("Parse Message big endian") {
-    const auto* filename = std::tmpnam(NULL);
-    create_tmp_dbc_with(filename, R"(BO_ 541 STATUS: 8 DEVICE1
- SG_ Temperature : 48|16@0+ (0.01,-40) [-40|125] "C"  DEVICE1
- SG_ SOH : 0|16@0+ (0.01,0) [0|100] "%"  DEVICE1
- SG_ SOE : 32|16@0+ (0.01,0) [0|100] "%"  DEVICE1
- SG_ SOC : 16|16@0+ (0.01,0) [0|100] "%"  DEVICE1)");
-
-    libdbc::DbcParser p;
-    p.parse_file(filename);
-
-    std::vector<uint8_t> data{0x27, 0x08, 0x22, 0xa3, 0x1f, 0xe5, 0x14, 0x45}; // big endian
-    std::vector<double> result_values;
-    REQUIRE(p.parseMessage(0x21d, data, result_values) == libdbc::Message::ParseSignalsStatus::ErrorBigEndian);
-    // Big endian not yet supported
-//    REQUIRE(result_values.size() == 4);
-//    REQUIRE(Catch::Approx(result_values.at(0)) == 99.92);
-//    REQUIRE(Catch::Approx(result_values.at(1)) == 88.67);
-//    REQUIRE(Catch::Approx(result_values.at(2)) == 81.65);
-//    REQUIRE(Catch::Approx(result_values.at(3)) == 11.89);
-}

test/test_parseMessage.cpp

@@ -119,4 +119,48 @@ TEST_CASE("Parse Message Big Number not aligned little endian") {
     }
 }
 
+TEST_CASE("Parse Message little endian") {
+    const auto* filename = std::tmpnam(NULL);
+
+    create_tmp_dbc_with(filename, R"(BO_ 541 STATUS: 8 DEVICE1
+ SG_ Temperature : 48|16@1+ (0.01,-40) [-40|125] "C"  DEVICE1
+ SG_ SOH : 0|16@1+ (0.01,0) [0|100] "%"  DEVICE1
+ SG_ SOE : 32|16@1+ (0.01,0) [0|100] "%"  DEVICE1
+ SG_ SOC : 16|16@1+ (0.01,0) [0|100] "%"  DEVICE1)");
+
+    libdbc::DbcParser p;
+    p.parse_file(filename);
+
+    std::vector<uint8_t> data{0x08, 0x27, 0xa3, 0x22, 0xe5, 0x1f, 0x45, 0x14}; // little endian
+    std::vector<double> result_values;
+    REQUIRE(p.parseMessage(0x21d, data, result_values) == libdbc::Message::ParseSignalsStatus::Success);
+    REQUIRE(result_values.size() == 4);
+    REQUIRE(Catch::Approx(result_values.at(0)) == 99.92);
+    REQUIRE(Catch::Approx(result_values.at(1)) == 88.67);
+    REQUIRE(Catch::Approx(result_values.at(2)) == 81.65);
+    REQUIRE(Catch::Approx(result_values.at(3)) == 11.89);
+}
+
+TEST_CASE("Parse Message big endian") {
+    const auto* filename = std::tmpnam(NULL);
+    create_tmp_dbc_with(filename, R"(BO_ 541 STATUS: 8 DEVICE1
+ SG_ Temperature : 48|16@0+ (0.01,-40) [-40|125] "C"  DEVICE1
+ SG_ SOH : 0|16@0+ (0.01,0) [0|100] "%"  DEVICE1
+ SG_ SOE : 32|16@0+ (0.01,0) [0|100] "%"  DEVICE1
+ SG_ SOC : 16|16@0+ (0.01,0) [0|100] "%"  DEVICE1)");
+
+    libdbc::DbcParser p;
+    p.parse_file(filename);
+
+    std::vector<uint8_t> data{0x27, 0x08, 0x22, 0xa3, 0x1f, 0xe5, 0x14, 0x45}; // big endian
+    std::vector<double> result_values;
+    REQUIRE(p.parseMessage(0x21d, data, result_values) == libdbc::Message::ParseSignalsStatus::ErrorBigEndian);
+    // Big endian not yet supported
+//    REQUIRE(result_values.size() == 4);
+//    REQUIRE(Catch::Approx(result_values.at(0)) == 99.92);
+//    REQUIRE(Catch::Approx(result_values.at(1)) == 88.67);
+//    REQUIRE(Catch::Approx(result_values.at(2)) == 81.65);
+//    REQUIRE(Catch::Approx(result_values.at(3)) == 11.89);
+}
+
 // TODO: create also for big endian!