Skip to content

test(ci): migrate suite to Vitest - #82

Merged
phodal merged 1 commit into
mainfrom
fix/vitest-ci-reporting
Aug 13, 2026
Merged

test(ci): migrate suite to Vitest#82
phodal merged 1 commit into
mainfrom
fix/vitest-ci-reporting

Conversation

@phodal

@phodal phodal commented Aug 13, 2026

Copy link
Copy Markdown
Member

Summary

  • replace the custom node:test capability scheduler and reporter with Vitest 4.1.10
  • keep existing Node assertions while migrating cleanup, dynamic skip, suite hooks, and nested adapter cases
  • show the failing file, full test name, assertion, and source location in the main log
  • retain GitHub annotations, Job Summary output, and a single JUnit artifact
  • remove the custom CI runner/reporter and update contributor test commands

Spec

  • docs/specs/2026-08-13-vitest-migration.md

Validation

  • npm ci
  • npm test — 91 files, 1,305 tests
  • focused CI reporter, JUnit, and GitHub Job Summary simulation
  • doc-link graph — 6 tests
  • npm run pack:verify — 508 npm entries, 530 runtime zip entries
  • git diff --check

Merge gate

Do not merge until the exact PR head passes Linux Node 22.20.0, macOS Node 22.20.0, Windows Node 22.20.0, and Linux Node 24.x.

Replace the custom node:test scheduler and reporter with Vitest default, GitHub Actions, and JUnit reporting so failures identify their owning file, test, assertion, and source location in the main CI log.

Migrate the existing lifecycle APIs without rewriting assertions, document the new focused-test commands, and retain fork isolation across the 91-file suite.

Validated with npm test (91 files, 1305 tests), focused CI/JUnit and Job Summary checks, npm ci, the doc-link graph suite, and npm run pack:verify.

Co-authored-by: Codex (GPT 5.6 Sol) <codex@openai.com>
@phodal
phodal merged commit 2b9a5d3 into main Aug 13, 2026
4 checks passed
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