Code review comment for lp:~diegosarmentero/ubuntuone-windows-installer/setup-buttons

Revision history for this message
Natalia Bidart (nataliabidart) wrote :

> > * If you're patching self.patch(self.ui, 'wizard', FakeWizard) in every test
> > method in SetupAccountTestCase, please add that to the setUp method to avoid
> > duplication.
>
> I don't need that in every tests, just in the ones i added, otherwise if i
> patch the wizard in the setup cause the other tests to fail.

If you need a subset of tests to have a specific setup, then you should unify them in another testcase.

Thanks!

« Back to merge proposal