3491 - Fixed a bug where stdout and stderr were logged twice by junitxml for xfail tests.

[jeffreyrack]

Before this commit. stdout and stderr were being logged twice in the junitxml report.

This change fixes the issue so that stdout and stderr are now only logged once:

<?xml version="1.0" encoding="utf-8"?>
<testsuite errors="0" failures="0" name="pytest" skips="1" tests="1" time="0.062">
	<testcase classname="test" file="test.py" line="3" name="test_fail" time="0.0">
	<skipped message="expected test failure"></skipped>
	<system-out>XFAIL This is stdout
	</system-out>
	<system-err>XFAIL This is stderr
	</system-err>
	</testcase>
</testsuite>

[coveralls]

Coverage increased (+0.05%) to 92.742% when pulling b4e0265 on jeffreyrack:3491-junit-logging into afde9f0 on pytest-dev:master.

[nicoddemus]

Thanks a lot!

[nicoddemus]

@jeffreyrack a quick tip: if you put Fix #<issue> in the body of the PR, it will close the issue automatically when it gets merged. 😉

You can read more about it here.

Thanks again for the PR!