Skip to content

Commit

Permalink
Re-enable executor test on rmw_connextdds. (ros2#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]>
Signed-off-by: HarunTeper <[email protected]>
  • Loading branch information
clalancette authored and HarunTeper committed Dec 9, 2024
1 parent c962f5e commit 2cd456e
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 2cd456e

Please sign in to comment.