sonarcloud fix

Author: NeptuneHub

tests/unit/test_memory_utils.py

@@ -222,7 +222,7 @@ def test_non_memory_error_reraises_same_object(self):
 
     def test_memory_error_triggers_cleanup_and_returns_retry_result(self):
         """A BFCArena error calls cleanup once and returns the retry result."""
-        err = Exception("BFCArena failed")
+        err = RuntimeError("BFCArena failed")
         cleanup = Mock()
         retry = Mock(return_value="retried")
 
@@ -236,7 +236,7 @@ def test_memory_error_triggers_cleanup_and_returns_retry_result(self):
 
     def test_cpu_fallback_returns_result_session_provider_tuple(self):
         """CPU fallback returns (result, new_session, provider)."""
-        err = Exception("BFCArena failed")
+        err = RuntimeError("BFCArena failed")
         session_mock = MagicMock()
         creator = Mock(return_value=(session_mock, "CPUExecutionProvider"))
         retry = Mock(return_value="r")
@@ -255,7 +255,7 @@ def test_cpu_fallback_returns_result_session_provider_tuple(self):
 
     def test_retry_failure_propagates_retry_exception(self):
         """If the retry itself fails, that exception propagates."""
-        err = Exception("Failed to allocate memory for requested buffer")
+        err = RuntimeError("Failed to allocate memory for requested buffer")
         retry_err = RuntimeError("still failing")
         retry = Mock(side_effect=retry_err)
 
@@ -266,9 +266,9 @@ def test_retry_failure_propagates_retry_exception(self):
 
     def test_memory_error_without_retry_or_fallback_reraises_original(self):
         """A memory error with no retry and no fallback re-raises the original."""
-        err = Exception("out of memory")
+        err = RuntimeError("out of memory")
 
-        with pytest.raises(Exception) as excinfo:
+        with pytest.raises(RuntimeError) as excinfo:
             handle_onnx_memory_error(err, "test context")
 
         assert excinfo.value is err

tests/unit/test_radius_walk_helper.py

@@ -19,7 +19,7 @@ class TestAvoidTripleAdjacent:
     def test_triple_swapped_with_later_different_artist(self):
         ids = ['a1', 'a2', 'a3', 'b1']
         id_to_author = {'a1': 'A', 'a2': 'A', 'a3': 'A', 'b1': 'B'}
-        original = list(ids)
+        original = ids.copy()
 
         result = avoid_triple_adjacent(ids, id_to_author)
 

tests/unit/test_taskqueue.py

@@ -36,11 +36,11 @@ def test_redis_socket_options_tls_url_keeps_keepalive():
 
 def test_build_embedded_redis_argv_binary_flags_and_url():
     argv, url = taskqueue.build_embedded_redis_argv(
-        '/usr/bin/redis-server', '/tmp/r.sock', '/data'
+        '/usr/bin/redis-server', '/var/lib/audiomuse/redis.sock', '/data'
     )
     assert argv[0] == '/usr/bin/redis-server'
     for flag, value in (
-        ('--unixsocket', '/tmp/r.sock'),
+        ('--unixsocket', '/var/lib/audiomuse/redis.sock'),
         ('--unixsocketperm', '700'),
         ('--port', '0'),
         ('--save', ''),
@@ -49,7 +49,7 @@ def test_build_embedded_redis_argv_binary_flags_and_url():
     ):
         idx = argv.index(flag)
         assert argv[idx + 1] == value
-    assert url == 'unix:///tmp/r.sock?db=0'
+    assert url == 'unix:///var/lib/audiomuse/redis.sock?db=0'
 
 
 def test_redis_conn_connection_kwargs():