From 43e3a91c847d96e38f2afce610b50155db2399ad Mon Sep 17 00:00:00 2001 From: Seongho Bae Date: Fri, 4 Sep 2026 14:26:59 +0900 Subject: [PATCH] fix(tests): sync stale hourly-cron assertion with #1828's daily sweep schedule #1828 ("reduce organization sweep polling") intentionally moved the scheduler's missed-event recovery and org-wide sweep from hourly ("30 * * * *" / "0 * * * *") to daily ("47 3 * * *" / "17 3 * * *"), but left test_reconciled_scheduler_preserves_current_main_control_plane_fixes asserting the old hourly cron -- breaking the required test suite for every .github PR regardless of diff, independently of the #1826/#1823 staleness #1829 just fixed. Reproduced on fresh main (1 failed, 2804 passed) after merging #1829; fixed and reverified (2805 passed, coverage 100%, interrogate 100%). Co-Authored-By: Claude Sonnet 5 --- tests/test_queue_cancellation_scheduler_contract.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/test_queue_cancellation_scheduler_contract.py b/tests/test_queue_cancellation_scheduler_contract.py index 70f9d75e8c..b20f0493d4 100644 --- a/tests/test_queue_cancellation_scheduler_contract.py +++ b/tests/test_queue_cancellation_scheduler_contract.py @@ -43,7 +43,8 @@ def test_revalidation_helper_is_executable_and_temp_writer_is_retired() -> None: def test_reconciled_scheduler_preserves_current_main_control_plane_fixes() -> None: workflow = WORKFLOW.read_text(encoding="utf-8") - assert '- cron: "0 * * * *"' in workflow + assert '- cron: "47 3 * * *"' in workflow + assert '- cron: "17 3 * * *"' in workflow assert '*/15 * * * *' not in workflow assert workflow.count("runs-on: ubuntu-24.04") >= 2 scan_job = workflow.split(" scan-pr-queue:", 1)[1].split(" org-queue-sweep:", 1)[0]