Add audio playback test to hardware suite

Split test_playback_completes into test_playback_silent_completes and
test_playback_with_audio_completes so the sound replay path through
reachy_mini.media.play_sound() is exercised during hardware testing.

TESTING.md

@@ -123,7 +123,7 @@ pytest tests/ --browser chromium
 |-------|------:|-------------|
 | TestHardwareStartup | 1 | Robot reaches idle after startup |
 | TestHardwareRecording | 2 | Record and verify motion capture |
-| TestHardwarePlayback | 2 | Playback completes and returns to idle |
+| TestHardwarePlayback | 3 | Playback with and without audio |
 
 ## View the matrix without running tests
 

tests/run_tests.py

@@ -57,7 +57,7 @@ TEST_CLASS_DESCRIPTIONS: dict[str, tuple[str, str]] = {
     # Hardware
     "TestHardwareStartup": ("hardware", "Robot reaches idle after startup"),
     "TestHardwareRecording": ("hardware", "Record and verify motion capture"),
-    "TestHardwarePlayback": ("hardware", "Playback completes and returns to idle"),
+    "TestHardwarePlayback": ("hardware", "Playback with and without audio"),
 }
 
 

tests/test_hardware.py

@@ -184,30 +184,50 @@ class TestHardwareRecording:
 
 
 class TestHardwarePlayback:
-    def test_playback_completes(self, base_url: str):
-        """Play back a recorded move, verify mode returns to idle."""
+    def test_playback_silent_completes(self, base_url: str):
+        """Play back a silent move, verify mode returns to idle."""
         import httpx
 
-        # Ensure we have at least one move to play
         state = httpx.get(f"{base_url}/api/state", timeout=5).json()
-        moves = state["moves"]
-        assert len(moves) > 0, "No moves available for playback test"
+        move = next((m for m in state["moves"] if not m["has_audio"]), None)
+        assert move is not None, "No silent move available for playback test"
 
-        move_id = moves[0]["id"]
-        duration = moves[0]["duration"]
+        resp = httpx.post(
+            f"{base_url}/api/play",
+            json={"move_id": move["id"]},
+            timeout=5,
+        )
+        assert resp.status_code == 200
+
+        _wait_for_mode(base_url, "idle", timeout=move["duration"] + 15)
+
+        state = httpx.get(f"{base_url}/api/state", timeout=5).json()
+        assert state["mode"] == "idle"
+
+    def test_playback_with_audio_completes(self, base_url: str):
+        """Play back a move that has audio, verify sound path is exercised."""
+        import httpx
+
+        state = httpx.get(f"{base_url}/api/state", timeout=5).json()
+        move = next((m for m in state["moves"] if m["has_audio"]), None)
+        assert move is not None, (
+            "No audio move available — test_record_with_audio must run first"
+        )
 
         resp = httpx.post(
             f"{base_url}/api/play",
-            json={"move_id": move_id},
+            json={"move_id": move["id"]},
             timeout=5,
         )
         assert resp.status_code == 200
 
-        # Wait for playback to complete (duration + buffer)
-        _wait_for_mode(base_url, "idle", timeout=duration + 15)
+        _wait_for_mode(base_url, "idle", timeout=move["duration"] + 15)
 
         state = httpx.get(f"{base_url}/api/state", timeout=5).json()
         assert state["mode"] == "idle"
+        assert "finished" in state["message"].lower(), (
+            f"Expected 'Finished playing' message, got: {state['message']}"
+        )
 
     def test_record_and_delete(self, base_url: str):
         """Record a move, then delete it and verify file is removed."""