From be8d9d32ff15839892008bc25879944785de5046 Mon Sep 17 00:00:00 2001 From: Tully Foote Date: Thu, 9 Apr 2020 15:52:35 -0700 Subject: [PATCH] Reduce the internal timeouts Debugging #611 these timeouts were so long that the overall test timeout was being triggered before these internal timeouts were triggered. 1000 retries at 100ms -> 10 seconds each. I cut it down to 1 second for each to establish the connection. Signed-off-by: Tully Foote --- rcl/test/rcl/client_fixture.cpp | 4 ++-- rcl/test/rcl/service_fixture.cpp | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/rcl/test/rcl/client_fixture.cpp b/rcl/test/rcl/client_fixture.cpp index a8aa9b709..19292f604 100644 --- a/rcl/test/rcl/client_fixture.cpp +++ b/rcl/test/rcl/client_fixture.cpp @@ -179,7 +179,7 @@ int main(int argc, char ** argv) }); // Wait until server is available - if (!wait_for_server_to_be_available(&node, &client, 1000, 100)) { + if (!wait_for_server_to_be_available(&node, &client, 10, 100)) { RCUTILS_LOG_ERROR_NAMED(ROS_PACKAGE_NAME, "Server never became available"); return -1; } @@ -214,7 +214,7 @@ int main(int argc, char ** argv) memset(&client_response, 0, sizeof(test_msgs__srv__BasicTypes_Response)); test_msgs__srv__BasicTypes_Response__init(&client_response); - if (!wait_for_client_to_be_ready(&client, &context, 1000, 100)) { + if (!wait_for_client_to_be_ready(&client, &context, 10, 100)) { RCUTILS_LOG_ERROR_NAMED(ROS_PACKAGE_NAME, "Client never became ready"); return -1; } diff --git a/rcl/test/rcl/service_fixture.cpp b/rcl/test/rcl/service_fixture.cpp index 74a1d2795..3bed7a14a 100644 --- a/rcl/test/rcl/service_fixture.cpp +++ b/rcl/test/rcl/service_fixture.cpp @@ -164,7 +164,7 @@ int main(int argc, char ** argv) // Block until a client request comes in. - if (!wait_for_service_to_be_ready(&service, &context, 1000, 100)) { + if (!wait_for_service_to_be_ready(&service, &context, 10, 100)) { RCUTILS_LOG_ERROR_NAMED(ROS_PACKAGE_NAME, "Service never became ready"); return -1; }