Skip to content

Commit 710e5ea

Browse files
bmagyarchristophfroehlich
authored andcommitted
remove copy pasta remnants from comments
Co-authored-by: Christoph Fröhlich <[email protected]>
1 parent 7a9ad21 commit 710e5ea

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

controller_manager/test/test_release_interfaces.cpp

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -360,7 +360,9 @@ TEST_F(TestReleaseExclusiveInterfaces, test_exclusive_interface_switching_failur
360360
lifecycle_msgs::msg::State::PRIMARY_STATE_INACTIVE,
361361
abstract_test_controller2.c->get_lifecycle_state().id());
362362

363-
{ // Test starting the first controller
363+
{
364+
// Test starting the first controller
365+
// test_controller1 activation always fails
364366
RCLCPP_INFO(cm_->get_logger(), "Starting controller #1");
365367
std::vector<std::string> start_controllers = {controller_name1};
366368
std::vector<std::string> stop_controllers = {};
@@ -379,8 +381,9 @@ TEST_F(TestReleaseExclusiveInterfaces, test_exclusive_interface_switching_failur
379381
abstract_test_controller2.c->get_lifecycle_state().id());
380382
}
381383

382-
{ // Test starting the second controller when the first is running
383-
// Fails as they have the same command interface
384+
{
385+
// Test starting the second controller, interfaces should have been released
386+
// test_controller2 always successfully activates
384387
RCLCPP_INFO(cm_->get_logger(), "Starting controller #2");
385388
std::vector<std::string> start_controllers = {controller_name2};
386389
std::vector<std::string> stop_controllers = {};

0 commit comments

Comments
 (0)