From b7c09cba44708ab67cbc5d5b4b9e7464cdfb848e Mon Sep 17 00:00:00 2001 From: Jerry-Xin Date: Mon, 16 Mar 2026 16:09:23 +0800 Subject: [PATCH] fix(cron): improve test helper error reporting for lastEmbeddedCall Replace embedded expect assertion with an explicit throw so failures report at the call-site rather than inside the helper, making it easier to identify which test case triggered the problem. --- src/cron/isolated-agent.payload-model-override.test.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/cron/isolated-agent.payload-model-override.test.ts b/src/cron/isolated-agent.payload-model-override.test.ts index 62107b16a3e..ae264324531 100644 --- a/src/cron/isolated-agent.payload-model-override.test.ts +++ b/src/cron/isolated-agent.payload-model-override.test.ts @@ -28,8 +28,9 @@ const withTempHome = withTempCronHome; */ function lastEmbeddedCall(): { provider?: string; model?: string } { const calls = vi.mocked(runEmbeddedPiAgent).mock.calls; - expect(calls.length).toBeGreaterThan(0); - return calls.at(-1)?.[0] as { provider?: string; model?: string }; + const last = calls.at(-1)?.[0] as { provider?: string; model?: string } | undefined; + if (!last) throw new Error("runEmbeddedPiAgent was never called"); + return last; } async function runCronWithModel(