Code review comment for lp:~bac/launchpad/bug-753965

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

Thanks for the review and suggestions Benji.

As mentioned earlier, the test loader is a vast improvement.

The ProductMilestoneView calling BrowserTestCase.setUp directly was a leftover attempt to get failing tests to work. You are correct that it is confusing and has been reverted.

« Back to merge proposal