From b734ed29d024b5fd9723c0ce1353d93735f9273e Mon Sep 17 00:00:00 2001 From: Dotta Date: Mon, 10 Aug 2026 07:29:40 +0000 Subject: [PATCH] test(runner): handle close rejection before teardown Attach the pending-request assertion before transport shutdown so Vitest never observes a transient unhandled rejection. Co-Authored-By: Paperclip --- .../src/drivers/codex/app-server-transport.test.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/paperclip-runner/src/drivers/codex/app-server-transport.test.ts b/packages/paperclip-runner/src/drivers/codex/app-server-transport.test.ts index 75ab1a717ce..0c8b67a361d 100644 --- a/packages/paperclip-runner/src/drivers/codex/app-server-transport.test.ts +++ b/packages/paperclip-runner/src/drivers/codex/app-server-transport.test.ts @@ -34,8 +34,9 @@ describe("Codex app-server transport limits", () => { await expect(transport.request("second", {})).rejects.toThrow( "codex app-server pending request limit 1 exceeded", ); + const firstRejection = expect(first).rejects.toThrow("codex app-server transport closed"); await transport.close(); - await expect(first).rejects.toThrow("codex app-server transport closed"); + await firstRejection; }); it("fails closed when queued notifications exceed their count bound", async () => {