[Maintenance] Add JUnit XML test reporting#127
Open
korbinian90 wants to merge 3 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds JUnit-style test result reporting to the MATLAB test runner and wires that reporting into the GitHub Actions test workflow so CI can publish per-test results.
Changes:
- Add a new
junitxmloption tospm_testsand attach a JUnit XML plugin when enabled. - Enable JUnit XML output in the MATLAB CI test commands and upload each matrix job’s XML as an artifact.
- Add a follow-up workflow job that downloads those artifacts and publishes aggregated test results in GitHub.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
spm_tests.m |
Adds the new junitxml option and JUnit XML plugin hookup in the MATLAB test runner. |
.github/workflows/matlab.yml |
Enables JUnit XML generation in CI, uploads per-job XML artifacts, and adds a result-publication job. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+136
to
+137
| plugin = XMLPlugin.producingJUnitFormat(junitFile); | ||
| runner.addPlugin(plugin); |
| uses: matlab-actions/run-command@v1 | ||
| with: | ||
| command: addpath(pwd); results = spm_tests('class','regression','display',true); assert(all(~[results.Failed])); | ||
| command: addpath(pwd); results = spm_tests('class','regression','display',true,'junitxml',true); assert(all(~[results.Failed])); |
| % cobertura: save code coverage results in the Cobertura XML format [default: false] | ||
| % tag: test tag selector [default: '', ie all tests] | ||
| % tap: save a Test Anything Protocol (TAP) file [default: false] | ||
| % junitxml: save test results in JUnit-style XML [default: false] |
Contributor
Test Results (per test) 5 files 445 suites 28m 53s ⏱️ For more details on these failures, see this check. Results for commit 596252e. ♻️ This comment has been updated with latest results. |
korbinian90
force-pushed
the
ci-ui-improvement
branch
from
3b1aeec to
596252e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.