Skip to content

Commit abf0881

Browse files
waleedlatif1claude
andcommitted
fix(testing): add AuthType to all hybrid auth test mocks
Route code now imports AuthType from @/lib/auth/hybrid, so test mocks must export it too. Added AuthTypeMock to @sim/testing and included it in all 15 test files that mock the hybrid auth module. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 6305395 commit abf0881

File tree

18 files changed

+29
-2
lines changed

18 files changed

+29
-2
lines changed

apps/sim/app/api/auth/oauth/credentials/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ const { mockCheckSessionOrInternalAuth, mockLogger } = vi.hoisted(() => {
2424
})
2525

2626
vi.mock('@/lib/auth/hybrid', () => ({
27+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
2728
checkSessionOrInternalAuth: mockCheckSessionOrInternalAuth,
2829
}))
2930

apps/sim/app/api/auth/oauth/token/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ vi.mock('@/lib/auth/credential-access', () => ({
5151
}))
5252

5353
vi.mock('@/lib/auth/hybrid', () => ({
54+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
5455
checkHybridAuth: vi.fn(),
5556
checkSessionOrInternalAuth: mockCheckSessionOrInternalAuth,
5657
checkInternalAuth: vi.fn(),

apps/sim/app/api/files/delete/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@ vi.mock('@/lib/auth', () => ({
9191
}))
9292

9393
vi.mock('@/lib/auth/hybrid', () => ({
94+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
9495
checkHybridAuth: mocks.mockCheckHybridAuth,
9596
checkSessionOrInternalAuth: mocks.mockCheckSessionOrInternalAuth,
9697
checkInternalAuth: mocks.mockCheckInternalAuth,

apps/sim/app/api/files/parse/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,7 @@ vi.mock('@/lib/auth', () => ({
106106
}))
107107

108108
vi.mock('@/lib/auth/hybrid', () => ({
109+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
109110
checkInternalAuth: mockCheckInternalAuth,
110111
checkHybridAuth: mockCheckHybridAuth,
111112
checkSessionOrInternalAuth: mockCheckSessionOrInternalAuth,

apps/sim/app/api/files/serve/[...path]/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@ vi.mock('fs/promises', () => ({
4949
}))
5050

5151
vi.mock('@/lib/auth/hybrid', () => ({
52+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
5253
checkSessionOrInternalAuth: mockCheckSessionOrInternalAuth,
5354
}))
5455

apps/sim/app/api/files/upload/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,6 +100,7 @@ vi.mock('@/lib/auth', () => ({
100100
}))
101101

102102
vi.mock('@/lib/auth/hybrid', () => ({
103+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
103104
checkHybridAuth: mocks.mockCheckHybridAuth,
104105
checkSessionOrInternalAuth: mocks.mockCheckSessionOrInternalAuth,
105106
checkInternalAuth: mocks.mockCheckInternalAuth,

apps/sim/app/api/function/execute/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ vi.mock('@/lib/execution/isolated-vm', () => ({
1818
}))
1919

2020
vi.mock('@/lib/auth/hybrid', () => ({
21+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
2122
checkInternalAuth: mockCheckInternalAuth,
2223
}))
2324

apps/sim/app/api/knowledge/search/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ vi.mock('@sim/db', () => ({
6868
}))
6969

7070
vi.mock('@/lib/auth/hybrid', () => ({
71+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
7172
checkSessionOrInternalAuth: mockCheckSessionOrInternalAuth,
7273
}))
7374

apps/sim/app/api/mcp/serve/[serverId]/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ vi.mock('@sim/db/schema', () => ({
5959
}))
6060

6161
vi.mock('@/lib/auth/hybrid', () => ({
62+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
6263
checkHybridAuth: mockCheckHybridAuth,
6364
checkSessionOrInternalAuth: vi.fn(),
6465
checkInternalAuth: vi.fn(),

apps/sim/app/api/tools/custom/route.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -182,6 +182,7 @@ vi.mock('@/lib/auth', () => ({
182182
}))
183183

184184
vi.mock('@/lib/auth/hybrid', () => ({
185+
AuthType: { SESSION: 'session', API_KEY: 'api_key', INTERNAL_JWT: 'internal_jwt' },
185186
checkSessionOrInternalAuth: (...args: unknown[]) => mockCheckSessionOrInternalAuth(...args),
186187
}))
187188

0 commit comments

Comments
 (0)