Code review comment for lp://staging/~gary/launchpad/bug741684-3

Revision history for this message
Brad Crittenden (bac) wrote :

Gary this branch looks good. On IRC we discussed the oddity of the bare assert statement in your test. Please either put in a comment explaining why it is there or change it to self.assertEqual.

review: Approve (code)

« Back to merge proposal