Summary
Major test coverage gaps — several modules have zero tests:
| Module |
Tests |
Coverage |
src/commands/run.ts |
0 |
None |
src/commands/list.ts |
0 |
None |
src/runners/claude-code.ts |
0 |
None |
src/utils/git.ts |
0 |
None — critical git operations untested |
src/utils/display.ts |
0 |
None |
Priority tests to add
git.ts (highest priority)
createWorktree() — creates worktree, returns path
removeWorktree() — cleans up worktree
getDiff() — returns diff string
getDiffStats() — parses diff stats correctly
cleanupBranches() — removes thinktank/* branches
display.ts
padRight() — handles ANSI codes in length calculation
formatDuration() — seconds and minutes formatting
Integration tests
- Full
run → apply workflow against a test fixture
- Worktree cleanup on failure
- Concurrent agent execution
Acceptance criteria
Summary
Major test coverage gaps — several modules have zero tests:
src/commands/run.tssrc/commands/list.tssrc/runners/claude-code.tssrc/utils/git.tssrc/utils/display.tsPriority tests to add
git.ts(highest priority)createWorktree()— creates worktree, returns pathremoveWorktree()— cleans up worktreegetDiff()— returns diff stringgetDiffStats()— parses diff stats correctlycleanupBranches()— removes thinktank/* branchesdisplay.tspadRight()— handles ANSI codes in length calculationformatDuration()— seconds and minutes formattingIntegration tests
run → applyworkflow against a test fixtureAcceptance criteria