Ensure reporting is aware of the two test executions.#176
Merged
edburns merged 3 commits intogithub:mainfrom May 8, 2026
Merged
Ensure reporting is aware of the two test executions.#176edburns merged 3 commits intogithub:mainfrom
edburns merged 3 commits intogithub:mainfrom
Conversation
1. **action.yml** — Widens the default `report-path` glob from `target/surefire-reports/TEST-*.xml` to `target/surefire-reports*/TEST-*.xml` so both report directories are picked up. Replaces the `-d "$REPORT_DIR"` directory check with `compgen -G` since the path now contains a wildcard in the directory component.
2. **build-test.yml** — Adds `target/surefire-reports-isolated/` to the artifact upload path so the isolated execution's reports are preserved in CI.
3. **pom.xml** — Adds `<reportsDirectory>${project.build.directory}/surefire-reports-isolated</reportsDirectory>` (with a comment) to the `isolated-resume-tests` Surefire execution so its XML reports don't overwrite the default-test reports for the same classes.
Signed-off-by: Ed Burns <edburns@microsoft.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates CI test reporting so results from the project’s two Surefire test executions are both preserved and included in the reported summary, avoiding report overwrites and missing counts.
Changes:
- Configure the
isolated-resume-testsSurefire execution to write XML reports to a dedicatedtarget/surefire-reports-isolated/directory. - Update the composite test-report action default glob and existence check so it detects reports across both report directories.
- Upload the isolated report directory as part of the CI artifact used for site generation.
Show a summary per file
| File | Description |
|---|---|
pom.xml |
Writes isolated Surefire execution XML into target/surefire-reports-isolated to prevent overwriting main execution reports. |
.github/workflows/build-test.yml |
Uploads target/surefire-reports-isolated/ along with other test artifacts on main. |
.github/actions/test-report/action.yml |
Broadens the report glob and uses compgen -G to detect report files when the directory component is a wildcard. |
Copilot's findings
- Files reviewed: 3/3 changed files
- Comments generated: 2
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.
action.yml — Widens the default
report-pathglob fromtarget/surefire-reports/TEST-*.xmltotarget/surefire-reports*/TEST-*.xmlso both report directories are picked up. Replaces the-d "$REPORT_DIR"directory check since the path now contains a wildcard in the directory component.build-test.yml — Adds
target/surefire-reports-isolated/to the artifact upload path so the isolated execution's reports are preserved in CI.pom.xml — Adds
<reportsDirectory>${project.build.directory}/surefire-reports-isolated</reportsDirectory>(with a comment) to theisolated-resume-testsSurefire execution so its XML reports don't overwrite the default-test reports for the same classes.