Repository navigation
GitHub Actions: per-test results from the JUnit reports, and shorter action references - #194
Merged
Merged
Conversation
…ob summary and annotations
…e/ at the repo root for shorter references
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.
Follow-up to #193.
Per-test results
The
validateandacceptanceactions wrote a JUnit report per plugin / config but only reported pass / fail from the exit code.common/junit.awk(POSIX awk, no dependencies) reads each report for the test count and the name and message of every failed test.17, or2 of 17 failed), followed by a "Failed tests" list grouped by plugin / config.report-dirinput on both actions, so the reports can be written inside the workspace.checks: write, so not available to fork PRs).Pass / fail is still decided by the exit code only.
Shorter references
The actions are now
Tracktion/pluginval/validate@v2andTracktion/pluginval/acceptance@v2(wereTracktion/pluginval/.github/actions/...). Only eachaction.ymlmoved to the repo root (validate/,acceptance/); the scripts,common/and the README stay in.github/actions/and are found relative to$GITHUB_ACTION_PATH. Nothing has been released against the old paths.Testing
acceptance.ymlnow use./validateand./acceptance, so they exercise the new locations and the parser on Linux, macOS and Windows. They don't assert on the summary text.