Skip to content

Commit 74162b1

Browse files
Re-enable executor test on rmw_connextdds. (#2693) (#2695)
It supports the events executor now, so re-enable the test. Signed-off-by: Chris Lalancette <[email protected]> (cherry picked from commit d724536) Co-authored-by: Chris Lalancette <[email protected]>
1 parent e217532 commit 74162b1

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

rclcpp/test/rclcpp/executors/test_executors.cpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -799,13 +799,6 @@ TYPED_TEST(TestExecutors, releaseOwnershipEntityAfterSpinningCancel)
799799
TYPED_TEST(TestExecutors, testRaceDropCallbackGroupFromSecondThread)
800800
{
801801
using ExecutorType = TypeParam;
802-
// rmw_connextdds doesn't support events-executor
803-
if (
804-
std::is_same<ExecutorType, rclcpp::experimental::executors::EventsExecutor>() &&
805-
std::string(rmw_get_implementation_identifier()).find("rmw_connextdds") == 0)
806-
{
807-
GTEST_SKIP();
808-
}
809802

810803
// Create an executor
811804
ExecutorType executor;

0 commit comments

Comments
 (0)