From 1e9bcee0e1aba453e429c10d47620b9b336a3d07 Mon Sep 17 00:00:00 2001 From: Chris Parsons Date: Tue, 17 Mar 2026 16:17:20 -0700 Subject: [PATCH] Fix draft.force-manual-pick tests broken by isUserAdminByClerkId The route now calls isUserAdminByClerkId which hits database().query.users, but the test's mock DB had no query.users entry. Add a vi.mock for ~/models/user and default isUserAdminByClerkId to false in beforeEach. Co-Authored-By: Claude Sonnet 4.6 --- app/routes/api/__tests__/draft.force-manual-pick.test.ts | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/app/routes/api/__tests__/draft.force-manual-pick.test.ts b/app/routes/api/__tests__/draft.force-manual-pick.test.ts index d1f5c0a..4164fa5 100644 --- a/app/routes/api/__tests__/draft.force-manual-pick.test.ts +++ b/app/routes/api/__tests__/draft.force-manual-pick.test.ts @@ -28,6 +28,9 @@ vi.mock("~/models/draft-utils", () => ({ checkAndTriggerNextAutodraft: vi.fn(), calculatePickInfo: vi.fn().mockReturnValue({ round: 1, pickInRound: 1, teamIndex: 0 }), })); +vi.mock("~/models/user", () => ({ + isUserAdminByClerkId: vi.fn(), +})); // ── Fixtures ───────────────────────────────────────────────────────────────── @@ -115,6 +118,10 @@ describe("draft.force-manual-pick action", () => { const { getAuth } = await import("@clerk/react-router/server"); vi.mocked(getAuth).mockResolvedValue({ userId: COMMISSIONER_ID } as any); + // User model: default to non-admin + const { isUserAdminByClerkId } = await import("~/models/user"); + vi.mocked(isUserAdminByClerkId).mockResolvedValue(false); + // Socket mockSocketIO = { to: vi.fn().mockReturnThis(), emit: vi.fn() }; const socketModule = await import("~/server/socket");