style: renamed ros2launcher

Author: jmatejcz

src/rai_sim/rai_sim/launch_manager.py

@@ -20,7 +20,7 @@
 from launch import LaunchDescription, LaunchService
 
 
-class Ros2LaunchManager:
+class ROS2LaunchManager:
     def __init__(self) -> None:
         self._stop_event: Optional[Event] = None
         self._process: Optional[multiprocessing.Process] = None

src/rai_sim/rai_sim/o3de/o3de_bridge.py

@@ -34,7 +34,7 @@
 from tf2_geometry_msgs import do_transform_pose, do_transform_pose_stamped
 
 from rai_interfaces.srv import ManipulatorMoveTo
-from rai_sim.launch_manager import Ros2LaunchManager
+from rai_sim.launch_manager import ROS2LaunchManager
 from rai_sim.simulation_bridge import (
     Entity,
     SceneConfig,
@@ -67,7 +67,7 @@ def __init__(
     ):
         super().__init__(logger=logger)
         self.connector = connector
-        self.manager = Ros2LaunchManager()
+        self.manager = ROS2LaunchManager()
         self.current_sim_process = None
         self.current_binary_path = None
 

tests/rai_sim/test_o3de_bridge.py

@@ -35,7 +35,7 @@
 from rclpy.node import Node
 from rclpy.qos import QoSProfile
 
-from rai_sim.launch_manager import Ros2LaunchManager
+from rai_sim.launch_manager import ROS2LaunchManager
 from rai_sim.o3de.o3de_bridge import O3DExROS2Bridge, O3DExROS2SimulationConfig
 from rai_sim.simulation_bridge import Entity, SceneConfig, SpawnedEntity
 
@@ -69,12 +69,12 @@ def test_load_scene_config(sample_base_yaml_config: Path):
 
 
 class TestO3DExROS2Bridge(unittest.TestCase):
-    @patch("rai_sim.o3de.o3de_bridge.Ros2LaunchManager")
+    @patch("rai_sim.o3de.o3de_bridge.ROS2LaunchManager")
     def setUp(self, mock_launch_manager_class):
         self.mock_connector = MagicMock(spec=ROS2Connector)
         self.mock_logger = MagicMock()
 
-        self.mock_launch_manager = MagicMock(spec=Ros2LaunchManager)
+        self.mock_launch_manager = MagicMock(spec=ROS2LaunchManager)
         mock_launch_manager_class.return_value = self.mock_launch_manager
 
         self.bridge = O3DExROS2Bridge(
@@ -327,9 +327,9 @@ def test_get_transform_signature(self):
             "timeout_sec": float,
         }
 
-        assert list(parameters.keys()) == list(expected_params.keys()), (
-            f"Parameter names do not match, expected: {list(expected_params.keys())}, got: {list(parameters.keys())}"
-        )
+        assert list(parameters.keys()) == list(
+            expected_params.keys()
+        ), f"Parameter names do not match, expected: {list(expected_params.keys())}, got: {list(parameters.keys())}"
 
         for param_name, expected_type in expected_params.items():
             param = parameters[param_name]
@@ -340,9 +340,9 @@ def test_get_transform_signature(self):
             )
 
         # Check return type explicitly
-        assert signature.return_annotation is ROS2TransformStamped, (
-            f"Return type is incorrect, expected: TransformStamped, got: {signature.return_annotation}"
-        )
+        assert (
+            signature.return_annotation is ROS2TransformStamped
+        ), f"Return type is incorrect, expected: TransformStamped, got: {signature.return_annotation}"
 
     def test_send_message_signature(self):
         signature = inspect.signature(self.connector.send_message)
@@ -394,9 +394,9 @@ def test_get_topics_names_and_types_signature(self):
 
         expected_params: dict[str, type] = {}
 
-        assert list(parameters.keys()) == list(expected_params.keys()), (
-            f"Parameter names do not match, expected: {list(expected_params.keys())}, got: {list(parameters.keys())}"
-        )
+        assert list(parameters.keys()) == list(
+            expected_params.keys()
+        ), f"Parameter names do not match, expected: {list(expected_params.keys())}, got: {list(parameters.keys())}"
 
         for param_name, expected_type in expected_params.items():
             param = parameters[param_name]