Turn off rmf_rxcpp tests that are producing spurious CI failures due …

…to issues in the underlying middleware implementation Signed-off-by: Michael X. Grey <[email protected]>
open-rmf · Aug 17, 2023 · 2662e61 · 2662e61
1 parent 2cff1a6
commit 2662e61
Showing 1 changed file with 42 additions and 36 deletions.
diff --git a/rmf_fleet_adapter/rmf_rxcpp/CMakeLists.txt b/rmf_fleet_adapter/rmf_rxcpp/CMakeLists.txt
@@ -24,40 +24,46 @@ target_link_libraries(rmf_rxcpp
     ${rclcpp_LIBRARIES}
 )
 
-if(BUILD_TESTING)
-  find_package(ament_cmake_catch2 REQUIRED)
-  find_package(rmf_utils REQUIRED)
-  find_package(std_msgs REQUIRED)
-  find_package(ament_cmake_uncrustify REQUIRED)
-  find_file(uncrustify_config_file NAMES
-    NAMES "rmf_code_style.cfg"
-    PATHS "${rmf_utils_DIR}/../../../share/rmf_utils/")
-
-  ament_add_catch2(
-    test_rmf_rxcpp
-      test/main.cpp
-      test/test_RxJobs.cpp
-      test/test_Transport.cpp
-  )
-  target_include_directories(test_rmf_rxcpp
-    PRIVATE
-      ${std_msgs_INCLUDE_DIRS}
-  )
-  target_link_libraries(test_rmf_rxcpp
-    PRIVATE
-      rmf_rxcpp
-      rmf_utils::rmf_utils
-      ${std_msgs_LIBRARIES}
-  )
-
-  find_package(rmf_traffic REQUIRED)
-
-  add_executable(rmf_rxcpp_example_plan_path examples/PlanPath.cpp)
-
-  target_link_libraries(rmf_rxcpp_example_plan_path
-    PUBLIC
-      rmf_traffic::rmf_traffic
-      rmf_rxcpp
-  )
+# Issues in RMW implementations are causing spurious failures for these tests,
+# possibly due to race conditions in the memory management of the underlying
+# middleware implementations while the test spins up and tears down rclcpp nodes
+# too quickly. This is giving us spurious test failures in the CI, so we will
+# simply disable these tests since this code is heading towards retirement
+# anyway.
+# if(BUILD_TESTING)
+#   find_package(ament_cmake_catch2 REQUIRED)
+#   find_package(rmf_utils REQUIRED)
+#   find_package(std_msgs REQUIRED)
+#   find_package(ament_cmake_uncrustify REQUIRED)
+#   find_file(uncrustify_config_file NAMES
+#     NAMES "rmf_code_style.cfg"
+#     PATHS "${rmf_utils_DIR}/../../../share/rmf_utils/")
 
-endif()
+#   ament_add_catch2(
+#     test_rmf_rxcpp
+#       test/main.cpp
+#       test/test_RxJobs.cpp
+#       test/test_Transport.cpp
+#   )
+#   target_include_directories(test_rmf_rxcpp
+#     PRIVATE
+#       ${std_msgs_INCLUDE_DIRS}
+#   )
+#   target_link_libraries(test_rmf_rxcpp
+#     PRIVATE
+#       rmf_rxcpp
+#       rmf_utils::rmf_utils
+#       ${std_msgs_LIBRARIES}
+#   )
+
+#   find_package(rmf_traffic REQUIRED)
+
+#   add_executable(rmf_rxcpp_example_plan_path examples/PlanPath.cpp)
+
+#   target_link_libraries(rmf_rxcpp_example_plan_path
+#     PUBLIC
+#       rmf_traffic::rmf_traffic
+#       rmf_rxcpp
+#   )
+
+# endif()