From 1d8659fdcbb5a99f324d49e49546ff49cd3e9d02 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 8 May 2026 12:35:30 +0100 Subject: [PATCH] test: clarify cron store validator assertions --- src/cron/service/store.load-missing-session-target.test.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/cron/service/store.load-missing-session-target.test.ts b/src/cron/service/store.load-missing-session-target.test.ts index bdcaade72c9..88a22044a27 100644 --- a/src/cron/service/store.load-missing-session-target.test.ts +++ b/src/cron/service/store.load-missing-session-target.test.ts @@ -62,7 +62,7 @@ describe("cron service store load: missing sessionTarget", () => { toolsAllow: ["exec"], }); expect(job.state.nextRunAtMs).toEqual(expect.any(Number)); - expect(() => assertSupportedJobSpec(job)).not.toThrow(); + expect(assertSupportedJobSpec(job)).toBeUndefined(); }); it('defaults missing sessionTarget to "main" for systemEvent payloads', async () => { @@ -85,7 +85,7 @@ describe("cron service store load: missing sessionTarget", () => { const job = findJobOrThrow(state, "missing-session-target-system-event"); expect(job.sessionTarget).toBe("main"); - expect(() => assertSupportedJobSpec(job)).not.toThrow(); + expect(assertSupportedJobSpec(job)).toBeUndefined(); }); it('defaults missing sessionTarget to "isolated" for agentTurn payloads', async () => { @@ -108,7 +108,7 @@ describe("cron service store load: missing sessionTarget", () => { const job = findJobOrThrow(state, "missing-session-target-agent-turn"); expect(job.sessionTarget).toBe("isolated"); - expect(() => assertSupportedJobSpec(job)).not.toThrow(); + expect(assertSupportedJobSpec(job)).toBeUndefined(); }); it("assertSupportedJobSpec throws a clear error when sessionTarget is missing", () => {