Skip to content

ci: match the per-JDK test-report artifacts in the results publisher - #3

Merged
fprochazka merged 1 commit into
masterfrom
fp/fix-test-results-publisher
Jul 7, 2026
Merged

ci: match the per-JDK test-report artifacts in the results publisher#3
fprochazka merged 1 commit into
masterfrom
fp/fix-test-results-publisher

Conversation

@fprochazka

Copy link
Copy Markdown
Member

What broke

The build-parallelization migration renamed the test-report artifact in gradle-build.yml from junit-test-results to per-matrix-cell names (junit-test-results-jdk${{ matrix.jdk }}), but the workflow_run-triggered publisher in publish-test-results.yml still downloads by the exact name junit-test-results via dawidd6/action-download-artifact. No artifact matches, so every "Publish Test Results" run now fails.

The fix

Switch the download step to a regex match (name: junit-test-results.* with name_is_regexp: true). With name_is_regexp, each matched artifact is extracted into a subdirectory named after itself, and the downstream report_paths: '**/build/test-results/test/TEST-*.xml' glob still matches inside those subdirectories — the published report now covers the results from all matrix JDKs.

This repo's publisher was also lagging behind its siblings, so it is aligned to their known-working configuration: dawidd6/action-download-artifact bumped from @v2 to @v9 (regex matching needs a recent version), the redundant workflow: input dropped (implied by run_id), and the permissions block extended with actions: read + contents: read which v9 needs to read the source run's artifacts.

Verification caveat

workflow_run workflows always execute the default branch's workflow definition, so this fix only takes effect after merge — this PR itself cannot demonstrate the publisher passing. The red "Publish Test Results" run that appears alongside this PR's checks is the pre-fix master definition failing, i.e. exactly the symptom this PR fixes.

@fprochazka
fprochazka merged commit 33fb9d3 into master Jul 7, 2026
3 checks passed
@fprochazka
fprochazka deleted the fp/fix-test-results-publisher branch July 7, 2026 13:12
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