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 option to write test cases to JSON file #366
Add option to write test cases to JSON file #366
Conversation
be166d4
to
4ba3dbe
Compare
4ba3dbe
to
19d9600
Compare
@EnricoMi let me know what you think about this approach! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very good PR, just minor changes required.
Co-authored-by: Enrico Minack <github@enrico.minack.dev>
Co-authored-by: Enrico Minack <github@enrico.minack.dev>
Co-authored-by: Enrico Minack <github@enrico.minack.dev>
@EnricoMi I think this is ready for your eyes again. Thanks for the quick and thorough feedback! |
I have added this to the CI/CD, so this feature is used there as well: 5d01af9 Please pull from your remote git. |
I am going to rework |
I suspected the failure was related to this. Thanks! Looking forward to what you come up with! |
After reworking the I have also added an example to the README.md. |
Tests are failing due to PyPi outage: https://status.python.org/incidents/77qzc7qsb5wb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Test Results (macOS python installed) 27 files - 36 27 suites - 36 10m 31s ⏱️ - 18m 44s For more details on these errors, see this check. Results for commit 8a4febb. ± Comparison against base commit 71fba75. |
Nice work! |
@EnricoMi pypi said that they resolved the problem if you want to try retriggering the builds (I can't do it since it's my first pr here) |
It does, that is precisely what I need. Thanks for helping quickly get this in! |
Test Results (Test Files) 67 files ±0 6 errors 358 suites ±0 1h 6m 6s ⏱️ ±0s For more details on these parsing errors, failures and errors, see this check. Results for commit 2b89517. ± Comparison against base commit 71fba75. |
Thanks for the contribution! |
This has been released with v2.2.0, enjoy! |
Adds the ability to optionally write out all of the individual unit test results to json file.
Intended to resolve #365