Comment 5 for bug 1596595

> 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?