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

testutils: github: add capability to upload test reports to Git repo #229

Merged
merged 4 commits into from
Jul 8, 2021

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Jul 8, 2021

As discussed in #228 (comment), this PR adds the capability that test reports are uploaded into a Git repository during an RC test run and linked in the sticky comment in the tracking issue for easier evaluation. That Git repository is either identified by the two environment variables RESULT_OUTPUT_DIR (the local checkout of the git repository with properly configured remotes) and RESULT_OUTPUT_URL (a Github-based HTML-URL of the repository) or is generated on the fly for each RC over at gist.github.com.

Currently the tests pass, but the coverage went way down, so I still need to provide new tests for the new code segments.

@miri64 miri64 force-pushed the github/gist-rc-results branch 2 times, most recently from 50bffed to a53bfe8 Compare July 8, 2021 13:31
@miri64 miri64 force-pushed the github/gist-rc-results branch 2 times, most recently from e3132de to ed456df Compare July 8, 2021 14:01
@miri64
Copy link
Member Author

miri64 commented Jul 8, 2021

Ok, tests are now passing. I started a gist based run for which the results are shown in this #227 (comment).

Copy link
Contributor

@MrKevinWeiss MrKevinWeiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lots of code but the output looks good and the tests are there. ACK. Anything else can happen in a followup PR as I would like to test this for RC2

@MrKevinWeiss MrKevinWeiss merged commit 8f69a6b into RIOT-OS:master Jul 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants