From c468f05eaf53f5de177bd405dc48a3f64dafefbf Mon Sep 17 00:00:00 2001 From: Meta Vens Date: Thu, 1 Oct 2026 01:53:18 +0200 Subject: [PATCH] Watchdog notice reader test: the closure posted beside a late opening has one attempt The finally-sends test expected six attempts on all three journal rows, but the closure entry is first tried in the run that completes its opening, so it has one. Test-only fix; the previous commit's test failed on this line. The previous commit (69e12e3867a9e4136164fa24e8fd6e1c0b07e7e4) builds; its suite has one failing test, fixed here. Co-Authored-By: Claude Sonnet 5.5 --- tests/watchdog-notice.test.mjs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/watchdog-notice.test.mjs b/tests/watchdog-notice.test.mjs index 5ce633a..1817f7e 100644 --- a/tests/watchdog-notice.test.mjs +++ b/tests/watchdog-notice.test.mjs @@ -248,7 +248,7 @@ test("a notice that finally sends after many failures posts once", async () => { const before = (await w.posts()).length; for (const minute of [32, 100, 1000]) assert.equal(w.run({ env: at(minute) }).status, 0); assert.equal((await w.posts()).length, before, "completed is never posted again"); - assert.deepEqual((await w.status()).rows.map((row) => `${row.status}:${row.attempts}`), ["completed:6", "completed:6", "completed:6"]); + assert.deepEqual((await w.status()).rows.map((row) => `${row.status}:${row.attempts}`), ["completed:1", "completed:6", "completed:6"]); }); test("the reader never writes or locks the incident store", async () => {