Code review comment for lp:~jcsackett/launchpad/add-enums-to-models

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

Hi Jon,

In def test_codehosting_usage(self): why do you make a new distribution instead of using self.distribution?

Same question for def test_bug_tracking_usage(self):

We discussed on IRC refactoring the tests since they are mostly the same.

Otherwise this branch rocks. Look forward to seeing it landed.

review: Approve (code)

« Back to merge proposal