testtools-dev team mailing list archive
-
testtools-dev team
-
Mailing list archive
-
Message #00097
Re: lp:~gz/testtools/use_reason_only_on_decorator_fallback_625583 into lp:testtools
Okay, so my reasoning for this being the right thing is:
* So far, everyone's been using Python 2.6 and lower, and have not been clamouring for all details to be printed out rather than just the reason.
* When vila and I first saw logs being printed in the babune log we thought there was more than one writer mangling the subunit stream. If something looks like a mistake, it's not likely to be taken as useful information.
* When people want the details, all then need to do is add `details=None` to their test result `addSkip` method, and use them.
* In fact, because of this poor fallback behaviour, we've got some code added to bzr specifically to strip logs from skipped tests entirely, even though it *is* useful information for some clients.
--
https://code.launchpad.net/~gz/testtools/use_reason_only_on_decorator_fallback_625583/+merge/36667
Your team testtools developers is requested to review the proposed merge of lp:~gz/testtools/use_reason_only_on_decorator_fallback_625583 into lp:testtools.
References