From 4da83296a9d43839e118e8a173c2bbbd1671f28b Mon Sep 17 00:00:00 2001 From: Devin Foley Date: Tue, 24 Mar 2026 12:33:11 -0700 Subject: [PATCH] test(codex): move OPENAI_API_KEY stub to beforeEach for all tests Consolidate the env stub into beforeEach so the pre-existing cwd test is also isolated from host OPENAI_API_KEY, avoiding non-deterministic filesystem side effects. Co-Authored-By: Claude Opus 4.6 --- .../src/__tests__/codex-local-adapter-environment.test.ts | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/server/src/__tests__/codex-local-adapter-environment.test.ts b/server/src/__tests__/codex-local-adapter-environment.test.ts index 5cb3ae5e..ba92a224 100644 --- a/server/src/__tests__/codex-local-adapter-environment.test.ts +++ b/server/src/__tests__/codex-local-adapter-environment.test.ts @@ -1,4 +1,4 @@ -import { afterEach, describe, expect, it, vi } from "vitest"; +import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"; import fs from "node:fs/promises"; import os from "node:os"; import path from "node:path"; @@ -7,6 +7,9 @@ import { testEnvironment } from "@paperclipai/adapter-codex-local/server"; const itWindows = process.platform === "win32" ? it : it.skip; describe("codex_local environment diagnostics", () => { + beforeEach(() => { + vi.stubEnv("OPENAI_API_KEY", ""); + }); afterEach(() => { vi.unstubAllEnvs(); }); @@ -36,7 +39,6 @@ describe("codex_local environment diagnostics", () => { }); it("emits codex_native_auth_present when ~/.codex/auth.json exists and OPENAI_API_KEY is unset", async () => { - vi.stubEnv("OPENAI_API_KEY", ""); const root = path.join( os.tmpdir(), `paperclip-codex-auth-${Date.now()}-${Math.random().toString(16).slice(2)}`, @@ -69,7 +71,6 @@ describe("codex_local environment diagnostics", () => { }); it("emits codex_openai_api_key_missing when neither env var nor native auth exists", async () => { - vi.stubEnv("OPENAI_API_KEY", ""); const root = path.join( os.tmpdir(), `paperclip-codex-noauth-${Date.now()}-${Math.random().toString(16).slice(2)}`,