From b626ee75ae5c47057472edfef38707b4298d9fad Mon Sep 17 00:00:00 2001 From: Joey Krug Date: Sat, 21 Mar 2026 00:29:29 -0400 Subject: [PATCH] test: remove duplicate compaction fallback coverage --- .../pi-extensions/compaction-safeguard.test.ts | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/src/agents/pi-extensions/compaction-safeguard.test.ts b/src/agents/pi-extensions/compaction-safeguard.test.ts index 68b0243360e..cb90e1daf85 100644 --- a/src/agents/pi-extensions/compaction-safeguard.test.ts +++ b/src/agents/pi-extensions/compaction-safeguard.test.ts @@ -1480,7 +1480,7 @@ describe("compaction-safeguard extension model fallback", () => { expect(retrieved?.model).toEqual(model); }); - it("cancels compaction when both ctx.model and runtime.model are undefined", async () => { + it("falls back to built-in compaction when both ctx.model and runtime.model are undefined", async () => { const sessionManager = stubSessionManager(); // Do NOT set runtime.model (both ctx.model and runtime.model will be undefined) @@ -1501,19 +1501,6 @@ describe("compaction-safeguard extension model fallback", () => { expect(getApiKeyMock).not.toHaveBeenCalled(); }); - it("falls back to built-in compaction when model is undefined (returns undefined, not cancel)", async () => { - const sessionManager = stubSessionManager(); - // Do NOT set runtime model - const mockEvent = createCompactionEvent({ messageText: "test", tokensBefore: 500 }); - const { result, getApiKeyMock } = await runCompactionScenario({ - sessionManager, - event: mockEvent, - apiKey: null, - }); - expect(result).toBeUndefined(); - expect(getApiKeyMock).not.toHaveBeenCalled(); - }); - it("falls back to built-in compaction when API key is missing (returns undefined, not cancel)", async () => { const sessionManager = stubSessionManager(); const model = createAnthropicModelFixture();