Skip to content

Commit

Permalink
Re-enable executor test on rmw_connextdds. (#2693)
Browse files Browse the repository at this point in the history
It supports the events executor now, so re-enable the test.

Signed-off-by: Chris Lalancette <[email protected]>
(cherry picked from commit d724536)
  • Loading branch information
clalancette authored and mergify[bot] committed Dec 3, 2024
1 parent df09c6e commit e6d1126
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions rclcpp/test/rclcpp/executors/test_executors.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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<ExecutorType, rclcpp::experimental::executors::EventsExecutor>() &&
std::string(rmw_get_implementation_identifier()).find("rmw_connextdds") == 0)
{
GTEST_SKIP();
}

// Create an executor
auto executor = std::make_shared<ExecutorType>();
Expand Down

0 comments on commit e6d1126

Please sign in to comment.