[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