fix types in tests

[nicknisi]

These are cuasing build errors

[greptile-apps]

Greptile Summary

Fixed TypeScript type errors in test files that were causing build failures.

Changes made:

• In src/server/actions.spec.ts: Removed invalid { data: {} } argument from getAuthAction() calls, as this function is a GET endpoint that accepts no parameters
• In src/client/AuthKitProvider.spec.tsx: Replaced the deprecated pattern of using delete window.location with TypeScript-safe Object.defineProperty() to properly mock window.location in tests

Both changes align the test code with strict TypeScript type checking and modern JavaScript practices.

Confidence Score: 5/5

• This PR is safe to merge with minimal risk
• Both changes are straightforward type fixes in test files only, with no impact on production code. The fixes correctly align test code with actual function signatures and modern TypeScript practices.
• No files require special attention

Important Files Changed

Filename	Overview
src/client/AuthKitProvider.spec.tsx	Replaced deprecated delete window.location pattern with Object.defineProperty to properly mock window.location in tests
src/server/actions.spec.ts	Removed incorrect { data: {} } parameter from getAuthAction() calls to match function signature that takes no parameters

Sequence Diagram

sequenceDiagram
    participant Test as Test Suite
    participant Mock as Mock Setup
    participant Action as getAuthAction
    participant Context as AuthKit Context
    
    Note over Test,Context: actions.spec.ts - Function Call Fix
    Test->>Mock: Setup mockAuthContext
    Test->>Action: getAuthAction()
    Note right of Action: No parameters (GET request)
    Action->>Context: getRawAuthFromContext()
    Context-->>Action: auth data
    Action-->>Test: sanitized user info
    
    Note over Test,Context: AuthKitProvider.spec.tsx - Window Mock Fix
    Test->>Mock: Object.defineProperty(window, 'location', {...})
    Note right of Mock: Proper TypeScript-safe mocking
    Test->>Action: signOut({returnTo: '/home'})
    Action-->>Mock: window.location.href = 'https://auth.workos.com/logout'
    Test->>Mock: Object.defineProperty(window, 'location', {value: originalLocation})
    Note right of Mock: Restore original location

Loading

[greptile-apps]

_{2 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}