[Merge] lp:~canonical-platform-qa/gallery-app/clean_delete_autopilot_tests into lp:~artmello/gallery-app/gallery-app-fix_autopilot_tests

[Leo Arias]

The proposal to merge lp:~canonical-platform-qa/gallery-app/clean_delete_autopilot_tests into lp:~artmello/gallery-app/gallery-app-fix_autopilot_tests has been updated.

Description changed to:

Please note a couple of things from this branch:

1. The helpers are split into classes that represent the parts of the screen that we are interacting with: all the helpers related to the PopupPhotoViewer are written in the respective class.

2. The public methods of the classes are named after actions that a user does: delete_current_photo, click_yes.

3. The public methods are annotated so they leave a log trace when they are called. In case of error, you can read the log and see what the test was excecutiong when an error was found.

4. A test does a single action and has a single check at the end:
delete a photo and check that it no longer exists in the filesystem.

I left an XXX comment on the old delete test because it has to be split in smaller tests. And as those smaller tests are not representing a user goal, they should be written in QML. Arthur said he was going to start writing those tests, when he is done the old delete autopilot test can be removed.

For more details, see:
https://code.launchpad.net/~canonical-platform-qa/gallery-app/clean_delete_autopilot_tests/+merge/256592
-- 
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~canonical-platform-qa/gallery-app/clean_delete_autopilot_tests into lp:~artmello/gallery-app/gallery-app-fix_autopilot_tests.