From 43b778aa04848e8da3216ee2ce7eb30d629946c6 Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Mon, 1 Jun 2026 22:09:02 -0400 Subject: [PATCH] fix(test): include 'scanned' in all backfill result assertions MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Same 'change shared shape, miss pinned tests' trap as cfa4fb4 — the prior commit updated only test_backfill_mixed_aggregate; three sibling tests in the same file also pinned the old {enqueued,ok,regenerated} shape without 'scanned' and CI bounced. --- tests/test_backfill_thumbnails.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/test_backfill_thumbnails.py b/tests/test_backfill_thumbnails.py index 878b1e7..58fb408 100644 --- a/tests/test_backfill_thumbnails.py +++ b/tests/test_backfill_thumbnails.py @@ -125,7 +125,7 @@ def test_backfill_null_path_enqueued(db_sync, tmp_path, monkeypatch): ) result = m.backfill_thumbnails() - assert result == {"enqueued": 1, "ok": 0, "regenerated": 0} + assert result == {"scanned": 1, "enqueued": 1, "ok": 0, "regenerated": 0} assert delayed == [rec.id] @@ -148,7 +148,7 @@ def test_backfill_missing_file_clears_and_enqueues(db_sync, tmp_path, monkeypatc result = m.backfill_thumbnails() db_sync.expire_all() - assert result == {"enqueued": 1, "ok": 0, "regenerated": 1} + assert result == {"scanned": 1, "enqueued": 1, "ok": 0, "regenerated": 1} assert delayed == [rec.id] assert db_sync.get(ImageRecord, rec.id).thumbnail_path is None @@ -170,7 +170,7 @@ def test_backfill_valid_jpeg_skipped(db_sync, tmp_path, monkeypatch): result = m.backfill_thumbnails() db_sync.expire_all() - assert result == {"enqueued": 0, "ok": 1, "regenerated": 0} + assert result == {"scanned": 1, "enqueued": 0, "ok": 1, "regenerated": 0} assert delayed == [] assert db_sync.get(ImageRecord, rec.id).thumbnail_path == str(thumb) @@ -191,7 +191,7 @@ def test_backfill_valid_png_skipped(db_sync, tmp_path, monkeypatch): ) result = m.backfill_thumbnails() - assert result == {"enqueued": 0, "ok": 1, "regenerated": 0} + assert result == {"scanned": 1, "enqueued": 0, "ok": 1, "regenerated": 0} assert delayed == [] @@ -212,7 +212,7 @@ def test_backfill_corrupt_magic_clears_and_enqueues(db_sync, tmp_path, monkeypat result = m.backfill_thumbnails() db_sync.expire_all() - assert result == {"enqueued": 1, "ok": 0, "regenerated": 1} + assert result == {"scanned": 1, "enqueued": 1, "ok": 0, "regenerated": 1} assert delayed == [rec.id] assert db_sync.get(ImageRecord, rec.id).thumbnail_path is None