fix(TestExecutors): Fixed race in testcase

Signed-off-by: Janosch Machowinski <[email protected]>
ros2 · Dec 18, 2024 · 83b01e2 · 83b01e2
1 parent f0285e6
commit 83b01e2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/rclcpp/test/rclcpp/executors/test_executors.cpp b/rclcpp/test/rclcpp/executors/test_executors.cpp
@@ -749,8 +749,8 @@ TYPED_TEST(TestExecutors, notifyTwiceWhileSpinning)
   this->publisher->publish(test_msgs::msg::Empty());
   start = std::chrono::steady_clock::now();
   while (
-    sub1_msg_count == 1 &&
-    sub2_msg_count == 0 &&
+    (sub1_msg_count != 2 ||
+    sub2_msg_count != 1) &&
     (std::chrono::steady_clock::now() - start) < 10s)
   {
     std::this_thread::sleep_for(1ms);