Code review comment for lp:~edwin-grubbs/launchpad/bug-615654-jobqueue-cron-script

Revision history for this message
Aaron Bentley (abentley) wrote :

This looks good, except that there are now two definitions assertTextMatchesExpressionIgnoreWhitespace; one in TestCaseWithFactory and one in BrowserTestCase. Actually, it's not factory-specific, so it should probably go in TestCase.

review: Approve

« Back to merge proposal