Code review comment for lp:~dev-nigelj/launchpad/bug-810551

Revision history for this message
Nigel Jones (dev-nigelj) wrote :

> It does, however, need a test, even though there wasn't one before. I haven't
> done any coding in a while, and found myself too tempted to resist writing one
> myself - http://paste.ubuntu.com/679849/ - if you'd like to use it.

I don't think I could have written the test better myself, so I'd be more than happy to include it (why double up on work too?).

review: Needs Resubmitting

« Back to merge proposal