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

…lel builds (#1877) See #1651 (comment) for details. Co-authored-by: Johannes Meyer <[email protected]>
ros · Feb 5, 2020 · 0830a4f · 0830a4f
1 parent 5506085
commit 0830a4f
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)