Skip to content

Commit 5da7a31

Browse files
remove copy pasta remnants from comments
Co-authored-by: Christoph Fröhlich <[email protected]>
1 parent 3a19634 commit 5da7a31

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
@@ -238,7 +238,9 @@ TEST_F(TestReleaseExclusiveInterfaces, test_exclusive_interface_switching_failur
238238
lifecycle_msgs::msg::State::PRIMARY_STATE_INACTIVE,
239239
abstract_test_controller2.c->get_lifecycle_state().id());
240240

241-
{ // Test starting the first controller
241+
{
242+
// Test starting the first controller
243+
// test_controller1 activation always fails
242244
RCLCPP_INFO(cm_->get_logger(), "Starting controller #1");
243245
std::vector<std::string> start_controllers = {controller_name1};
244246
std::vector<std::string> stop_controllers = {};
@@ -257,8 +259,9 @@ TEST_F(TestReleaseExclusiveInterfaces, test_exclusive_interface_switching_failur
257259
abstract_test_controller2.c->get_lifecycle_state().id());
258260
}
259261

260-
{ // Test starting the second controller when the first is running
261-
// Fails as they have the same command interface
262+
{
263+
// Test starting the second controller, interfaces should have been released
264+
// test_controller2 always successfully activates
262265
RCLCPP_INFO(cm_->get_logger(), "Starting controller #2");
263266
std::vector<std::string> start_controllers = {controller_name2};
264267
std::vector<std::string> stop_controllers = {};

0 commit comments

Comments
 (0)