Merge lp:~cwayne/address-book-app/autopilot-upstart into lp:address-book-app

Proposed by Chris Wayne
Status: Needs review
Proposed branch: lp:~cwayne/address-book-app/autopilot-upstart
Merge into: lp:address-book-app
Diff against target: 17 lines (+1/-5)
1 file modified
tests/autopilot/address_book_app/tests/__init__.py (+1/-5)
To merge this branch: bzr merge lp:~cwayne/address-book-app/autopilot-upstart
Reviewer Review Type Date Requested Status
Leo Arias (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Ubuntu Phablet Team Pending
Review via email: mp+212747@code.launchpad.net

Commit message

Use upstart for launching in autopilot tests, to fix issues within the touch_custom suite

Description of the change

Use launch_upstart_application to get the correct XDG_DATA_DIRS set

To post a comment you must log in.
156. By Chris Wayne

Only set upstart launch for test_installed, not test_local

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

FAILED: Continuous integration, rev:155
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/483/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-amd64-ci/193
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/193
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/193/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-i386-ci/193
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/4258
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/3843/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/3692
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4336
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4336/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3845
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3845/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/6130/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/5213

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/address-book-app-ci/483/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:156
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/484/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-amd64-ci/194
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/194
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/194/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-i386-ci/194
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/4259
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/3844/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/3693
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4337
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4337/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3846
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3846/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/6131/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/5214

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/address-book-app-ci/484/rebuild

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

Unmerged revisions

156. By Chris Wayne

Only set upstart launch for test_installed, not test_local

155. By Chris Wayne

Change autopilot tests to launch with upstart

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/autopilot/address_book_app/tests/__init__.py'
2--- tests/autopilot/address_book_app/tests/__init__.py 2014-01-30 08:35:04 +0000
3+++ tests/autopilot/address_book_app/tests/__init__.py 2014-03-26 00:18:00 +0000
4@@ -86,12 +86,8 @@
5 emulator_base=toolkit_emulators.UbuntuUIToolkitEmulatorBase)
6
7 def launch_test_installed(self):
8- df = "/usr/share/applications/address-book-app.desktop"
9- self.ARGS.append("--desktop_file_hint=" + df)
10- self.app = self.launch_test_application(
11+ self.app = self.launch_upstart_application(
12 "address-book-app",
13- *AddressBookAppTestCase.ARGS,
14- app_type='qt',
15 emulator_base=toolkit_emulators.UbuntuUIToolkitEmulatorBase)
16
17 def launch_click_installed(self):

Subscribers

People subscribed via source and target branches