From f1b0a5352002da98e6a3f4fdefea540e790a6a31 Mon Sep 17 00:00:00 2001 From: Flea Flicker Date: Wed, 5 Aug 2026 10:04:31 +0000 Subject: [PATCH] test(GRO-2652): stub fetch in auth tests to fix OIDC discovery timeout flake AbortSignal.timeout(5000) in initAuth's discovery fetch races with vitest's 5000ms default test timeout, causing intermittent failures on CI push runs. Stub fetch to return ok:false instantly so tests complete in <1s. Co-Authored-By: Paperclip --- apps/api/src/__tests__/auth.test.ts | 5 +++++ src/__tests__/auth.test.ts | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/apps/api/src/__tests__/auth.test.ts b/apps/api/src/__tests__/auth.test.ts index 5446a3e..c338680 100644 --- a/apps/api/src/__tests__/auth.test.ts +++ b/apps/api/src/__tests__/auth.test.ts @@ -69,9 +69,14 @@ describe("auth init", () => { beforeEach(() => { dbSelectResult = []; vi.clearAllMocks(); + // Stub fetch so OIDC discovery requests resolve instantly during tests. + // Without this, AbortSignal.timeout(5000) in auth.ts races with vitest's + // 5000ms default test timeout and causes flaky failures. + vi.stubGlobal("fetch", vi.fn().mockResolvedValue({ ok: false, status: 503 })); }); afterEach(() => { + vi.unstubAllGlobals(); process.env = { ...originalEnv }; }); diff --git a/src/__tests__/auth.test.ts b/src/__tests__/auth.test.ts index 7b4db22..cf5ff60 100644 --- a/src/__tests__/auth.test.ts +++ b/src/__tests__/auth.test.ts @@ -69,9 +69,14 @@ describe("auth init", () => { beforeEach(() => { dbSelectResult = []; vi.clearAllMocks(); + // Stub fetch so OIDC discovery requests resolve instantly during tests. + // Without this, AbortSignal.timeout(5000) in auth.ts races with vitest's + // 5000ms default test timeout and causes flaky failures. + vi.stubGlobal("fetch", vi.fn().mockResolvedValue({ ok: false, status: 503 })); }); afterEach(() => { + vi.unstubAllGlobals(); process.env = { ...originalEnv }; });