Skip to content

Commit

Permalink
Fix controller manager tests
Browse files Browse the repository at this point in the history
  • Loading branch information
saikishor committed Oct 11, 2024
1 parent e411420 commit e1e2dc8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion controller_manager/test/test_controller_manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -398,7 +398,7 @@ TEST_P(TestControllerManagerWithUpdateRates, per_controller_equal_and_higher_upd
// if we do 2 times of the controller_manager update rate, the internal counter should be
// similarly incremented
EXPECT_EQ(test_controller->internal_counter, pre_internal_counter + (2 * cm_->get_update_rate()));
EXPECT_EQ(test_controller->get_update_rate(), ctrl_update_rate);
EXPECT_EQ(test_controller->get_update_rate(), cm_->get_update_rate());

auto deactivate_future = std::async(
std::launch::async, &controller_manager::ControllerManager::switch_controller, cm_,
Expand Down
2 changes: 1 addition & 1 deletion controller_manager/test/test_load_controller.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ TEST_F(TestLoadController, can_set_and_get_non_default_update_rate)
EXPECT_EQ(
lifecycle_msgs::msg::State::PRIMARY_STATE_INACTIVE, controller_if->get_lifecycle_state().id());

EXPECT_EQ(1337u, controller_if->get_update_rate());
EXPECT_EQ(cm_->get_update_rate(), controller_if->get_update_rate());
}

class TestLoadedController : public TestLoadController
Expand Down

0 comments on commit e1e2dc8

Please sign in to comment.