Comment 5 for bug 1596595
Revision history for this message
| Galen Charlton (gmc) wrote Re: [Bug 1596595] Re: Hold targeter features and refactoring | #5 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
> The branch includes release notes, but no unit tests. My concern with
> unit tests is that we'd have to create /a lot/ of unit test code to have
> even a modest amount of coverage for the hold targeter. Thoughts
> welcome on that, of course.
The lack of unit tests really concerns me, especially since the
patches *both* refactor the code and add new features; unexpected
regressions and changes in behavior would be painful.
Is there any chance of breaking some of this up into smaller pieces?