-
Notifications
You must be signed in to change notification settings - Fork 31
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
xunit2subunit filter to convert xunitxml to subunit #24
base: master
Are you sure you want to change the base?
Conversation
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.
I know that currently there are no tests for the other filter scripts. But what do you think of starting adding them? This particular filter could be the first step.
Something as simple as running the filter against some sample data and validating the output would do it.
Thanks for your review @freeekanayaka I will submit a test for this filter soon |
Hi yes - I second Free's request for tests - there are actually unit tests of most of the filters core, its only the very edge code that is untested (or somewhat untested) |
Conflicts: filters/xunit2subunit
Work in progress |
Conflicts: filters/xunit2subunit
This reverts commit 4a15ed1. Conflicts: filters/xunit2subunit
Modify xunit2subunit test time with proper duration
ping :) any news on this? |
fwiw, I actually recently implemented basically this as a standalone tool (mostly as just an exercise to teach myself rust) here: https://github.com/mtreinish/junitxml2subunit I had an application generating junit that I needed to convert to subunit which is using that now. |
Adding a subunit filter to convert xunitxml files to subunit format