test_rosbag: add target dependency to fix unit test failures in paral…

…lel builds See #1651 (comment) for details.
ros · Apr 12, 2019 · 76710de · 76710de
1 parent 1eb7dfa
commit 76710de
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/test/test_rosbag/bag_migration_tests/CMakeLists.txt b/test/test_rosbag/bag_migration_tests/CMakeLists.txt
@@ -97,3 +97,7 @@ configure_file(test/random_play_sim.xml.in
   ${PROJECT_BINARY_DIR}/test/random_play_sim.xml)
 add_rostest(${PROJECT_BINARY_DIR}/test/random_play_sim.xml
   DEPENDENCIES run_tests_test_rosbag_rostest_test_random_record.xml)
+
+# Make sure that rostest random_record.xml is not executed twice, running in
+# parallel (https://github.com/ros/ros_comm/pull/1651#issuecomment-482148146):
+add_dependencies(run_tests_test_rosbag_rostest_test_random_play.xml run_tests_test_rosbag_rostest_test_random_play_sim.xml)