From 5917f2df6d1c7e0bb8d09ed065d8c7528f478745 Mon Sep 17 00:00:00 2001 From: Vito Sansevero Date: Thu, 29 Oct 2026 14:24:57 -0700 Subject: [PATCH] test(images): record the return route in the legacy contract snapshot; removed_total in the workspace keys (#498) The scan runner passes returned only when set, so a stand-in events function without that keyword still runs. --- py/api/admin.py | 3 ++- tests/fixtures/legacy_contract_snapshot.json | 1 + tests/test_api_images.py | 2 ++ 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/py/api/admin.py b/py/api/admin.py index 4637afee..97765b52 100644 --- a/py/api/admin.py +++ b/py/api/admin.py @@ -1422,7 +1422,8 @@ class AdminRoutesMixin: async def _run_scan_job(self, job, output_dirs=None, returned=None): final = None - async for payload in self._scan_images_events(job, output_dirs, returned): + extra = {} if returned is None else {"returned": returned} + async for payload in self._scan_images_events(job, output_dirs, **extra): job.publish(payload) final = payload.get("type") state = SCAN_FINAL_STATES.get(final, "finished") diff --git a/tests/fixtures/legacy_contract_snapshot.json b/tests/fixtures/legacy_contract_snapshot.json index 4d3dc248..2e9e744d 100644 --- a/tests/fixtures/legacy_contract_snapshot.json +++ b/tests/fixtures/legacy_contract_snapshot.json @@ -122,6 +122,7 @@ "POST /prompt_manager/images/generate-thumbnails", "POST /prompt_manager/images/generate-thumbnails/progress", "POST /prompt_manager/images/link", + "POST /prompt_manager/images/removed/return", "POST /prompt_manager/images/rescope", "POST /prompt_manager/images/reveal", "POST /prompt_manager/jobs/autotag", diff --git a/tests/test_api_images.py b/tests/test_api_images.py index edb3c363..c1772f66 100644 --- a/tests/test_api_images.py +++ b/tests/test_api_images.py @@ -3020,6 +3020,8 @@ class TestSharedListing(ImageAPITestCase): "limit", "has_more", "next_cursor", + # #498: Show removed images (N). + "removed_total", }, ) _, folders = await self._json("GET", "/prompt_manager/gallery/subfolders")