Review: Approve code Broadly, this looks excellent. Have you considered using login_person() from lp.testing so that you don't have to set e-mail addresses when you create roles for tests? -- https://code.launchpad.net/~cjwatson/launchpad/pocket-queue-admin/+merge/117630 Your team Launchpad code reviewers is subscribed to branch lp:launchpad.
_______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
