Skip to content

feat: tests-changed report in CI summary and PR comments#23

Merged
maximn merged 4 commits into
mainfrom
feat/7-3-tests-changed-report
Apr 2, 2026
Merged

feat: tests-changed report in CI summary and PR comments#23
maximn merged 4 commits into
mainfrom
feat/7-3-tests-changed-report

Conversation

@maximn
Copy link
Copy Markdown
Contributor

@maximn maximn commented Apr 2, 2026

Summary

Adds a "Tests Changed" section to the CI job summary and a compact summary line to PR comments, giving developers per-test visibility into what changed between runs (new tests, removed tests, status flips).

Changes

  • src/history/types.ts — New TestsChangedEntry and TestsChangedReport types
  • src/history/comparison.ts — New computeTestsChanged() pure function using suite::name composite keys
  • src/output/summary.ts — New renderTestsChangedSection() renders per-category tables with 20-row cap and collapsible overflow; SummaryOptions extended with testsChanged
  • src/output/pr-comment.ts — Compact summary line with newly-failing warning; PrCommentSection extended with testsChanged
  • src/index.tscomputeTestsChanged called alongside computeDelta, errors handled with core.debug()
  • Tests: 13 new tests across all modules (410 total, up from 397)

Test plan

  • All 410 tests pass (pnpm test)
  • TypeScript clean (pnpm typecheck)
  • Lint clean (pnpm lint)
  • In a real workflow with history enabled and 2+ runs: CI summary shows "Tests Changed" section after "Changes Since Last Run"
  • PR comment shows compact "📝 N new tests, N removed, N status changes" line
  • Newly failing tests show "⚠️ N newly failing" prefix in PR comment
  • Sections are omitted when no tests changed
  • Categories with 0 tests are omitted from the section
  • Tables with >20 entries collapse into <details>

maximn added 2 commits April 2, 2026 21:47
Add per-category "Tests Changed" section to CI summary (new tests,
status changes, removed tests) with 20-row cap and collapsible overflow.
Add compact summary line to PR comments with newly-failing warning.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

maximn added 2 commits April 2, 2026 21:58
Handle parameterized tests with duplicate suite::name keys using
status-grouped pairing (matches computeDelta approach). Treat errored
as a failing status in PR comment newly-failing detection.
@maximn maximn merged commit 102604d into main Apr 2, 2026
6 checks passed
@maximn maximn deleted the feat/7-3-tests-changed-report branch April 2, 2026 21:00
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