Add a message copy test and add a test timeout

Re-enable tests
matt-attack · Feb 15, 2025 · 50a3cd9 · 50a3cd9
1 parent 016014b
commit 50a3cd9
Show file tree

Hide file tree

Showing 4 changed files with 32 additions and 14 deletions.
diff --git a/.github/workflows/ccpp.yml b/.github/workflows/ccpp.yml
@@ -20,5 +20,5 @@ jobs:
       run: cmake .
     - name: cmake build
       run: cmake --build .
-#    - name: Run Tests
-#      run: cd tests && ctest -VV --timeout 5
+    - name: Run Tests
+      run: cd tests && ctest -VV
diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
@@ -17,5 +17,6 @@ foreach(test_file ${tests})
       STRING(REGEX REPLACE ${TEST_REGEX} "\\1" test ${test})
       message(STATUS "  add test: ${test}")
       add_test(NAME ${test_file}_${test} COMMAND ${test_file} ${test})
+      set_tests_properties(${test_file}_${test} PROPERTIES TIMEOUT 10)
   endforeach()
 endforeach()
diff --git a/tests/test_pubsub_c.cpp b/tests/test_pubsub_c.cpp
@@ -13,17 +13,11 @@
 
 also add a test to test subscriber/publisher numbers
 
-add a way to add a timeout to tests
-
 add a test for generic message handling
 
 make the pose viewer also be able to view odom in pubviz
 (maybe think of a way to view velocities)*/
 
-//lets test queue size too
-
-//so for that test, shove over N messages
-
 TEST(test_publish_subscribe_generic, []() {
   struct ps_node_t node;
   ps_node_init(&node, "test_node", "", true);

diff --git a/tests/test_serialization.cpp b/tests/test_serialization.cpp
@@ -6,9 +6,8 @@
 #include "mini_mock.hpp"
 
 TEST(test_joy_serialization, []() {
-
 	// try serializing then deserializing a message to make sure it all matches
-    pubsub::msg::Joy msg;
+  pubsub::msg::Joy msg;
 	msg.buttons = 0x12345678;
 	for (int i = 0; i < 8; i++)
 		msg.axes[i] = i;
@@ -29,7 +28,7 @@ TEST(test_joy_serialization, []() {
 void test2()
 {
 	// verify that the C type matches the C++ one memory wise
-    pubsub::msg::Costmap msg;
+  pubsub::msg::Costmap msg;
 	msg.width = 100;
 	msg.height = 200;
 	msg.resolution = 1.0;
@@ -57,7 +56,7 @@ TEST(test_costmap_c_cpp, []() {
 
 TEST(test_costmap_serialization, []() {
 	// try serializing then deserializing a message to make sure it all matches
-    pubsub::msg::Costmap msg;
+  pubsub::msg::Costmap msg;
 	msg.width = 100;
 	msg.height = 200;
 	msg.resolution = 1.0;
@@ -88,7 +87,7 @@ TEST(test_costmap_serialization, []() {
 
 TEST(test_path2d_serialization, []() {
 	// try serializing then deserializing a message to make sure it all matches
-    pubsub::msg::Path2D msg;
+  pubsub::msg::Path2D msg;
 	msg.frame = 100;
 	msg.points.resize(123);
 	for (int i = 0; i < msg.points.size(); i++)
@@ -114,9 +113,33 @@ TEST(test_path2d_serialization, []() {
 	delete out;
 });
 
+TEST(test_path2d_copy, []() {
+	// make sure copying a message works
+  pubsub::msg::Path2D msg;
+	msg.frame = 100;
+	msg.points.resize(123);
+	for (int i = 0; i < msg.points.size(); i++)
+	{
+		msg.points[i].x = i*2;
+		msg.points[i].y = i*2 + 1;
+	}
+
+	pubsub::msg::Path2D msg2 = msg;
+
+	EXPECT(msg2.frame == msg.frame);
+	EXPECT(msg2.points.size() == msg.points.size());
+	for (int i = 0; i < msg.points.size(); i++)
+	{
+		if (msg2.points[i].x != msg.points[i].x)
+			EXPECT(false);
+		if (msg2.points[i].y != msg.points[i].y)
+			EXPECT(false);
+	}
+});
+
 TEST(test_path2d_foreach, []() {
 	// try serializing then deserializing a message, making sure the foreach loop over it works as expected
-    pubsub::msg::Path2D msg;
+  pubsub::msg::Path2D msg;
 	msg.frame = 100;
 	msg.points.resize(3);
 	for (int i = 0; i < msg.points.size(); i++)