camera-app

Merge lp://qastaging/~om26er/camera-app/test-fix into lp://qastaging/camera-app

  1. test-fix
  2. Merge into trunk
Proposed by Omer Akram
Status: Merged
Approved by: Florian Boucault
Approved revision: 551
Merged at revision: 556
Proposed branch: lp://qastaging/~om26er/camera-app/test-fix
Merge into: lp://qastaging/camera-app
Diff against target: 28 lines (+10/-9)
1 file modified
tests/autopilot/camera_app/tests/__init__.py (+10/-9)
To merge this branch: bzr merge lp://qastaging/~om26er/camera-app/test-fix
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Leo Arias (community) Approve
Ubuntu Phablet Team Pending
Review via email: mp+256382@code.qastaging.launchpad.net

Commit message

Autopilot: Ensure the media directories exist before trying to delete data from them.

Description of the change

Autopilot: Ensure the media directories exist before trying to delete data from them.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:549
http://jenkins.qa.ubuntu.com/job/camera-app-ci/415/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/111
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/111
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/111/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/111
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/206
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-vivid/809/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-mako/856
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/524
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/19674
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-vivid/649/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-amd64/968
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-amd64/968/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/camera-app-ci/415/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Leo Arias (elopio) wrote :

Thanks Omer.
You can extract a method for the common bits of both cases.

review: Needs Fixing
lp://qastaging/~om26er/camera-app/test-fix updated
550. By Omer Akram

reuse code

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

> Thanks Omer.
> You can extract a method for the common bits of both cases.

Done. I am not able to decide though on what should be the name of the method which checks if a directory exists or not.

I named is directory_exists() would is_directory_there() better or does_directory_exist() ? Or anything else.

lp://qastaging/~om26er/camera-app/test-fix updated
551. By Omer Akram

no need to write a method to check if a directory exists, rather make direct call to os.path.exists

Revision history for this message
Leo Arias (elopio) wrote :

thank you.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:551
http://jenkins.qa.ubuntu.com/job/camera-app-ci/422/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-amd64-ci/118
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/118
        deb: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-armhf-ci/118/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/camera-app-vivid-i386-ci/118
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-vivid-touch/215
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-vivid/854/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-click-autopilot-runner-mako/865
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-click-builder-vivid-armhf/549
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/20097
    FAILURE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-vivid/683/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-amd64/1022
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-amd64/1022/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/camera-app-ci/422/rebuild

review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches