As indicated on IRC, I didn't add a TODO, I just moved it.
It took me a while to refactor the code to use stopTestRun rather than output_run. I have had to change some of the behaviour to do so.
Specifically:
* 'testr failing' now shows run id, total tests and skip count as well failure count
* 'testr failing' now has return code 1 when there are failing tests
* 'testr load' will show skips
There's probably other stuff, although I tried to minimize it. Big enough that it definitely needs review.
As indicated on IRC, I didn't add a TODO, I just moved it.
It took me a while to refactor the code to use stopTestRun rather than output_run. I have had to change some of the behaviour to do so.
Specifically:
* 'testr failing' now shows run id, total tests and skip count as well failure count
* 'testr failing' now has return code 1 when there are failing tests
* 'testr load' will show skips
There's probably other stuff, although I tried to minimize it. Big enough that it definitely needs review.