Fix test type errors (#51)

Author: chaance

src/auth.spec.ts

@@ -99,7 +99,9 @@ describe('auth', () => {
       updatedAt: '2021-01-01T00:00:00Z',
       lastSignInAt: '2021-01-01T00:00:00Z',
       externalId: null,
-    } as User;
+      locale: null,
+      metadata: {},
+    } satisfies User;
 
     // Mock the return type of refreshSession
     const mockAuthResponse = {
@@ -327,7 +329,9 @@ describe('auth', () => {
           updatedAt: '2023-01-01T00:00:00Z',
           lastSignInAt: '2023-01-01T00:00:00Z',
           externalId: null,
-        },
+          locale: null,
+          metadata: {},
+        } satisfies User,
         impersonator: {
           email: '[email protected]',
           reason: 'testing',
@@ -389,7 +393,9 @@ describe('auth', () => {
           updatedAt: '2023-01-01T00:00:00Z',
           lastSignInAt: '2023-01-01T00:00:00Z',
           externalId: null,
-        },
+          locale: null,
+          metadata: {},
+        } satisfies User,
         headers: {},
       };
 
@@ -456,7 +462,9 @@ describe('auth', () => {
           updatedAt: '2023-01-01T00:00:00Z',
           lastSignInAt: '2023-01-01T00:00:00Z',
           externalId: null,
-        },
+          locale: null,
+          metadata: {},
+        } satisfies User,
         refreshToken: 'refresh-token',
         headers: {},
         accessToken: '', // Empty string to meet type requirement but it will be treated as falsy

src/authkit-callback-route.spec.ts

@@ -8,6 +8,7 @@ import {
 import { configureSessionStorage } from './sessionStorage.js';
 import { isDataWithResponseInit } from './utils.js';
 import { DataWithResponseInit } from './interfaces.js';
+import type { LoaderFunctionArgs } from 'react-router';
 
 // Mock dependencies
 const fakeWorkosInstance = {
@@ -51,15 +52,19 @@ describe('authLoader', () => {
         request: new Request('https://example.com'),
         params: {},
         context: {},
-      });
+      } as LoaderFunctionArgs);
 
       expect(response).toBeUndefined();
     });
 
     it('should handle authentication failure', async () => {
       authenticateWithCode.mockRejectedValue(new Error('Auth failed'));
       request = createRequestWithSearchParams(request, { code: 'invalid-code' });
-      const response = (await loader({ request, params: {}, context: {} })) as DataWithResponseInit<unknown>;
+      const response = (await loader({
+        request,
+        params: {},
+        context: {},
+      } as LoaderFunctionArgs)) as DataWithResponseInit<unknown>;
       expect(isDataWithResponseInit(response)).toBeTruthy();
 
       expect(response?.init?.status).toBe(500);
@@ -68,7 +73,11 @@ describe('authLoader', () => {
     it('should handle authentication failure with string error', async () => {
       authenticateWithCode.mockRejectedValue('Auth failed');
       request = createRequestWithSearchParams(request, { code: 'invalid-code' });
-      const response = (await loader({ request, params: {}, context: {} })) as DataWithResponseInit<unknown>;
+      const response = (await loader({
+        request,
+        params: {},
+        context: {},
+      } as LoaderFunctionArgs)) as DataWithResponseInit<unknown>;
       expect(isDataWithResponseInit(response)).toBeTruthy();
 
       expect(response?.init?.status).toBe(500);
@@ -80,7 +89,7 @@ describe('authLoader', () => {
       request,
       params: {},
       context: {},
-    });
+    } as LoaderFunctionArgs);
 
     expect(workos.userManagement.authenticateWithCode).toHaveBeenCalledWith({
       clientId: process.env.WORKOS_CLIENT_ID,
@@ -98,7 +107,7 @@ describe('authLoader', () => {
       request,
       params: {},
       context: {},
-    });
+    } as LoaderFunctionArgs);
 
     assertIsResponse(response);
     expect(response.status).toBe(302);
@@ -111,7 +120,7 @@ describe('authLoader', () => {
       request,
       params: {},
       context: {},
-    });
+    } as LoaderFunctionArgs);
 
     assertIsResponse(response);
     expect(response.status).toBe(302);
@@ -125,7 +134,7 @@ describe('authLoader', () => {
       request,
       params: {},
       context: {},
-    });
+    } as LoaderFunctionArgs);
 
     expect(onSuccess).toHaveBeenCalled();
   });
@@ -137,7 +146,7 @@ describe('authLoader', () => {
       }),
       params: {},
       context: {},
-    });
+    } as LoaderFunctionArgs);
     assertIsResponse(response);
     expect(response.status).toBe(302);
     expect(response.headers.get('Location')).toBe('http://example.com/profile');
@@ -159,7 +168,7 @@ describe('authLoader', () => {
       request,
       params: {},
       context: {},
-    });
+    } as LoaderFunctionArgs);
 
     expect(onSuccess).toHaveBeenCalledWith(expect.objectContaining({ impersonator: { email: '[email protected]' } }));
   });
@@ -183,7 +192,7 @@ describe('authLoader', () => {
       request,
       params: {},
       context: {},
-    });
+    } as LoaderFunctionArgs);
 
     expect(onSuccess).toHaveBeenCalledWith(
       expect.objectContaining({
@@ -207,7 +216,7 @@ describe('authLoader', () => {
         request,
         params: {},
         context: {},
-      });
+      } as LoaderFunctionArgs);
 
       // Should be a redirect response
       assertIsResponse(response);
@@ -242,7 +251,7 @@ describe('authLoader', () => {
         request,
         params: {},
         context: {},
-      });
+      } as LoaderFunctionArgs);
 
       // Should be a redirect response
       assertIsResponse(response);

src/session.spec.ts

@@ -1,5 +1,5 @@
 import { LoaderFunctionArgs, Session as ReactRouterSession, redirect } from 'react-router';
-import { AuthenticationResponse } from '@workos-inc/node';
+import { AuthenticationResponse, type User } from '@workos-inc/node';
 import * as ironSession from 'iron-session';
 import * as jose from 'jose';
 import {
@@ -142,7 +142,9 @@ describe('session', () => {
           createdAt: '2021-01-01T00:00:00Z',
           updatedAt: '2021-01-01T00:00:00Z',
           lastSignInAt: '2021-01-01T00:00:00Z',
-        },
+          locale: null,
+          metadata: {},
+        } satisfies User,
         impersonator: undefined,
         headers: {},
       } satisfies Session;
@@ -265,11 +267,12 @@ describe('session', () => {
   });
 
   describe('authkitLoader', () => {
-    const createLoaderArgs = (request: Request): LoaderFunctionArgs => ({
-      request,
-      params: {},
-      context: {},
-    });
+    const createLoaderArgs = (request: Request): LoaderFunctionArgs =>
+      ({
+        request,
+        params: {},
+        context: {},
+      }) as LoaderFunctionArgs;
 
     describe('unauthenticated flows', () => {
       beforeEach(() => {

src/test-utils/test-helpers.ts

@@ -1,5 +1,7 @@
 /* istanbul ignore file */
 
+import type { User } from '@workos-inc/node';
+
 type SearchParamsModifier = Record<string, string> | ((params: URLSearchParams) => void);
 
 /**
@@ -54,7 +56,9 @@ export function createAuthWithCodeResponse(overrides: Record<string, unknown> =
       updatedAt: '2024-01-01T00:00:00Z',
       lastSignInAt: '2024-01-01T00:00:00Z',
       externalId: null,
-    },
+      locale: null,
+      metadata: {},
+    } satisfies User,
     ...overrides,
   };
 }