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]>
  • Loading branch information
clalancette authored Dec 3, 2024
1 parent 3310f9e commit d724536
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 @@ -861,13 +861,6 @@ TYPED_TEST(TestExecutors, releaseOwnershipEntityAfterSpinningCancel)
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
ExecutorType executor;
Expand Down

0 comments on commit d724536

Please sign in to comment.