Added flag to always include output in junit reports

gotestyourself / gotestsum

'go test' runner with output optimized for humans, JUnit XML for CI integration, and a summary of the test results.

Apache License 2.0

2.03k stars 119 forks source link

Added flag to always include output in junit reports #327

Open alexxed opened 1 year ago

alexxed commented 1 year ago

fixes #118

ethan-gallant commented 1 year ago

I've tested this change in my Jenkins environment and can confirm it works as expected. It would be awesome if this could make it into a release 🚀

alexxed commented 1 year ago

I'll get back to this eventually to address the feedback, meanwhile I pushed small change to address reruns problems. Currently if there is a rerun that passes, junit report will include both fail and pass, and tools that read that (e.g. Jenkins) will sum those up and consider the test failed.

dnephin commented 10 months ago

Does anyone have an example of the schema that puts system-out under test-case ? If I look at all the top hits on google (https://github.com/windyroad/JUnit-Schema/blob/master/JUnit.xsd, https://github.com/testmoapp/junitxml, https://stackoverflow.com/a/9410271/444646) they put system-out as a sibling to test-case, which does not match the implementation in this PR.