uncaged/nerve
Archived
10
0
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

test(e2e): comprehensive e2e tests for nerve thread list/inspect/show #172

Closed
scottwei wants to merge 1 commits from test/160-workflow-e2e into main
pull from: test/160-workflow-e2e
merge into: uncaged:main
Conversation 0 Commits 1 Files Changed 1
+233 -22
scottwei commented 2026-04-27 06:45:44 +00:00
Owner
Copy Link
Copy Source

Closes #160

Changes

Rewrote workflow-cli-e2e.test.ts with comprehensive output verification for all three nerve thread subcommands:

nerve thread list

  • Active-only default (excludes completed/failed runs)
  • --all shows all runs with correct status icons (✅▶❌)
  • Exit codes displayed where present
  • --workflow filter narrows by workflow name
  • Pagination with --limit and --offset

nerve thread inspect <runId>

  • Run details (runId, workflow name, status)
  • Log entries with correct types (started, completed, thread_command_event)
  • Event count in header
  • Pagination of log entries
  • Failed run shows error payload

nerve thread show <runId>

  • Role round content output
  • Chronological ordering of rounds
  • Run header with workflow name and status
  • Total round count
  • Budget limit handling

Test fixtures

3 seeded runs: completed (demo), active/started (worker), failed (demo) — covering the main status variants.

18 tests total, all passing.

Closes #160 ## Changes Rewrote `workflow-cli-e2e.test.ts` with comprehensive output verification for all three `nerve thread` subcommands: ### `nerve thread list` - Active-only default (excludes completed/failed runs) - `--all` shows all runs with correct status icons (✅▶❌) - Exit codes displayed where present - `--workflow` filter narrows by workflow name - Pagination with `--limit` and `--offset` ### `nerve thread inspect <runId>` - Run details (runId, workflow name, status) - Log entries with correct types (started, completed, thread_command_event) - Event count in header - Pagination of log entries - Failed run shows error payload ### `nerve thread show <runId>` - Role round content output - Chronological ordering of rounds - Run header with workflow name and status - Total round count - Budget limit handling ### Test fixtures 3 seeded runs: completed (demo), active/started (worker), failed (demo) — covering the main status variants. **18 tests total, all passing.**
scottwei added 1 commit 2026-04-27 06:45:45 +00:00
test(e2e): comprehensive e2e tests for nerve thread list/inspect/show (#160) d358c9ee73 
- thread list: active-only default, --all with all statuses, status icons, exit codes, --workflow filter, pagination
- thread inspect: run details, log entries, event count, pagination, failed run payloads
- thread show: role round output, chronological order, run header, round count, budget truncation
scottwei closed this pull request 2026-04-27 06:47:46 +00:00
This repo is archived. You cannot comment on pull requests.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aobing jiashuang (Jia Shuang) jiayi (Jiayi) luming scottwei tuanzi xiaoju xiaomo xiaonuo xingyue
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/nerve#172
Delete Branch "test/160-workflow-e2e"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?