Skip to content

test(coverage): remove core exclusions and enforce gates#1627

Merged
OneStepAt4time merged 1 commit intodevelopfrom
fix/1616-core-coverage-gate
Apr 10, 2026
Merged

test(coverage): remove core exclusions and enforce gates#1627
OneStepAt4time merged 1 commit intodevelopfrom
fix/1616-core-coverage-gate

Conversation

@OneStepAt4time
Copy link
Copy Markdown
Owner

Summary

  • remove src/server.ts, src/session.ts, and src/tmux.ts from Vitest coverage exclusions
  • add server-core-coverage.test.ts integration coverage test that boots real server wiring (with controlled tmux/pipeline boundaries) and exercises high-value server/session/tmux API paths
  • keep existing global thresholds unchanged (lines/statements/functions 70, �ranches 60) while making them pass with core modules included

Validation

  • npm run lint
  • npm run build
  • npx tsc --noEmit
  • npm test -- --coverage

Closes #1616

Remove server/session/tmux from coverage exclusions and add a high-signal integration test that drives real server/session/tmux paths via Fastify inject to raise enforced coverage back above global thresholds.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant