From e6d11262a58505a1d44793bcfbf77acce74c697c Mon Sep 17 00:00:00 2001 From: Chris Lalancette Date: Tue, 3 Dec 2024 03:03:03 -0500 Subject: [PATCH] Re-enable executor test on rmw_connextdds. (#2693) It supports the events executor now, so re-enable the test. Signed-off-by: Chris Lalancette (cherry picked from commit d7245365ed867db9b309ed3efbfb0391bda09bd5) --- rclcpp/test/rclcpp/executors/test_executors.cpp | 7 ------- 1 file changed, 7 deletions(-) diff --git a/rclcpp/test/rclcpp/executors/test_executors.cpp b/rclcpp/test/rclcpp/executors/test_executors.cpp index f34f81e8ab..26ee3bdfb3 100644 --- a/rclcpp/test/rclcpp/executors/test_executors.cpp +++ b/rclcpp/test/rclcpp/executors/test_executors.cpp @@ -799,13 +799,6 @@ TYPED_TEST(TestExecutors, release_ownership_entity_after_spinning_cancel) TYPED_TEST(TestExecutors, testRaceDropCallbackGroupFromSecondThread) { using ExecutorType = TypeParam; - // rmw_connextdds doesn't support events-executor - if ( - std::is_same() && - std::string(rmw_get_implementation_identifier()).find("rmw_connextdds") == 0) - { - GTEST_SKIP(); - } // Create an executor auto executor = std::make_shared();