Skip to content

Commit

Permalink
Fix spawner tests timeout on source builds (backport #1692) (#1697)
Browse files Browse the repository at this point in the history
* Fix spawner tests timeout (#1692)

(cherry picked from commit 079392b)

# Conflicts:
#	controller_manager/test/test_spawner_unspawner.cpp

* Fix merge conflicts

---------

Co-authored-by: Sai Kishor Kothakota <[email protected]>
Co-authored-by: Christoph Fröhlich <[email protected]>
  • Loading branch information
3 people authored Oct 8, 2024
1 parent f115f4f commit b009881
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions controller_manager/test/test_spawner_unspawner.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -221,6 +221,7 @@ TEST_F(TestLoadController, multi_ctrls_test_type_in_param)

TEST_F(TestLoadController, spawner_test_type_in_arg)
{
ControllerManagerRunner cm_runner(this);
// Provide controller type via -t argument
EXPECT_EQ(
call_spawner(
Expand All @@ -239,6 +240,7 @@ TEST_F(TestLoadController, unload_on_kill)
{
// Launch spawner with unload on kill
// timeout command will kill it after the specified time with signal SIGINT
ControllerManagerRunner cm_runner(this);
std::stringstream ss;
ss << "timeout --signal=INT 5 "
<< std::string(coveragepy_script) +
Expand Down Expand Up @@ -507,6 +509,7 @@ TEST_F(TestLoadControllerWithNamespacedCM, spawner_test_type_in_params_file)
const std::string test_file_path = ament_index_cpp::get_package_prefix("controller_manager") +
"/test/test_controller_spawner_with_type.yaml";

ControllerManagerRunner cm_runner(this);
// Provide controller type via the parsed file
EXPECT_EQ(
call_spawner(
Expand Down Expand Up @@ -567,6 +570,7 @@ TEST_F(
const std::string test_file_path = ament_index_cpp::get_package_prefix("controller_manager") +
"/test/test_controller_spawner_with_type.yaml";

ControllerManagerRunner cm_runner(this);
// Provide controller type via the parsed file
EXPECT_EQ(
call_spawner(
Expand Down

0 comments on commit b009881

Please sign in to comment.