Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Upload Surefire reports for later troubleshooting #604

Merged
merged 24 commits into from
Aug 14, 2021
Merged

Upload Surefire reports for later troubleshooting #604

merged 24 commits into from
Aug 14, 2021

Conversation

brenuart
Copy link
Collaborator

@brenuart brenuart commented Aug 14, 2021 •
edited
Loading

Upload surefire reports to ease troubleshooting failed tests afterwards.

  • configure build with actions/upload-artifact
  • configure surefire plugin to redirect output to file instead of console when building on Github (keep logs on the console when building on dev workstation)
  • automatically register a Junit5 Extension to log the start of each test to help determine test output boundaries in log output file
  • cleanup build.xml(indent, normalise action names, etc)
  • remove META-INF from .gitignore (auto-registration of Junit5 extension makes use of the "service" facility and requires some files in META-INF/services)
  • move Maven default JVM configuration into .mvn/jvm.config instead of environment variable

@brenuart
Configure workflow to show surefire test results as PR checks
3153d53
@brenuart
Simulate failing test
fefcf50
@brenuart
Tell Maven to not fail build on failed test
476e49c 
Don't fail  the build and let the surefire action report results and mark build as failed.
@brenuart
Upload test reports
7106248
@brenuart
Redirect test output into file
6425744
@brenuart
Uptdae surefire action version
b20f13c
@brenuart
Test with a modified Surefire report
801d927
@brenuart
Postprocess surefire XML reports to make their sysout accessible thro…
6b695c8 
…ugh the Surefire Github annotation
@brenuart
Install XML tools
7bb3e5b
@brenuart
Limit to target/surefire-reports-github only
8ad3a86
@brenuart
Check if xalan is installed
c8cf627
@brenuart
Revert "Check if xalan is installed"
4de715c 
This reverts commit c8cf627.
@brenuart
Use mikepenz/action-junit-report instead of scacap/action/surefire-re…
1b028e4 
…port
@brenuart
Fail the build early when test failure but always process test reports
9cb7fd4
@brenuart
Use MAVEN_OPTS to declare maven options applicable to all invocations
4073a7e
@brenuart
Revert "Use MAVEN_OPTS to declare maven options applicable to all inv…
51c5158 
…ocations"

This reverts commit 4073a7e.
@brenuart
Move JVM options into .mvn/jvm.config
6c4ef58
@brenuart
Test action brenuart/action-junit-report
1a19f59
@brenuart
Try without explicit github token (and rely on default)
350846a
@brenuart
Revert "Try without explicit github token (and rely on default)"
fb28ba3 
This reverts commit 350846a.
@brenuart
Discard surefire action
cfbd5f0 
action does not have write permissions when executed from withi a workflow triggered by a fork issued from a PR
@brenuart
Remove failing test
cb3e671
@brenuart
Add JUnit5 extension to log the test name before it starts
841cc69
@brenuart
Cleanup and remove useless files
62c98aa
@brenuart brenuart merged commit 940fec4 into main Aug 14, 2021
@brenuart brenuart deleted the surefire-check branch August 14, 2021 09:07
@philsttr philsttr added this to the 7.0 milestone Aug 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
type/enhancement
Projects
None yet
Milestone
7.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@brenuart @philsttr