Skip to content

ci: add test.yml workflow for Node 18 and 20 matrix#267

Merged
snipcodeit merged 1 commit intomainfrom
issue/256-add-github-actions-test-workflow-running
Mar 6, 2026
Merged

ci: add test.yml workflow for Node 18 and 20 matrix#267
snipcodeit merged 1 commit intomainfrom
issue/256-add-github-actions-test-workflow-running

Conversation

@snipcodeit
Copy link
Copy Markdown
Owner

Summary

  • Adds .github/workflows/test.yml — a dedicated CI workflow that runs npm test across Node 18 and 20 on every PR and push to main
  • No GitHub token required; the test suite uses the mock layer exclusively and runs fully offline
  • Each matrix leg reports as a separate named check ("Test / 18", "Test / 20") in the PR status panel

Closes #256

Milestone Context

Changes

File Change
.github/workflows/test.yml Created — dedicated test workflow, 21 lines

Design Notes

The existing ci.yml already covers Node 18, 20, 22 with npm test + npm run build + lint. This test.yml is intentionally separate and minimal — it creates distinct named checks in the GitHub PR UI, fulfilling the requirement to "report test results as a PR check" under the Test workflow name. Only Node 18 and 20 are in scope per the issue spec.

Test Plan

  • Verify workflow file exists at .github/workflows/test.yml
  • Confirm workflow triggers on pull_request and push to main
  • Confirm matrix includes Node 18 and 20 only
  • Confirm npm test is the test command (no token needed)
  • Confirm actions/checkout@v4 and actions/setup-node@v4 with cache: npm are used
  • Observe PR check panel shows "Test / 18" and "Test / 20" checks once this PR is open
  • All 387 existing tests pass on Node current (verified locally)

Creates .github/workflows/test.yml that runs npm test against Node 18
and 20 on every pull_request and push to main. No GitHub token required
— the test suite uses the mock layer exclusively and runs offline.

Closes #256
@github-actions github-actions bot added the github-config GitHub configuration changes label Mar 6, 2026
@snipcodeit snipcodeit merged commit 0ae8a8a into main Mar 6, 2026
6 of 7 checks passed
@snipcodeit snipcodeit deleted the issue/256-add-github-actions-test-workflow-running branch March 6, 2026 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github-config GitHub configuration changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add GitHub Actions test workflow running on every PR against Node 18 and 20

1 participant