> [7] ... > I don't really see what the second test adds… since you're reloading the > objects in the first test and you're getting what you expect, you can be sure > the objects have been saved… ?
Good point. I was adding belts and braces where dungarees were being worn :) > [8] ... > Similar remark here. Although I'd refactor the first test to reload the > object before checking the owner, and I'd get rid of the second test. Done. -- https://code.launchpad.net/~allenap/maas/shared-to-per-tenant-storage/+merge/151858 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~allenap/maas/shared-to-per-tenant-storage into lp:maas. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
