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 <noreply@anthropic.com>
This commit is contained in:
Devin Foley
2026-03-24 12:33:11 -07:00
parent 0ce4134ce1
commit 4da83296a9

View File

@@ -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)}`,