fix: missing decode

Author: p-gentili

checkbox-support/checkbox_support/helpers/audio_utils.py

@@ -168,7 +168,9 @@ def _load_pw_dump(self):
         for _ in range(3):
             try:
                 try:
-                    result = subprocess.check_output(["pw-dump"])
+                    result = subprocess.check_output(["pw-dump"]).decode(
+                        "utf-8"
+                    )
                     return json.loads(result)
                 except subprocess.CalledProcessError as e:
                     raise RuntimeError("Failed to run pw-dump: {}".format(e))

checkbox-support/checkbox_support/helpers/tests/test_audio_utils.py

@@ -92,17 +92,22 @@ def setUp(self):
     def test_load_pw_dump_success(self, mock_check_output):
         """Test successful pw-dump load."""
         mock_check_output.return_value = b'[{"id": 1, "type": "test"}]'
+        mock_check_output.return_value.decode = Mock(
+            return_value='[{"id": 1, "type": "test"}]'
+        )
         result = self.pipewire._load_pw_dump()
         self.assertEqual(result, [{"id": 1, "type": "test"}])
 
     @patch("time.sleep", Mock())
     @patch("checkbox_support.helpers.audio_utils.subprocess.check_output")
     def test_load_pw_dump_retry(self, mock_check_output):
         """Test pw-dump retry on failure."""
+        success_mock = Mock()
+        success_mock.decode = Mock(return_value='[{"id": 2, "type": "test3"}]')
         mock_check_output.side_effect = [
             subprocess.CalledProcessError(1, ""),
             json.JSONDecodeError("", "", 0),
-            b'[{"id": 2, "type": "test3"}]',
+            success_mock,
         ]
         result = self.pipewire._load_pw_dump()
         self.assertEqual(result, [{"id": 2, "type": "test3"}])