janpeter/Scrum4Me
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat(ST-006): voeg restartClaudeJobAction toe aan actions/claude-jobs.ts (#174)

Browse source 
- Exporteert restartClaudeJobAction(jobId) die FAILED/CANCELLED/SKIPPED jobs atomair reset naar QUEUED
- Valideert auth, demo-blokkade, ownership en restartbare status
- Gebruikt prisma.$transaction: claudeJob.updateMany + conditionale sprintTaskExecution.updateMany reset
- Verstuurt pg_notify claude_job_status zodat Jobs-pagina via SSE ververst
- Unit-tests: happy-path (FAILED/CANCELLED/SKIPPED), demo-blokkade, not-found, niet-restartbare status, race-conditie en sprint sub-task reset
This commit is contained in:
Janpeter Visser 2026-05-09 13:59:06 +02:00 • committed by GitHub
parent 3c773421da
commit 35e37dac09
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 218 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

147
__tests__/actions/claude-jobs.test.ts
View file

@ -8,13 +8,24 @@ const {
mockGetSession,
mockFindFirstJob,
mockUpdateJob,
mockUpdateManyJob,
mockUpdateManySprintTaskExecution,
mockTransaction,
mockExecuteRaw,
} = vi.hoisted(() => ({
mockGetSession: vi.fn(),
mockFindFirstJob: vi.fn(),
mockUpdateJob: vi.fn(),
mockExecuteRaw: vi.fn().mockResolvedValue(undefined),
}))
} = vi.hoisted(() => {
const mockUpdateManyJob = vi.fn()
const mockUpdateManySprintTaskExecution = vi.fn()
const mockTransaction = vi.fn()
return {
mockGetSession: vi.fn(),
mockFindFirstJob: vi.fn(),
mockUpdateJob: vi.fn(),
mockUpdateManyJob,
mockUpdateManySprintTaskExecution,
mockTransaction,
mockExecuteRaw: vi.fn().mockResolvedValue(undefined),
}
})
vi.mock('next/cache', () => ({ revalidatePath: vi.fn() }))
vi.mock('@/lib/auth', () => ({ getSession: mockGetSession }))
@ -23,7 +34,12 @@ vi.mock('@/lib/prisma', () => ({
claudeJob: {
findFirst: mockFindFirstJob,
update: mockUpdateJob,
updateMany: mockUpdateManyJob,
},
sprintTaskExecution: {
updateMany: mockUpdateManySprintTaskExecution,
},
$transaction: mockTransaction,
$executeRaw: mockExecuteRaw,
},
}))
@ -32,6 +48,7 @@ import {
enqueueClaudeJobAction,
enqueueAllTodoJobsAction,
cancelClaudeJobAction,
restartClaudeJobAction,
} from '@/actions/claude-jobs'
const SESSION_USER = { userId: 'user-1', isDemo: false }
@ -39,6 +56,12 @@ const SESSION_USER = { userId: 'user-1', isDemo: false }
beforeEach(() => {
vi.clearAllMocks()
mockExecuteRaw.mockResolvedValue(undefined)
mockTransaction.mockImplementation(async (fn: (tx: unknown) => Promise<unknown>) =>
fn({
claudeJob: { updateMany: mockUpdateManyJob },
sprintTaskExecution: { updateMany: mockUpdateManySprintTaskExecution },
})
)
})
describe('enqueueClaudeJobAction (deprecated)', () => {
@ -104,3 +127,115 @@ describe('cancelClaudeJobAction', () => {
expect(result).toMatchObject({ error: expect.stringContaining('actieve') })
})
})
describe('restartClaudeJobAction', () => {
const FAILED_JOB = {
id: 'job-1',
status: 'FAILED',
kind: 'TASK_IMPLEMENTATION',
task_id: 'task-1',
idea_id: null,
sprint_run_id: null,
product_id: 'prod-1',
}
it('reset een FAILED job naar QUEUED (happy path)', async () => {
mockGetSession.mockResolvedValue(SESSION_USER)
mockFindFirstJob.mockResolvedValue(FAILED_JOB)
mockUpdateManyJob.mockResolvedValue({ count: 1 })
const result = await restartClaudeJobAction('job-1')
expect(result).toEqual({ success: true })
expect(mockUpdateManyJob).toHaveBeenCalledWith(
expect.objectContaining({
where: expect.objectContaining({ id: 'job-1', status: { in: ['FAILED', 'CANCELLED', 'SKIPPED'] } }),
data: expect.objectContaining({ status: 'QUEUED' }),
})
)
expect(mockExecuteRaw).toHaveBeenCalled()
})
it('reset een CANCELLED job naar QUEUED', async () => {
mockGetSession.mockResolvedValue(SESSION_USER)
mockFindFirstJob.mockResolvedValue({ ...FAILED_JOB, status: 'CANCELLED' })
mockUpdateManyJob.mockResolvedValue({ count: 1 })
const result = await restartClaudeJobAction('job-1')
expect(result).toEqual({ success: true })
})
it('reset een SKIPPED job naar QUEUED', async () => {
mockGetSession.mockResolvedValue(SESSION_USER)
mockFindFirstJob.mockResolvedValue({ ...FAILED_JOB, status: 'SKIPPED' })
mockUpdateManyJob.mockResolvedValue({ count: 1 })
const result = await restartClaudeJobAction('job-1')
expect(result).toEqual({ success: true })
})
it('weigert demo-sessie', async () => {
mockGetSession.mockResolvedValue({ userId: 'demo', isDemo: true })
const result = await restartClaudeJobAction('job-1')
expect(result).toMatchObject({ error: expect.stringContaining('demo') })
expect(mockUpdateManyJob).not.toHaveBeenCalled()
})
it('retourneert error als job niet gevonden', async () => {
mockGetSession.mockResolvedValue(SESSION_USER)
mockFindFirstJob.mockResolvedValue(null)
const result = await restartClaudeJobAction('job-1')
expect(result).toMatchObject({ error: expect.stringContaining('niet gevonden') })
})
it('weigert wanneer job een niet-restartbare status heeft', async () => {
mockGetSession.mockResolvedValue(SESSION_USER)
mockFindFirstJob.mockResolvedValue({ ...FAILED_JOB, status: 'DONE' })
const result = await restartClaudeJobAction('job-1')
expect(result).toMatchObject({ error: expect.stringContaining('mislukte') })
expect(mockUpdateManyJob).not.toHaveBeenCalled()
})
it('retourneert error bij race-conditie (updateMany count === 0)', async () => {
mockGetSession.mockResolvedValue(SESSION_USER)
mockFindFirstJob.mockResolvedValue(FAILED_JOB)
mockUpdateManyJob.mockResolvedValue({ count: 0 })
const result = await restartClaudeJobAction('job-1')
expect(result).toMatchObject({ error: expect.stringContaining('gewijzigd') })
})
it('reset ook SprintTaskExecution-rows bij SPRINT_IMPLEMENTATION', async () => {
mockGetSession.mockResolvedValue(SESSION_USER)
mockFindFirstJob.mockResolvedValue({
...FAILED_JOB,
kind: 'SPRINT_IMPLEMENTATION',
sprint_run_id: 'run-1',
})
mockUpdateManyJob.mockResolvedValue({ count: 1 })
mockUpdateManySprintTaskExecution.mockResolvedValue({ count: 3 })
const result = await restartClaudeJobAction('job-1')
expect(result).toEqual({ success: true })
expect(mockUpdateManySprintTaskExecution).toHaveBeenCalledWith(
expect.objectContaining({
where: { sprint_job_id: 'job-1' },
data: expect.objectContaining({ status: 'PENDING' }),
})
)
})
it('reset geen SprintTaskExecution-rows bij TASK_IMPLEMENTATION', async () => {
mockGetSession.mockResolvedValue(SESSION_USER)
mockFindFirstJob.mockResolvedValue(FAILED_JOB)
mockUpdateManyJob.mockResolvedValue({ count: 1 })
await restartClaudeJobAction('job-1')
expect(mockUpdateManySprintTaskExecution).not.toHaveBeenCalled()
})
})