feat(ci): add JUnit XML test report rendering to ci-svelte-e2e - #61
Conversation
Mirrors the dorny/test-reporter@v1 step from ci-e2e.yml for the Svelte-only workflow. No matrix.suite here so check names are "e2e / results N". Adds checks: write permission. Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
📝 WalkthroughWalkthroughThe GitHub Actions CI workflow for Svelte E2E tests has been updated to add test reporting capabilities. The workflow permissions were extended to include Suggested reviewers
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![macroscopeapp[bot]](https://avatars.githubusercontent.com/in/900172?s=60&v=4){kind=small}
ApprovabilityVerdict: Approved Adds JUnit test report rendering to CI workflow with appropriate permissions. This is a non-breaking CI enhancement that doesn't affect runtime behavior. You can customize Macroscope's approvability policy. Learn more. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
1 participant
Summary
Mirrors the
dorny/test-reporter@v1step fromci-e2e.ymlintoci-svelte-e2e.ymlfor Svelte-only consumers likefx-glass/buffalo.checks: writetopermissionsE2E test reportstep after "Run Playwright tests", before "Upload test report"path: 'test-results/junit.xml'— root-level, nofrontend*/prefix since Svelte libs have no subdirectory structurename: 'e2e / results N'— nomatrix.suitein this workflow, so the check name omits itConsumers need
["junit", { outputFile: "test-results/junit.xml" }]in their Playwright reporter config to get summaries; those without it see the step pass silently viacontinue-on-error: true.Test plan
fx-glass/buffaloPR — verifye2e / results 1ande2e / results 2check runs appear on the PR checks tab with inline test summariesjunit.xmlsees the step pass silently🤖 Generated with Claude Code
Summary by CodeRabbit
Note
Add JUnit XML test report rendering to the Svelte E2E CI workflow
Adds a post-run step to the ci-svelte-e2e.yml workflow that publishes E2E test results from
test-results/junit.xmlto GitHub Checks usingdorny/test-reporter@v1. The step always runs (unless cancelled) and is configured withfail-on-error: falseso reporting failures do not block the job. The workflow also gainschecks: writepermission required by the reporter action.Macroscope summarized 2b3f337.