test: simplify execute logic by removing unnecessary else branch

Anusha-Sharma-2 · Anusha-Sharma-2 · commit d4074c411583 · 2026-03-25T20:48:14.000-04:00
diff --git a/rclcpp/test/rclcpp/executors/test_waitable.cpp b/rclcpp/test/rclcpp/executors/test_waitable.cpp
@@ -78,8 +78,6 @@ TestWaitable::execute(const std::shared_ptr<void> &)
   count_++;
   if (nullptr != on_execute_callback_) {
     on_execute_callback_();
-  } else {
-    // Removed unnecessary sleep based on TODO comment; tests should rely on explicit synchronization instead
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -78,8 +78,6 @@ TestWaitable::execute(const std::shared_ptr<void> &)`
`78`	`78`	`count_++;`
`79`	`79`	`if (nullptr != on_execute_callback_) {`
`80`	`80`	`on_execute_callback_();`
`81`		`- } else {`
`82`		`- // Removed unnecessary sleep based on TODO comment; tests should rely on explicit synchronization instead`
`83`	`81`	`}`
`84`	`82`	`}`
`85`	`83`