Code review comment for lp:~lifeless/zope.testing/subunit
Revision history for this message
| Benji York (benji) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
On Tue, Jul 20, 2010 at 11:15 AM, Robert Collins
<email address hidden> wrote:
> I agree there are tests missing; I don't see that holding improvements
> of untested code hostage to tests is a good balance.
I don't expect the patch to include all the missing tests, but a
regression test would be appropriate. Since you produced the patch, I
assume you can reproduce the behavior that the patch corrects.
--
Benji York