fix: retry init and test

Author: sidey79

.github/workflows/test-pr.yml

@@ -19,8 +19,7 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install -r requirements.txt
-        pip install pytest-cov
+        pip install -r requirements.txt -r requirements-dev.txt
         
     - name: Run tests with coverage
       run: |

requirements-dev.txt

@@ -1,2 +1,4 @@
 pytest
 pytest-mock
+pytest-asyncio
+pytest-cov

signalduino/controller.py

@@ -199,7 +199,10 @@ def _reset_device(self) -> None:
             # Wait briefly to ensure port is released/device resets
             threading.Event().wait(2.0)
             self.connect()
-            self.initialize()
+            # Manuell die Initialisierung starten, ohne die Zähler zurückzusetzen.
+            # XQ sollte direkt gesendet werden.
+            self._send_xq()
+            self._start_init()
         except Exception as e:
             self.logger.error("Failed to reset device: %s", e)
 

tests/test_controller.py

@@ -156,7 +156,7 @@ def side_effect(*args, **kwargs):
         nonlocal call_count
         call_count += 1
         payload = kwargs.get("payload") or args[0] if args else None
-        
+
         if payload == "XQ":
             return None
         if payload == "V":
@@ -165,7 +165,8 @@ def side_effect(*args, **kwargs):
             return "V 3.5.0-dev SIGNALduino"
         return None
 
-    controller.send_command = Mock(side_effect=side_effect)
+    mocked_send_command = Mock(side_effect=side_effect)
+    controller.commands._send = mocked_send_command
 
     # Use very short intervals for testing by patching the imported constants in the controller module
     import signalduino.controller
@@ -178,7 +179,7 @@ def side_effect(*args, **kwargs):
 
     try:
         controller.initialize()
-        time.sleep(1.5)  # Wait for timers and retries
+        time.sleep(3.0)  # Wait for timers and retries (increased from 1.5s due to potential race condition)
 
         # Verify calls:
         # 1. XQ
@@ -189,7 +190,8 @@ def side_effect(*args, **kwargs):
         # Note: Depending on timing and implementation details, call count might vary slighty
         # but we expect at least XQ, failed V, successful V, XE.
 
-        calls = [c.kwargs.get('payload') or c.args[0] for c in controller.send_command.call_args_list]
+        calls = [c.kwargs.get('payload') or c.args[0] for c in mocked_send_command.call_args_list]
+
         assert "XQ" in calls
         assert calls.count("V") >= 2
         assert "XE" in calls