tests: refactoring

Author: Apolo151

test/conftest.py

@@ -3,6 +3,32 @@
 import pytest
 from manager.manager.manager import Manager
 
+# Patch Server and FileWatchdog to avoid starting real servers
+
+
+class DummyServer:
+    """A dummy server to simulate server behavior for testing purposes."""
+
+    def __init__(self, port, update_callback):
+        """
+        Initialize the DummyServer with a port and update callback.
+
+        Args:
+            port (int): The port number for the dummy server.
+            update_callback (callable): The callback function for updates.
+        """
+        self.port = port
+        self.update_callback = update_callback
+        self.started = False
+
+    def start(self):
+        """Simulate starting the dummy server."""
+        self.started = True
+
+    def stop(self):
+        """Simulate stopping the dummy server."""
+        self.started = False
+
 
 class DummyConsumer:
     """A dummy consumer to capture messages sent by the Manager."""
@@ -71,19 +97,6 @@ def terminate(self):
 
     monkeypatch.setattr("manager.manager.manager.LauncherWorld", DummyLauncherWorld)
 
-    # Patch Server and FileWatchdog to avoid starting real servers
-    class DummyServer:
-        def __init__(self, port, update_callback):
-            self.port = port
-            self.update_callback = update_callback
-            self.started = False
-
-        def start(self):
-            self.started = True
-
-        def stop(self):
-            self.started = False
-
     class DummyFileWatchdog:
         def __init__(self, path, update_callback):
             self.path = path

test/test_terminate_transitions.py

@@ -2,6 +2,7 @@
 
 import pytest
 from transitions import MachineError
+from conftest import DummyServer
 from test_utils import setup_manager_to_application_running
 from test_utils import setup_manager_to_world_ready
 from test_utils import setup_manager_to_visualization_ready
@@ -100,10 +101,13 @@ def test_terminate_visualization_valid(manager, monkeypatch):
 
 def test_terminate_visualization_invalid_machine_error(manager, monkeypatch):
     """
-    Test the valid terminate visualization transition in the Manager.
+    Test the invalid terminate visualization transition in the Manager.
 
     Ensure that the transition raises an error when executed from an invalid state.
     """
+    monkeypatch.setattr(
+        "manager.libs.applications.compatibility.server.Server", DummyServer
+    )
     # Ensure the manager is in a state where it can stop
     setup_manager_to_application_running(manager, monkeypatch)