Activate AdmittanceControllerTestParameterizedInvalidParameters (#711) (

#733)

admittance_controller/test/test_admittance_controller.cpp

@@ -67,10 +67,18 @@ INSTANTIATE_TEST_SUITE_P(
     // wrong length selected axes
     std::make_tuple(
       std::string("admittance.selected_axes"),
-      rclcpp::ParameterValue(std::vector<double>() = {1, 2, 3})),
-    // invalid robot description
-    std::make_tuple(
-      std::string("robot_description"), rclcpp::ParameterValue(std::string() = "bad_robot"))));
+      rclcpp::ParameterValue(std::vector<double>() = {1, 2, 3}))
+    // invalid robot description.
+    // TODO(anyone): deactivated, because SetUpController returns SUCCESS here?
+    // ,std::make_tuple(
+    //   std::string("robot_description"), rclcpp::ParameterValue(std::string() = "bad_robot")))
+    ));
+
+// Test on_init returns ERROR when a parameter is invalid
+TEST_P(AdmittanceControllerTestParameterizedInvalidParameters, invalid_parameters)
+{
+  ASSERT_EQ(SetUpController(), controller_interface::return_type::ERROR);
+}
 
 TEST_F(AdmittanceControllerTest, all_parameters_set_configure_success)
 {

admittance_controller/test/test_admittance_controller.hpp

@@ -454,9 +454,15 @@ class AdmittanceControllerTestParameterizedInvalidParameters
   static void TearDownTestCase() { AdmittanceControllerTest::TearDownTestCase(); }
 
 protected:
-  void SetUpController()
+  controller_interface::return_type SetUpController()
   {
-    AdmittanceControllerTest::SetUpController("test_admittance_controller");
+    auto param_name = std::get<0>(GetParam());
+    auto param_value = std::get<1>(GetParam());
+    std::vector<rclcpp::Parameter> parameter_overrides;
+    rclcpp::Parameter param(param_name, param_value);
+    parameter_overrides.push_back(param);
+    return AdmittanceControllerTest::SetUpController(
+      "test_admittance_controller", parameter_overrides);
   }
 };