@civic/auth 0.0.1-tsc.alpha.3 → 0.1.0

package/test/unit/server/session.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, vi, beforeEach } from "vitest";
+import { describe, it, expect, vi } from "vitest";
 import type { JWT } from "oslo/jwt";
 import { parseJWT } from "oslo/jwt";
 import { getUser } from "@/shared/lib/session.js";
@@ -20,7 +20,7 @@ describe("getUser", () => {
   });
 
   it("should return null if no tokens are retrieved", async () => {
-    vi.mocked(retrieveTokens).mockReturnValue(null);
+    vi.mocked(retrieveTokens).mockResolvedValue(null);
 
     const result = await getUser(mockStorage);
     expect(result).toBeNull();
@@ -29,7 +29,7 @@ describe("getUser", () => {
 
   it("should return a User if tokens are retrieved and parsed successfully", async () => {
     const mockTokens = { id_token: "mockIdToken" } as OIDCTokenResponseBody;
-    vi.mocked(retrieveTokens).mockReturnValue(mockTokens);
+    vi.mocked(retrieveTokens).mockResolvedValue(mockTokens);
     vi.mocked(parseJWT).mockReturnValue({ payload: mockUser } as JWT);
 
     const result = await getUser(mockStorage);
@@ -40,7 +40,7 @@ describe("getUser", () => {
 
   it("should return null if parsing the ID token fails", async () => {
     const mockTokens = { id_token: "mockIdToken" } as OIDCTokenResponseBody;
-    vi.mocked(retrieveTokens).mockReturnValue(mockTokens);
+    vi.mocked(retrieveTokens).mockResolvedValue(mockTokens);
     vi.mocked(parseJWT).mockReturnValue(null); // Simulate parse failure
 
     const result = await getUser(mockStorage);

package/test/unit/services/AuthenticationService.test.ts

@@ -29,7 +29,7 @@ vi.mock("@/shared/lib/util.js", () => ({
   clearUser: vi.fn(),
 }));
 vi.mock("@/browser/storage");
-vi.mock("@/services/PKCE");
+vi.mock("@/services/PKCE.js");
 
 describe("Authentication Services", () => {
   const mockConfig = {
@@ -51,7 +51,7 @@ describe("Authentication Services", () => {
         new URL("http://mockLoginUrl"),
       );
 
-      const url = await initiator.signIn(null);
+      const url = await initiator.signIn({} as HTMLIFrameElement);
 
       expect(url.toString()).toBe("http://mockloginurl/");
     });
@@ -114,7 +114,7 @@ describe("Authentication Services", () => {
         ...mockConfig,
       });
       vi.mocked(exchangeTokens).mockResolvedValue(mockTokens);
-      vi.mocked(retrieveTokens).mockReturnValue(mockTokens);
+      vi.mocked(retrieveTokens).mockResolvedValue(mockTokens);
     });
 
     it("should exchange auth code for tokens and store them", async () => {

package/test/unit/services/ServerAuthenticationResolver.test.ts

@@ -6,13 +6,18 @@ import {
   retrieveTokens,
   storeTokens,
 } from "@/shared/lib/util.js";
-import type { Endpoints, OIDCTokenResponseBody, SessionData, AuthStorage } from "@/types.js";
-import type { AuthConfig } from "@/server/config.js";
+import type {
+  AuthStorage,
+  Endpoints,
+  OIDCTokenResponseBody,
+  SessionData,
+} from "@/types.js";
+import type { AuthConfig } from "@/server/config.ts";
 import { GenericPublicClientPKCEProducer } from "@/services/PKCE.js";
-import type { AuthenticationResolver } from '@/services/types.js';
+import type { AuthenticationResolver } from "@/services/types.ts";
 
 vi.mock("oslo/oauth2");
-vi.mock("@/services/PKCE");
+vi.mock("@/services/PKCE.js");
 vi.mock("@/shared/lib/util.js");
 
 describe("ServerAuthenticationResolver", () => {
@@ -84,7 +89,7 @@ describe("ServerAuthenticationResolver", () => {
 
   describe("getSessionData", () => {
     it("should retrieve session data from storage", async () => {
-      vi.mocked(retrieveTokens).mockReturnValue(mockTokens);
+      vi.mocked(retrieveTokens).mockResolvedValue(mockTokens);
 
       const sessionData: SessionData = {
         authenticated: true,
@@ -100,7 +105,7 @@ describe("ServerAuthenticationResolver", () => {
     });
 
     it("should return null if no tokens are found in storage", async () => {
-      vi.mocked(retrieveTokens).mockReturnValue(null);
+      vi.mocked(retrieveTokens).mockResolvedValue(null);
 
       const result = await resolver.getSessionData();
 

package/test/unit/shared/GenericAuthenticationRefresher.test.ts

@@ -5,9 +5,9 @@ import {
   retrieveTokens,
   storeTokens,
 } from "@/shared/lib/util.js";
-import type { Endpoints, OIDCTokenResponseBody, AuthStorage} from "@/types.js";
+import type { AuthStorage, Endpoints, OIDCTokenResponseBody } from "@/types.js";
 import { OAuth2Client } from "oslo/oauth2";
-import type { AuthConfig } from "@/server/config.js";
+import type { AuthConfig } from "@/server/config.ts";
 
 vi.mock("@/shared/lib/util.js");
 vi.mock("oslo/oauth2");
@@ -55,7 +55,7 @@ describe("GenericAuthenticationRefresher", () => {
 
   describe("refreshTokens", () => {
     it("should refresh tokens and store them", async () => {
-      vi.mocked(retrieveTokens).mockReturnValue(mockTokens);
+      vi.mocked(retrieveTokens).mockResolvedValue(mockTokens);
       const refreshedTokens: OIDCTokenResponseBody = {
         id_token: "newIdToken",
         access_token: "newAccessToken",
@@ -76,7 +76,7 @@ describe("GenericAuthenticationRefresher", () => {
     });
 
     it("should throw an error if no refresh token is available", async () => {
-      vi.mocked(retrieveTokens).mockReturnValue({
+      vi.mocked(retrieveTokens).mockResolvedValue({
         ...mockTokens,
         refresh_token: undefined,
       });

package/test/unit/shared/UserSession.test.ts

@@ -12,10 +12,10 @@ const mockUser: User = {
 
 let storage: Record<string, string> = {};
 class TestStorage implements AuthStorage {
-  get(key: string): string | null {
+  async get(key: string): Promise<string | null> {
     return storage[key] ?? null;
   }
-  set(key: string, value: string): void {
+  async set(key: string, value: string): Promise<void> {
     storage[key] = value;
   }
 }
@@ -26,16 +26,16 @@ describe("UserSession", () => {
   });
 
   describe("GenericUserSession", () => {
-    it("should get the user from session", () => {
+    it("should get the user from session", async () => {
       const userSession = new GenericUserSession(new TestStorage());
       userSession.set(mockUser);
-      const user = userSession.get();
+      const user = await userSession.get();
       expect(user).toEqual(mockUser);
     });
 
-    it("should return null if user it not found", () => {
+    it("should return null if user it not found", async () => {
       const userSession = new GenericUserSession(new TestStorage());
-      const user = userSession.get();
+      const user = await userSession.get();
       expect(user).toBeNull();
     });
   });

package/test/unit/shared/storage.test.ts

@@ -13,10 +13,10 @@ describe("CookieStorage", () => {
       super(settings);
     }
 
-    get(key: string): string | null {
+    async get(key: string): Promise<string | null> {
       return this.storage[key] ?? null;
     }
-    set(key: string, value: string): void {
+    async set(key: string, value: string): Promise<void> {
       this.storage[key] = value;
     }
   }
@@ -57,11 +57,11 @@ describe("CookieStorage", () => {
     expect(storage["settings"]).toMatchObject(customSettings);
   });
 
-  it("should set and get values correctly", () => {
+  it("should set and get values correctly", async () => {
     const storage = new MockCookieStorage({});
 
-    storage.set("testKey", "testValue");
-    expect(storage.get("testKey")).toBe("testValue");
-    expect(storage.get("nonExistentKey")).toBe(null);
+    await storage.set("testKey", "testValue");
+    expect(await storage.get("testKey")).toBe("testValue");
+    expect(await storage.get("nonExistentKey")).toBe(null);
   });
 });