From 7c84b724b4d5ba75e1b0348765fc371dc7ef7e37 Mon Sep 17 00:00:00 2001 From: brawner Date: Mon, 3 Aug 2020 13:02:58 -0700 Subject: [PATCH] Increase timeouts for connext for long tests (#1253) * Increase timeouts for connext for long tests Signed-off-by: Stephen Brawner * Fix cmakelists Signed-off-by: Stephen Brawner --- rclcpp/test/CMakeLists.txt | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/rclcpp/test/CMakeLists.txt b/rclcpp/test/CMakeLists.txt index bf2fa94af8..840c5de0e3 100644 --- a/rclcpp/test/CMakeLists.txt +++ b/rclcpp/test/CMakeLists.txt @@ -17,7 +17,13 @@ rosidl_generate_interfaces(${PROJECT_NAME}_test_msgs SKIP_INSTALL ) -ament_add_gtest(test_allocator_memory_strategy rclcpp/strategies/test_allocator_memory_strategy.cpp) +# Increasing timeout because connext can take a long time to destroy nodes +# TODO(brawner) remove when destroying Node for Connext is resolved. See: +# https://github.com/ros2/rclcpp/issues/1250 +ament_add_gtest( + test_allocator_memory_strategy + rclcpp/strategies/test_allocator_memory_strategy.cpp + TIMEOUT 360) if(TARGET test_allocator_memory_strategy) ament_target_dependencies(test_allocator_memory_strategy "rcl" @@ -470,8 +476,13 @@ if(TARGET test_interface_traits) target_link_libraries(test_interface_traits ${PROJECT_NAME}) endif() -ament_add_gtest(test_executors rclcpp/executors/test_executors.cpp - APPEND_LIBRARY_DIRS "${append_library_dirs}") +# TODO(brawner) remove when destroying Node for Connext is resolved. See: +# https://github.com/ros2/rclcpp/issues/1250 +ament_add_gtest( + test_executors + rclcpp/executors/test_executors.cpp + APPEND_LIBRARY_DIRS "${append_library_dirs}" + TIMEOUT 180) if(TARGET test_executors) ament_target_dependencies(test_executors "rcl")