You give us a far cleaner TestRunner, so whatever is left appears bigger now:
- the loop on result_decorators to propagate a boolean looks highly suspicious,
- the injection of BZRTransformingResult is even more ugly than before, couldn't
that be a result decorator now ?
- I still don't like that self.verbosity handling here, again, can't that
be turned into a decorator ?
Given that this patch is a clear improvement, please land, the above can come later
(if you agree on them).
This far better that what we had.
You give us a far cleaner TestRunner, so whatever is left appears bigger now: Result is even more ugly than before, couldn't
- the loop on result_decorators to propagate a boolean looks highly suspicious,
- the injection of BZRTransforming
that be a result decorator now ?
- I still don't like that self.verbosity handling here, again, can't that
be turned into a decorator ?
Given that this patch is a clear improvement, please land, the above can come later
(if you agree on them).