Make nice user interface to export controllers.

ros-controls · May 16, 2022 · 65bcd7c · 65bcd7c
1 parent 648d039
commit 65bcd7c
Show file tree

Hide file tree

Showing 9 changed files with 13 additions and 11 deletions.
diff --git a/controller_manager/test/test_chainable_controller/test_chainable_controller.cpp b/controller_manager/test/test_chainable_controller/test_chainable_controller.cpp
@@ -186,4 +186,5 @@ void TestChainableController::set_reference_interface_names(
 #include "pluginlib/class_list_macros.hpp"
 
 PLUGINLIB_EXPORT_CLASS(
-  test_chainable_controller::TestChainableController, controller_interface::ControllerInterfaceBase)
+  test_chainable_controller::TestChainableController,
+  controller_interface::ChainableControllerInterface)
diff --git a/controller_manager/test/test_chainable_controller/test_chainable_controller.xml b/controller_manager/test/test_chainable_controller/test_chainable_controller.xml
@@ -2,7 +2,7 @@
 
   <class name="controller_manager/test_chainable_controller"
          type="test_chainable_controller::TestChainableController"
-         base_class_type="controller_interface::ControllerInterfaceBase">
+         base_class_type="controller_interface::ChainableControllerInterface">
     <description>
       Controller used for testing
     </description>

diff --git a/controller_manager/test/test_controller/test_controller.cpp b/controller_manager/test/test_controller/test_controller.cpp
@@ -111,5 +111,4 @@ void TestController::set_state_interface_configuration(
 
 #include "pluginlib/class_list_macros.hpp"
 
-PLUGINLIB_EXPORT_CLASS(
-  test_controller::TestController, controller_interface::ControllerInterfaceBase)
+PLUGINLIB_EXPORT_CLASS(test_controller::TestController, controller_interface::ControllerInterface)
diff --git a/controller_manager/test/test_controller/test_controller.xml b/controller_manager/test/test_controller/test_controller.xml
@@ -1,6 +1,6 @@
 <library path="test_controller">
 
-  <class name="controller_manager/test_controller" type="test_controller::TestController" base_class_type="controller_interface::ControllerInterfaceBase">
+  <class name="controller_manager/test_controller" type="test_controller::TestController" base_class_type="controller_interface::ControllerInterface">
     <description>
       Controller used for testing
     </description>

diff --git a/controller_manager/test/test_controller_failed_init/test_controller_failed_init.cpp b/controller_manager/test/test_controller_failed_init/test_controller_failed_init.cpp
@@ -49,5 +49,4 @@ controller_interface::return_type TestControllerFailedInit::update(
 #include "pluginlib/class_list_macros.hpp"
 
 PLUGINLIB_EXPORT_CLASS(
-  test_controller_failed_init::TestControllerFailedInit,
-  controller_interface::ControllerInterfaceBase)
+  test_controller_failed_init::TestControllerFailedInit, controller_interface::ControllerInterface)
diff --git a/controller_manager/test/test_controller_failed_init/test_controller_failed_init.xml b/controller_manager/test/test_controller_failed_init/test_controller_failed_init.xml
@@ -1,6 +1,6 @@
 <library path="test_controller_failed_init">
 
-  <class name="controller_manager/test_controller_failed_init" type="test_controller_failed_init::TestControllerFailedInit" base_class_type="controller_interface::ControllerInterfaceBase">
+  <class name="controller_manager/test_controller_failed_init" type="test_controller_failed_init::TestControllerFailedInit" base_class_type="controller_interface::ControllerInterface">
     <description>
       Controller used for testing
     </description>

diff --git a/controller_manager/test/test_controller_manager_srvs.cpp b/controller_manager/test/test_controller_manager_srvs.cpp
@@ -48,7 +48,10 @@ TEST_F(TestControllerManagerSrvs, list_controller_types)
   ASSERT_EQ(controller_types, result->base_classes.size());
   ASSERT_THAT(result->types, ::testing::Contains(test_controller::TEST_CONTROLLER_CLASS_NAME));
   ASSERT_THAT(
-    result->base_classes, ::testing::Contains("controller_interface::ControllerInterfaceBase"));
+    result->base_classes, ::testing::Contains("controller_interface::ControllerInterface"));
+  ASSERT_THAT(
+    result->base_classes,
+    ::testing::Contains("controller_interface::ChainableControllerInterface"));
 }
 
 TEST_F(TestControllerManagerSrvs, list_controllers_srv)

diff --git a/controller_manager/test/test_controller_with_interfaces/test_controller_with_interfaces.cpp b/controller_manager/test/test_controller_with_interfaces/test_controller_with_interfaces.cpp
@@ -56,4 +56,4 @@ TestControllerWithInterfaces::on_cleanup(const rclcpp_lifecycle::State & /*previ
 
 PLUGINLIB_EXPORT_CLASS(
   test_controller_with_interfaces::TestControllerWithInterfaces,
-  controller_interface::ControllerInterfaceBase)
+  controller_interface::ControllerInterface)
diff --git a/controller_manager/test/test_controller_with_interfaces/test_controller_with_interfaces.xml b/controller_manager/test/test_controller_with_interfaces/test_controller_with_interfaces.xml
@@ -1,6 +1,6 @@
 <library path="test_controller_with_interfaces">
 
-  <class name="controller_manager/test_controller_with_interfaces" type="test_controller_with_interfaces::TestControllerWithInterfaces" base_class_type="controller_interface::ControllerInterfaceBase">
+  <class name="controller_manager/test_controller_with_interfaces" type="test_controller_with_interfaces::TestControllerWithInterfaces" base_class_type="controller_interface::ControllerInterface">
     <description>
       Controller used for testing
     </description>