From cc85feeac786c1d0736a54f6ba1c79de7b7ab4b7 Mon Sep 17 00:00:00 2001 From: Ben Barclay Date: Thu, 20 Aug 2026 20:52:37 +1000 Subject: [PATCH] test(cron): native scalar-fallback test asserts the live delivery actually ran MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit seed_mock.assert_not_called() alone could pass for the wrong reason — a harness failure before delivery also leaves the seed uncalled. Assert the real adapter recorded exactly one live send to the origin chat, so the test pins the D6 thread-fallback decision, not an accidental no-delivery. --- tests/cron/test_scheduler.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tests/cron/test_scheduler.py b/tests/cron/test_scheduler.py index 062d741cdb..00db344c5e 100644 --- a/tests/cron/test_scheduler.py +++ b/tests/cron/test_scheduler.py @@ -2605,6 +2605,11 @@ class TestCronContinuableSurfaceInChannel: {"slack": {"cron_continuable_surface": "in_channel"}}, adapter, attach_to_session=False, ) + # Delivery must have gone through the LIVE adapter — otherwise a + # broken harness that never delivers would also leave the seed + # uncalled and this test would pass for the wrong reason. + assert len(adapter.sent) == 1 + assert adapter.sent[0][0] == "C123" # Capability absent -> surface fails safe to thread -> flat seed # must NOT run (D6). seed_mock.assert_not_called()