From 00f783aa877b269d762e6d3a74c0bc83d1a4e56e Mon Sep 17 00:00:00 2001 From: Chris Lalancette Date: Tue, 3 Dec 2024 02:35:18 +0000 Subject: [PATCH] Re-enable executor test on rmw_connextdds. It supports the events executor now, so re-enable the test. Signed-off-by: Chris Lalancette --- 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 90508c732c..6df750d92c 100644 --- a/rclcpp/test/rclcpp/executors/test_executors.cpp +++ b/rclcpp/test/rclcpp/executors/test_executors.cpp @@ -861,13 +861,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();