Merge lp:~elopio/ubuntu-ui-toolkit/drag_toolbar into lp:ubuntu-ui-toolkit

Proposed by Leo Arias
Status: Merged
Approved by: Tim Peeters
Approved revision: 785
Merged at revision: 800
Proposed branch: lp:~elopio/ubuntu-ui-toolkit/drag_toolbar
Merge into: lp:ubuntu-ui-toolkit
Diff against target: 21 lines (+2/-2)
1 file modified
tests/autopilot/ubuntuuitoolkit/emulators.py (+2/-2)
To merge this branch: bzr merge lp:~elopio/ubuntu-ui-toolkit/drag_toolbar
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Ubuntu SDK team Pending
Review via email: mp+189206@code.launchpad.net

Commit message

On the autopilot emulator, drag the toolbar to its full height.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Cris Dywan (kalikiana) wrote :

Seems to fail on syntax: TypeError: __init__() takes exactly 4 arguments (3 given)

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

That's weird. Veebers already fixed that, it's code not even in trunk.

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

FAILED: Autolanding.
More details in the following jenkins job:
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-autolanding/376/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy-vm/414
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/3040
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-amd64-autolanding/303
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-armhf-autolanding/302
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-armhf-autolanding/302/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-vm-saucy/311
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/4562
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/4562/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/3042
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/3042/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2547
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2598
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/121
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/120

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

FAILED: Autolanding.
Approved revid is not set in launchpad. This is most likely a launchpad issue and re-approve should fix it. There is also a chance (although a very small one) this is a permission problem of the ps-jenkins bot.
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-autolanding/378/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy-vm/416
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/3042
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-amd64-autolanding/305
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-armhf-autolanding/304
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-armhf-autolanding/304/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-vm-saucy/313
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/4564
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/4564/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/3044
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/3044/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/2549
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/2600
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/129
    SUCCESS: http://10.97.0.26:8080/job/touch-flash-device/128

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/autopilot/ubuntuuitoolkit/emulators.py'
2--- tests/autopilot/ubuntuuitoolkit/emulators.py 2013-10-02 07:01:03 +0000
3+++ tests/autopilot/ubuntuuitoolkit/emulators.py 2013-10-08 21:31:14 +0000
4@@ -82,7 +82,7 @@
5 x, y, _, _ = self.globalRect
6 line_x = x + self.width * 0.50
7 start_y = y + self.height - 1
8- stop_y = y + self.height * 0.95
9+ stop_y = y + self.height - self.get_toolbar().height
10
11 self.pointing_device.drag(line_x, start_y, line_x, stop_y)
12
13@@ -98,7 +98,7 @@
14 def _drag_to_close_toolbar(self):
15 x, y, _, _ = self.globalRect
16 line_x = x + self.width * 0.50
17- start_y = y + self.height * 0.95
18+ start_y = y + self.height - self.get_toolbar().height
19 stop_y = y + self.height - 1
20
21 self.pointing_device.drag(line_x, start_y, line_x, stop_y)

Subscribers

People subscribed via source and target branches

to status/vote changes: