From 4d19dc867144f351c49e608da81ee123b967ddc3 Mon Sep 17 00:00:00 2001 From: ningding97 <810793091@qq.com> Date: Tue, 3 Mar 2026 07:15:54 +0800 Subject: [PATCH] test(cron): assert embedded model on last call to avoid bun ordering flake Bun runs can trigger multiple embedded agent invocations in a single cron turn (e.g. retries/fallbacks), making assertions against call[0] flaky. Assert against the last invocation instead. --- ...isolated-agent.uses-last-non-empty-agent-text-as.test.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/cron/isolated-agent.uses-last-non-empty-agent-text-as.test.ts b/src/cron/isolated-agent.uses-last-non-empty-agent-text-as.test.ts index cb5d03d6a..579d70dfc 100644 --- a/src/cron/isolated-agent.uses-last-non-empty-agent-text-as.test.ts +++ b/src/cron/isolated-agent.uses-last-non-empty-agent-text-as.test.ts @@ -427,7 +427,7 @@ describe("runCronIsolatedAgentTurn", () => { }); expect(res.status).toBe("ok"); - const call = vi.mocked(runEmbeddedPiAgent).mock.calls[0]?.[0] as { prompt?: string }; + const call = vi.mocked(runEmbeddedPiAgent).mock.calls.at(-1)?.[0] as { prompt?: string }; expect(call?.prompt).toContain("EXTERNAL, UNTRUSTED"); expect(call?.prompt).toContain("Hello"); }); @@ -449,7 +449,7 @@ describe("runCronIsolatedAgentTurn", () => { }); expect(res.status).toBe("ok"); - const call = vi.mocked(runEmbeddedPiAgent).mock.calls[0]?.[0] as { prompt?: string }; + const call = vi.mocked(runEmbeddedPiAgent).mock.calls.at(-1)?.[0] as { prompt?: string }; expect(call?.prompt).not.toContain("EXTERNAL, UNTRUSTED"); expect(call?.prompt).toContain("Hello"); }); @@ -486,7 +486,7 @@ describe("runCronIsolatedAgentTurn", () => { }); expect(res.status).toBe("ok"); - const call = vi.mocked(runEmbeddedPiAgent).mock.calls[0]?.[0] as { + const call = vi.mocked(runEmbeddedPiAgent).mock.calls.at(-1)?.[0] as { provider?: string; model?: string; };