Minor linting

Signed-off-by: Alberto Tudela <[email protected]>
grupo-avispa · Jul 26, 2024 · c2e15c3 · c2e15c3
1 parent dc73612
commit c2e15c3
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 19 deletions.
diff --git a/scitos2_core/include/scitos2_core/module.hpp b/scitos2_core/include/scitos2_core/module.hpp
@@ -48,8 +48,7 @@ class Module
    * @param name Name of the module
    */
   virtual void configure(
-    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent,
-    std::string name) = 0;
+    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent, std::string name) = 0;
 
   /**
    * @brief Method to cleanup resources.
@@ -194,8 +193,7 @@ class Module
    * @return std::string The value of the parameter
    */
   std::string get_mira_param(
-    const std::weak_ptr<mira::Authority> & authority,
-    std::string param_name)
+    const std::weak_ptr<mira::Authority> & authority, std::string param_name)
   {
     // Convert weak_ptr to shared_ptr
     auto sharedAuthority = authority.lock();

diff --git a/scitos2_modules/include/scitos2_modules/charger.hpp b/scitos2_modules/include/scitos2_modules/charger.hpp
@@ -64,8 +64,7 @@ class Charger : public scitos2_core::Module
    * @param name Name of plugin
    */
   void configure(
-    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent,
-    std::string name) override;
+    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent, std::string name) override;
 
   /**
    * @brief Cleanup the module state machine.
@@ -116,8 +115,7 @@ class Charger : public scitos2_core::Module
    * @return sensor_msgs::msg::BatteryState BatteryState for ROS
    */
   sensor_msgs::msg::BatteryState miraToRosBatteryState(
-    const mira::robot::BatteryState & state,
-    const mira::Time & timestamp);
+    const mira::robot::BatteryState & state, const mira::Time & timestamp);
 
   /**
    * @brief Convert MIRA ChargerStatus to ROS ChargerStatus.
@@ -127,8 +125,7 @@ class Charger : public scitos2_core::Module
    * @return scitos2_msgs::msg::ChargerStatus ChargerStatus for ROS
    */
   scitos2_msgs::msg::ChargerStatus miraToRosChargerStatus(
-    const uint8 & status,
-    const mira::Time & timestamp);
+    const uint8 & status, const mira::Time & timestamp);
 
   // MIRA Authority
   std::shared_ptr<mira::Authority> authority_;

diff --git a/scitos2_modules/include/scitos2_modules/display.hpp b/scitos2_modules/include/scitos2_modules/display.hpp
@@ -63,8 +63,7 @@ class Display : public scitos2_core::Module
    * @param name Name of plugin
    */
   void configure(
-    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent,
-    std::string name) override;
+    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent, std::string name) override;
 
   /**
    * @brief Cleanup the module state machine.

diff --git a/scitos2_modules/include/scitos2_modules/drive.hpp b/scitos2_modules/include/scitos2_modules/drive.hpp
@@ -83,8 +83,7 @@ class Drive : public scitos2_core::Module
    * @param name Name of plugin
    */
   void configure(
-    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent,
-    std::string name) override;
+    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent, std::string name) override;
 
   /**
    * @brief Cleanup the module state machine.

diff --git a/scitos2_modules/include/scitos2_modules/ebc.hpp b/scitos2_modules/include/scitos2_modules/ebc.hpp
@@ -61,8 +61,7 @@ class EBC : public scitos2_core::Module
    * @param name Name of plugin
    */
   void configure(
-    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent,
-    std::string name) override;
+    const rclcpp_lifecycle::LifecycleNode::WeakPtr & parent, std::string name) override;
 
   /**
    * @brief Cleanup the module state machine.

diff --git a/scitos2_modules/test/test_charger.cpp b/scitos2_modules/test/test_charger.cpp
@@ -51,9 +51,6 @@ TEST(ScitosChargerTest, configure) {
   // Cleaning up
   module->deactivate();
   module->cleanup();
-  node->deactivate();
-  node->cleanup();
-  node->shutdown();
   rclcpp::shutdown();
 }