Workaround for flaky test_play_services running with fastrtps (#1556)

- Workaround for #1311 - Gracefully stop player in TearDown() before calling rclcpp::shutdown() in destructor. Signed-off-by: Michael Orlov <[email protected]>
ros2 · Feb 2, 2024 · 54be479 · 54be479
1 parent a505a56
commit 54be479
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/rosbag2_transport/test/rosbag2_transport/test_play_services.cpp b/rosbag2_transport/test/rosbag2_transport/test_play_services.cpp
@@ -106,6 +106,12 @@ class PlaySrvsTest : public RosBag2PlayTestFixture
     ASSERT_TRUE(cli_stop_->wait_for_service(service_wait_timeout_));
   }
 
+  void TearDown() override
+  {
+    // Gracefully stop player before calling rclcpp::shutdown() in destructor
+    player_->stop();
+  }
+
   /// Call a service client, and expect it to successfully return within a reasonable timeout
   template<typename Srv>
   typename Srv::Response::SharedPtr successful_call(