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.

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

Add junit reporting for e2e tests #259

Merged
merged 1 commit into from
Jan 6, 2021
Merged

Add junit reporting for e2e tests #259

merged 1 commit into from
Jan 6, 2021

Conversation

ccremer
Copy link
Contributor

@ccremer ccremer commented Jan 5, 2021

Summary

  • Bats supports formatting tests with junit, and there's a cool junit GH action, so let's marry them!
  • On successful runs, the reports may be added to the "Lint" workflow suite. This is a bug by GitHub and just of a cosmetic nature.

An example of a failing test can be found here: https://github.com/vshn/k8up/pull/259/checks?check_run_id=1651595468

image

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking,
    as they show up in the changelog
  • Update tests.

@ccremer ccremer added the change Generic change that is neither a fix or feature label Jan 5, 2021
@ccremer ccremer force-pushed the junit-e2e branch 4 times, most recently from eaf5ff4 to 6a514e0 Compare January 5, 2021 17:45
@ccremer ccremer marked this pull request as ready for review January 5, 2021 17:47
.github/workflows/build.yml Show resolved Hide resolved
e2e/.gitignore Show resolved Hide resolved
@ccremer ccremer merged commit bf28f91 into master Jan 6, 2021
@ccremer ccremer deleted the junit-e2e branch January 6, 2021 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
change Generic change that is neither a fix or feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants