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 <noreply@anthropic.com>
This commit is contained in:
Chris Parsons 2026-03-17 16:17:20 -07:00
parent 57e6ed0215
commit 1e9bcee0e1

View file

@ -28,6 +28,9 @@ vi.mock("~/models/draft-utils", () => ({
checkAndTriggerNextAutodraft: vi.fn(), checkAndTriggerNextAutodraft: vi.fn(),
calculatePickInfo: vi.fn().mockReturnValue({ round: 1, pickInRound: 1, teamIndex: 0 }), calculatePickInfo: vi.fn().mockReturnValue({ round: 1, pickInRound: 1, teamIndex: 0 }),
})); }));
vi.mock("~/models/user", () => ({
isUserAdminByClerkId: vi.fn(),
}));
// ── Fixtures ───────────────────────────────────────────────────────────────── // ── Fixtures ─────────────────────────────────────────────────────────────────
@ -115,6 +118,10 @@ describe("draft.force-manual-pick action", () => {
const { getAuth } = await import("@clerk/react-router/server"); const { getAuth } = await import("@clerk/react-router/server");
vi.mocked(getAuth).mockResolvedValue({ userId: COMMISSIONER_ID } as any); 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 // Socket
mockSocketIO = { to: vi.fn().mockReturnThis(), emit: vi.fn() }; mockSocketIO = { to: vi.fn().mockReturnThis(), emit: vi.fn() };
const socketModule = await import("~/server/socket"); const socketModule = await import("~/server/socket");