Skip to content

Commit 2d4271d

Browse files
author
test
committed
test(workflows): fix auth route lint issues
1 parent 2c267c0 commit 2d4271d

File tree

6 files changed

+29
-19
lines changed

6 files changed

+29
-19
lines changed

apps/sim/app/api/workflows/[id]/deploy/route.test.ts

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,13 @@ vi.mock('@/lib/core/utils/request', () => ({
6666
vi.mock('@sim/db', () => ({
6767
db: { select: mockDbSelect, update: mockDbUpdate },
6868
workflow: { variables: 'variables', id: 'id' },
69-
workflowDeploymentVersion: { state: 'state', workflowId: 'workflowId', isActive: 'isActive', createdAt: 'createdAt', id: 'id' },
69+
workflowDeploymentVersion: {
70+
state: 'state',
71+
workflowId: 'workflowId',
72+
isActive: 'isActive',
73+
createdAt: 'createdAt',
74+
id: 'id',
75+
},
7076
}))
7177

7278
vi.mock('drizzle-orm', async (importOriginal) => {
@@ -80,7 +86,8 @@ vi.mock('drizzle-orm', async (importOriginal) => {
8086
})
8187

8288
vi.mock('@/lib/workflows/persistence/utils', () => ({
83-
loadWorkflowFromNormalizedTables: (...args: unknown[]) => mockLoadWorkflowFromNormalizedTables(...args),
89+
loadWorkflowFromNormalizedTables: (...args: unknown[]) =>
90+
mockLoadWorkflowFromNormalizedTables(...args),
8491
deployWorkflow: (...args: unknown[]) => mockDeployWorkflow(...args),
8592
undeployWorkflow: (...args: unknown[]) => mockUndeployWorkflow(...args),
8693
}))

apps/sim/app/api/workflows/[id]/deployments/[version]/revert/route.test.ts

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,8 @@ vi.mock('drizzle-orm', async (importOriginal) => {
7474
})
7575

7676
vi.mock('@/lib/workflows/persistence/utils', () => ({
77-
saveWorkflowToNormalizedTables: (...args: unknown[]) => mockSaveWorkflowToNormalizedTables(...args),
77+
saveWorkflowToNormalizedTables: (...args: unknown[]) =>
78+
mockSaveWorkflowToNormalizedTables(...args),
7879
}))
7980

8081
vi.mock('@/lib/mcp/workflow-mcp-sync', () => ({
@@ -118,13 +119,10 @@ describe('Workflow deployment version revert route', () => {
118119
auth: { success: true, userId: 'api-user', authType: 'api_key' },
119120
})
120121

121-
const req = new NextRequest(
122-
'http://localhost:3000/api/workflows/wf-1/deployments/3/revert',
123-
{
124-
method: 'POST',
125-
headers: { 'x-api-key': 'test-key' },
126-
}
127-
)
122+
const req = new NextRequest('http://localhost:3000/api/workflows/wf-1/deployments/3/revert', {
123+
method: 'POST',
124+
headers: { 'x-api-key': 'test-key' },
125+
})
128126
const response = await POST(req, { params: Promise.resolve({ id: 'wf-1', version: '3' }) })
129127

130128
expect(response.status).toBe(200)

apps/sim/app/api/workflows/[id]/execute/route.async.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,6 @@ vi.mock('@/lib/auth/hybrid', () => ({
2424
INTERNAL_JWT: 'internal_jwt',
2525
},
2626
checkHybridAuth: mockCheckHybridAuth,
27-
AuthType: {
28-
SESSION: 'session',
29-
API_KEY: 'api_key',
30-
INTERNAL_JWT: 'internal_jwt',
31-
},
3227
}))
3328

3429
vi.mock('@/lib/workflows/utils', () => ({

apps/sim/app/api/workflows/[id]/route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { type NextRequest, NextResponse } from 'next/server'
66
import { z } from 'zod'
77
import { getAuditActorMetadata } from '@/lib/audit/actor-metadata'
88
import { AuditAction, AuditResourceType, recordAudit } from '@/lib/audit/log'
9-
import { AuthType, checkHybridAuth, checkSessionOrInternalAuth } from '@/lib/auth/hybrid'
9+
import { AuthType, checkHybridAuth } from '@/lib/auth/hybrid'
1010
import { generateRequestId } from '@/lib/core/utils/request'
1111
import { archiveWorkflow } from '@/lib/workflows/lifecycle'
1212
import { loadWorkflowFromNormalizedTables } from '@/lib/workflows/persistence/utils'

apps/sim/app/api/workflows/[id]/status/route.test.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,8 @@ vi.mock('@/app/api/workflows/middleware', () => ({
3434
}))
3535

3636
vi.mock('@/lib/workflows/persistence/utils', () => ({
37-
loadWorkflowFromNormalizedTables: (...args: unknown[]) => mockLoadWorkflowFromNormalizedTables(...args),
37+
loadWorkflowFromNormalizedTables: (...args: unknown[]) =>
38+
mockLoadWorkflowFromNormalizedTables(...args),
3839
}))
3940

4041
vi.mock('@/lib/workflows/comparison', () => ({
@@ -50,7 +51,12 @@ vi.mock('@sim/db', () => ({
5051
select: mockDbSelect,
5152
},
5253
workflow: { variables: 'variables', id: 'id' },
53-
workflowDeploymentVersion: { state: 'state', workflowId: 'workflowId', isActive: 'isActive', createdAt: 'createdAt' },
54+
workflowDeploymentVersion: {
55+
state: 'state',
56+
workflowId: 'workflowId',
57+
isActive: 'isActive',
58+
createdAt: 'createdAt',
59+
},
5460
}))
5561

5662
vi.mock('drizzle-orm', async (importOriginal) => {

apps/sim/app/api/workflows/middleware.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,11 @@ export async function validateWorkflowAccess(
9494
}
9595

9696
const internalSecret = request.headers.get('X-Internal-Secret')
97-
if (allowInternalSecret && env.INTERNAL_API_SECRET && internalSecret === env.INTERNAL_API_SECRET) {
97+
if (
98+
allowInternalSecret &&
99+
env.INTERNAL_API_SECRET &&
100+
internalSecret === env.INTERNAL_API_SECRET
101+
) {
98102
return { workflow }
99103
}
100104

0 commit comments

Comments
 (0)