TAP results should still respect exit status code

[jasonkarns]

The TAP result parser correctly shows the pass/total test results in the tab. However, if there is an error in the process that isn't caught by the testing framework (say, outside of a test or in an after hook or similar), then it's possible for the process to exit with a non-zero status code. In such a scenario, the tab is rendered Green since the TAP results reported successful, even though the process errored.

I think that the exit status code should be respected, even when given TAP output. launcher 20/20 (or whatever) in the tab name is still fine, but it should be colored red if exit status code is non-zero.

[jasonkarns]

I think this is related to #131

[johanneswuerbach]

This should be resolved in master.