test(codex-local): isolate quota spawn test from host CODEX_HOME
After the mocked RPC spawn fails, getQuotaWindows() still calls readCodexToken(). Use an empty mkdtemp directory for CODEX_HOME for the duration of the test so we never read ~/.codex/auth.json or call WHAM.
This commit is contained in:
parent
c98af52590
commit
dc3aa8f31f
1 changed files with 29 additions and 1 deletions
|
|
@ -1,6 +1,9 @@
|
||||||
import { EventEmitter } from "node:events";
|
import { EventEmitter } from "node:events";
|
||||||
|
import fs from "node:fs";
|
||||||
|
import os from "node:os";
|
||||||
|
import path from "node:path";
|
||||||
import type { ChildProcess } from "node:child_process";
|
import type { ChildProcess } from "node:child_process";
|
||||||
import { describe, expect, it, vi, beforeEach } from "vitest";
|
import { describe, expect, it, vi, beforeEach, afterEach } from "vitest";
|
||||||
|
|
||||||
const { mockSpawn } = vi.hoisted(() => ({
|
const { mockSpawn } = vi.hoisted(() => ({
|
||||||
mockSpawn: vi.fn(),
|
mockSpawn: vi.fn(),
|
||||||
|
|
@ -34,8 +37,33 @@ function createChildThatErrorsOnMicrotask(err: Error): ChildProcess {
|
||||||
}
|
}
|
||||||
|
|
||||||
describe("CodexRpcClient spawn failures", () => {
|
describe("CodexRpcClient spawn failures", () => {
|
||||||
|
let previousCodexHome: string | undefined;
|
||||||
|
let isolatedCodexHome: string | undefined;
|
||||||
|
|
||||||
beforeEach(() => {
|
beforeEach(() => {
|
||||||
mockSpawn.mockReset();
|
mockSpawn.mockReset();
|
||||||
|
// After the RPC path fails, getQuotaWindows() calls readCodexToken() which
|
||||||
|
// reads $CODEX_HOME/auth.json (default ~/.codex). Point CODEX_HOME at an
|
||||||
|
// empty temp directory so we never hit real host auth or the WHAM network.
|
||||||
|
previousCodexHome = process.env.CODEX_HOME;
|
||||||
|
isolatedCodexHome = fs.mkdtempSync(path.join(os.tmpdir(), "paperclip-codex-spawn-test-"));
|
||||||
|
process.env.CODEX_HOME = isolatedCodexHome;
|
||||||
|
});
|
||||||
|
|
||||||
|
afterEach(() => {
|
||||||
|
if (isolatedCodexHome) {
|
||||||
|
try {
|
||||||
|
fs.rmSync(isolatedCodexHome, { recursive: true, force: true });
|
||||||
|
} catch {
|
||||||
|
/* ignore */
|
||||||
|
}
|
||||||
|
isolatedCodexHome = undefined;
|
||||||
|
}
|
||||||
|
if (previousCodexHome === undefined) {
|
||||||
|
delete process.env.CODEX_HOME;
|
||||||
|
} else {
|
||||||
|
process.env.CODEX_HOME = previousCodexHome;
|
||||||
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
it("does not crash the process when codex is missing; getQuotaWindows returns ok: false", async () => {
|
it("does not crash the process when codex is missing; getQuotaWindows returns ok: false", async () => {
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue