testtools-dev team mailing list archive
-
testtools-dev team
-
Mailing list archive
-
Message #00122
Re: lp:~gz/testtools/use_reason_only_on_decorator_fallback_625583 into lp:testtools
Review: Approve
I've also tweaked this one - you had no test coverage (your change only checked the error path), and I've made the no-reason fallback supply the whole details dict (because something has obviously gone wrong).
--
https://code.launchpad.net/~gz/testtools/use_reason_only_on_decorator_fallback_625583/+merge/36667
Your team testtools developers is subscribed to branch lp:testtools.
References