From fb478d66126057ddd86cf776f24694c329aa5e46 Mon Sep 17 00:00:00 2001
From: NikkeDoy <niklas.siltala@gmail.com>
Date: Thu, 4 Jun 2026 19:00:38 +0300
Subject: [PATCH] :bug: | Fix SonarQube violations - use items() and remove
 assert True

---
 tests/conftest.py            | 4 ++--
 tests/test_discord_rpc.py    | 1 -
 tests/test_player_helpers.py | 3 ---
 3 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/tests/conftest.py b/tests/conftest.py
index ffc3e7e..3438f92 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -144,11 +144,11 @@ def _install_stubs() -> None:
         (MOD_QTWIDGETS, "QSizePolicy"): _stub("QSizePolicy"),
     }
 
-    for mod_name, attr_name in _PYSIDE_STUBS:
+    for (mod_name, attr_name), stub_value in _PYSIDE_STUBS.items():
         if mod_name not in sys.modules:
             mod = mock.MagicMock(__name__=mod_name.rsplit(".", 1)[-1])
             sys.modules[mod_name] = mod
-        setattr(sys.modules[mod_name], attr_name, _PYSIDE_STUBS[(mod_name, attr_name)])
+        setattr(sys.modules[mod_name], attr_name, stub_value)
 
     # ── yt-dlp ─────────────────────────────────────────────────────────
     if "yt_dlp" not in sys.modules:
diff --git a/tests/test_discord_rpc.py b/tests/test_discord_rpc.py
index b3bec10..4b7e4ea 100644
--- a/tests/test_discord_rpc.py
+++ b/tests/test_discord_rpc.py
@@ -230,7 +230,6 @@ class TestDiscordRPC:
         drpc._start_ts = 1000
         # Should not crash
         drpc._send_presence()
-        assert True
 
     def test_connect_oserror_handled(self):
         """_connect handles OSError gracefully (covers lines 119-122)."""
diff --git a/tests/test_player_helpers.py b/tests/test_player_helpers.py
index 195c5e9..2719ece 100644
--- a/tests/test_player_helpers.py
+++ b/tests/test_player_helpers.py
@@ -301,13 +301,11 @@ class TestCleanupPath:
         """_cleanup_path does nothing for non-existent paths."""
         fn = self._import()
         fn("/nonexistent/path/file.txt")  # should not raise
-        assert True
 
     def test_noop_for_none(self):
         """_cleanup_path does nothing when path is None."""
         fn = self._import()
         fn(None)
-        assert True
 
     def test_oserror_handled_gracefully(self, tmp_path, monkeypatch):
         """_cleanup_path handles OSError gracefully."""
@@ -320,7 +318,6 @@ class TestCleanupPath:
 
         monkeypatch.setattr(os, "unlink", _fail_unlink)
         fn(str(path))  # should not raise
-        assert True
 
 
 # ── _tmp_dir ───────────────────────────────────────────────────────────────────