[Merge] lp:~om26er/messaging-app/use_fixture into lp:messaging-app

Omer Akram om26er at ubuntu.com
Thu Jul 10 23:04:42 UTC 2014


> +class MessagingFixture(fixtures.Fixture):
> 
> This needs a more meaningful name. Probably it would be better to split the
> fixture in one to backup the history, one to use phonesym, and one to reset
> the services.
> Then combine them in one that could be called MessagingTestEnvironment.

Ok, I split this fixture into multiple fixtures and incorporated that into one.
-- 
https://code.launchpad.net/~om26er/messaging-app/use_fixture/+merge/225350
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~om26er/messaging-app/use_fixture into lp:messaging-app.



More information about the Ubuntu-reviews mailing list