Clarify queue cleanup and detached sync logs

2026-05-26 07:20:54 +02:00
parent f5c476162e
commit 28410b2946
7 changed files with 42 additions and 5 deletions
@@ -622,6 +622,35 @@ class DownloadQueueWorkerFailureTests(unittest.TestCase):
        self.assertEqual(send_webhook.call_args.args[2]["category"], "finished")
        self.assertEqual(send_webhook.call_args.args[2]["moved_files"], ["/tmp/example/Queue Show - S01E01.mp4"])

+    def test_successful_detached_download_logs_sync_skipped(self):
+        queue = APP.DownloadQueue(
+            lambda: {
+                "mode": "sub",
+                "quality": "best",
+                "download_dir": "/tmp/example",
+            },
+            watchlist_sync_fn=lambda _job: None,
+            start_worker=False,
+        )
+        job = queue.add({"query": "Queue Show", "title": "Queue Show", "season": "1", "episodes": "1"})
+        job["skip_watchlist_sync"] = True
+
+        class FakeProc:
+            pid = 4321
+            stdout = []
+
+            def wait(self):
+                return 0
+
+        with mock.patch.object(queue_jobs.subprocess, "Popen", return_value=FakeProc()), mock.patch.object(
+            queue_jobs, "finalize_library_files", return_value=["/tmp/example/Queue Show - S01E01.mp4"]
+        ):
+            queue._run_job(job)
+
+        stored = queue._find(job["id"])
+        self.assertIn("Watchlist sync skipped for detached job.", stored["log"])
+        self.assertNotIn("Watchlist download state synced.", stored["log"])
+

 class WatchlistRefreshAllTests(unittest.TestCase):
    def test_refresh_all_iterates_only_watching_and_planned_show_ids(self):