Test-cases that passed are now properly omitted the junit summary.

Review Request #2930 - Created Oct. 5, 2015 and submitted

Information
Garrett Malmquist
pants
gmalmquist/shush-junit-false-positives
2310, 2318
c8477c7...
Reviewers
pants-reviews
jsirois, nhoward_tw, zundel
I introduced a bug in https://rbcommons.com/s/twitter/r/2916/
wherein all test-cases in a class were printed if any of the
test-cases in the class had failed. This occurred due to my
incorrect assumption that only testcases that failed were listed
in the .xml report from the junit runner.

Now junit_run.py only prints a summary of those test-cases that
actually failed.

This fixes #2310.

Manual testing + new integration test.

CI went green here: https://travis-ci.org/pantsbuild/pants/builds/83782147
CI went green here: https://travis-ci.org/pantsbuild/pants/builds/83907491

John Sirois
Garrett Malmquist
John Sirois
Garrett Malmquist
Eric Ayers
Garrett Malmquist
Review request changed

Status: Closed (submitted)

Change Summary:

In 366d221e7b7b3b07c7e45385e01ae469d2f30ce4. Thanks John & Eric

Loading...