Don't expect RCL_RET_TIMEOUT to set an error string (#900)

Signed-off-by: Andrea Sorbini <[email protected]>
ros2 · Mar 12, 2021 · fc3f93b · fc3f93b
1 parent ce2a436
commit fc3f93b
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/rcl_action/test/rcl_action/test_action_communication.cpp b/rcl_action/test/rcl_action/test_action_communication.cpp
@@ -1148,8 +1148,11 @@ TEST_F(CLASSNAME(TestActionCommunication, RMW_IMPLEMENTATION), test_valid_feedba
 
     ret = rcl_wait(&this->wait_set, RCL_S_TO_NS(10));
     if (RCL_RET_OK != ret) {
-      EXPECT_TRUE(rcl_error_is_set());
-      rcl_reset_error();
+      // All non-OK retcodes should set an error string, except RCL_RET_TIMEOUT.
+      if (RCL_RET_TIMEOUT != ret) {
+        EXPECT_TRUE(rcl_error_is_set());
+        rcl_reset_error();
+      }
       continue;
     }