mtreinish / stestr / 1149 / 4

47%

master: 68%

push

travis-ci

Fix failing tests

This commit fixes the failing tests by catching a couple of missing
things from the update. The biggest fix was that for the --no-discover
case we still use a subprocess and because of that we need to tell
output.ReturnCodeToSubunit to that the input is not dynamic (and
therefore a Popen object) so it can handle that properly. The other
major change is that the return code tests are updated so that the
stdout and stderr from the subprocess calls are always decoded in the
non-subunit test cases. This was done primarily for ease of debugging,
but it also enabled the removal of several decode() calls when the
output is parsed.

252 of 883 branches covered (28.54%)

1265 of 2684 relevant lines covered (47.13%)

0.47 hits per line