diff options
| author | Peter Stone <thepeterstone@gmail.com> | 2026-03-06 23:55:07 +0000 |
|---|---|---|
| committer | Peter Stone <thepeterstone@gmail.com> | 2026-03-06 23:55:07 +0000 |
| commit | fd42a54d96fcd3342941caaeb61a4b0d5d3f1b4f (patch) | |
| tree | 0b9ef3b7f0ac3981aa310435d014c9f5e21089d4 /web/test/active-tasks-tab.test.mjs | |
| parent | 7d4890cde802974b94db24071f63e7733c3670fd (diff) | |
recover: restore untracked work from recovery branch (no Gemini changes)
Recovered files with no Claude→Agent contamination:
- docs/adr/002-task-state-machine.md
- internal/api/logs.go/logs_test.go: task-level log streaming endpoint
- internal/api/validate.go/validate_test.go: POST /api/tasks/validate
- internal/api/server_test.go, storage/db_test.go: expanded test coverage
- scripts/reset-failed-tasks, reset-running-tasks
- web/app.js, index.html, style.css: frontend improvements
- web/test/: active-tasks-tab, delete-button, filter-tabs, sort-tasks tests
Manually applied from server.go diff (skipping Claude→Agent rename):
- taskLogStore field + validateCmdPath field
- DELETE /api/tasks/{id} route + handleDeleteTask
- GET /api/tasks/{id}/logs/stream route
- POST /api/tasks/{id}/resume route + handleResumeTimedOutTask
- handleCancelTask: allow cancelling PENDING/QUEUED tasks directly
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'web/test/active-tasks-tab.test.mjs')
| -rw-r--r-- | web/test/active-tasks-tab.test.mjs | 68 |
1 files changed, 68 insertions, 0 deletions
diff --git a/web/test/active-tasks-tab.test.mjs b/web/test/active-tasks-tab.test.mjs new file mode 100644 index 0000000..7b68c6f --- /dev/null +++ b/web/test/active-tasks-tab.test.mjs @@ -0,0 +1,68 @@ +// active-tasks-tab.test.mjs — TDD contract tests for filterActiveTasks +// +// filterActiveTasks is imported from app.js. Tests are RED until the function +// is exported from app.js. +// +// Run with: node --test web/test/active-tasks-tab.test.mjs + +import { describe, it } from 'node:test'; +import assert from 'node:assert/strict'; +import { filterActiveTasks } from '../app.js'; + +// ── Helpers ──────────────────────────────────────────────────────────────────── + +function makeTask(state) { + return { id: state, name: `task-${state}`, state }; +} + +// ── Tests ────────────────────────────────────────────────────────────────────── + +describe('filterActiveTasks', () => { + it('returns only RUNNING tasks when given mixed states', () => { + const tasks = [makeTask('RUNNING'), makeTask('COMPLETED'), makeTask('PENDING')]; + const result = filterActiveTasks(tasks); + assert.equal(result.length, 1); + assert.equal(result[0].state, 'RUNNING'); + }); + + it('returns only READY tasks', () => { + const tasks = [makeTask('READY'), makeTask('FAILED'), makeTask('TIMED_OUT')]; + const result = filterActiveTasks(tasks); + assert.equal(result.length, 1); + assert.equal(result[0].state, 'READY'); + }); + + it('returns only QUEUED tasks', () => { + const tasks = [makeTask('QUEUED'), makeTask('CANCELLED'), makeTask('PENDING')]; + const result = filterActiveTasks(tasks); + assert.equal(result.length, 1); + assert.equal(result[0].state, 'QUEUED'); + }); + + it('returns only BLOCKED tasks', () => { + const tasks = [makeTask('BLOCKED'), makeTask('BUDGET_EXCEEDED'), makeTask('COMPLETED')]; + const result = filterActiveTasks(tasks); + assert.equal(result.length, 1); + assert.equal(result[0].state, 'BLOCKED'); + }); + + it('returns all four active states together, excludes PENDING/COMPLETED/FAILED/TIMED_OUT/CANCELLED/BUDGET_EXCEEDED', () => { + const allStates = [ + 'RUNNING', 'READY', 'QUEUED', 'BLOCKED', + 'PENDING', 'COMPLETED', 'FAILED', 'TIMED_OUT', 'CANCELLED', 'BUDGET_EXCEEDED', + ]; + const tasks = allStates.map(makeTask); + const result = filterActiveTasks(tasks); + assert.equal(result.length, 4, 'exactly 4 active-state tasks should be returned'); + for (const state of ['RUNNING', 'READY', 'QUEUED', 'BLOCKED']) { + assert.ok(result.some(t => t.state === state), `${state} should be included`); + } + for (const state of ['PENDING', 'COMPLETED', 'FAILED', 'TIMED_OUT', 'CANCELLED', 'BUDGET_EXCEEDED']) { + assert.ok(!result.some(t => t.state === state), `${state} should be excluded`); + } + }); + + it('returns empty array for empty input', () => { + assert.deepEqual(filterActiveTasks([]), []); + }); +}); |
