Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixup Executor::spin_all() regression fix #2517

Merged
merged 9 commits into from
May 2, 2024
Prev Previous commit
Next Next commit
Update rclcpp/test/rclcpp/executors/test_executors.cpp
Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Signed-off-by: jmachowinski <jmachowinski@users.noreply.github.com>
2 people authored and wjwwood committed Apr 30, 2024
commit 8d639e6f013cfb7e00162f912f7653087e93fe40
2 changes: 1 addition & 1 deletion rclcpp/test/rclcpp/executors/test_executors.cpp
Original file line number Diff line number Diff line change
@@ -891,7 +891,7 @@ class TestBusyWaiting : public ::testing::Test
EXPECT_LT(
std::chrono::system_clock::now() - start_time,
max_duration / 2)
<< "spin_some() took a long time to execute when it should have done "
<< "executor took a long time to execute when it should have done "
<< "nothing and should not have blocked either, but this could be a "
<< "false negative if the computer is really slow";