test: mock token retrieval globally

Author: kantord

src/app/catalog/actions.test.ts

@@ -1,24 +1,13 @@
 import { mockedGetRegistryV01Servers } from "@mocks/fixtures/registry_v0_1_servers/get";
 import { HttpResponse } from "msw";
-import { beforeEach, describe, expect, it, vi } from "vitest";
+import { describe, expect, it } from "vitest";
 import { getServers } from "./actions";
 
-// Mock the auth to bypass authentication
-vi.mock("@/lib/api-client", async (importOriginal) => {
-  const original = await importOriginal<typeof import("@/lib/api-client")>();
-  return {
-    ...original,
-    getAuthenticatedClient: vi.fn(() =>
-      original.getAuthenticatedClient("mock-token"),
-    ),
-  };
-});
+// Authentication is mocked globally in vitest.setup.ts:
+// - auth.api.getSession returns a mock session
+// - getValidOidcToken returns "mock-test-token"
 
 describe("getServers", () => {
-  beforeEach(() => {
-    vi.clearAllMocks();
-  });
-
   it("returns servers from default fixture", async () => {
     const servers = await getServers();
 

src/lib/auth/__tests__/token.test.ts

@@ -1,10 +1,15 @@
 import { HttpResponse, http } from "msw";
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { server } from "@/mocks/node";
-import { getValidOidcToken } from "../token";
 import type { OidcTokenData } from "../types";
 import { encrypt } from "../utils";
 
+// Unmock @/lib/auth/token to test the real implementation
+// (overrides the global mock from vitest.setup.ts)
+vi.unmock("@/lib/auth/token");
+// Import after unmocking to get the real function
+const { getValidOidcToken } = await import("../token");
+
 const REFRESH_API_URL = "http://localhost:3000/api/auth/refresh-token";
 
 // Mock jose library

vitest.setup.ts

@@ -41,7 +41,11 @@ vi.mock("@/lib/auth/auth", async (importOriginal) => {
         ...actual.auth.api,
         getSession: vi.fn(() =>
           Promise.resolve({
-            user: { email: "[email protected]", name: "Test User" },
+            user: {
+              id: "mock-user-id",
+              email: "[email protected]",
+              name: "Test User",
+            },
           }),
         ),
       },
@@ -73,6 +77,12 @@ vi.mock("next-themes", () => ({
   ThemeProvider: ({ children }: { children: React.ReactNode }) => children,
 }));
 
+// Mock OIDC token retrieval to return a test token by default
+// This allows server action tests to bypass the full auth flow
+vi.mock("@/lib/auth/token", () => ({
+  getValidOidcToken: vi.fn(() => Promise.resolve("mock-test-token")),
+}));
+
 // Auth client baseline mock; individual tests can customize return values
 vi.mock("@/lib/auth/auth-client", () => ({
   authClient: {