Code review comment for lp:~om26er/messaging-app/use_fixture

Revision history for this message
Omer Akram (om26er) wrote :

> +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.

« Back to merge proposal