Merge lp:~bzoltan/ubuntu-ui-toolkit/more_1618581 into lp:ubuntu-ui-toolkit/staging

Proposed by Zoltan Balogh
Status: Needs review
Proposed branch: lp:~bzoltan/ubuntu-ui-toolkit/more_1618581
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 33 lines (+3/-2)
2 files modified
tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_flickable.py (+1/-1)
tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_header.py (+2/-1)
To merge this branch: bzr merge lp:~bzoltan/ubuntu-ui-toolkit/more_1618581
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Tim Peeters Approve
Review via email: mp+304455@code.launchpad.net

This proposal supersedes a proposal from 2016-08-31.

Commit message

Fix the header tests. Fixes LP: #1618581

Description of the change

Fix the header tests. Fixes LP: #1618581

To post a comment you must log in.
Revision history for this message
Tim Peeters (tpeeters) wrote :

Ideally these functions would call swipe_to_show_more_above(), but it will be extra work to figure out why they don't, and it is not a new thing introduced here, so good to go!

review: Approve
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)

Unmerged revisions

2089. By Zoltan Balogh

Fix the header tests. Fixes LP: #Bug #1618581

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_flickable.py'
2--- tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_flickable.py 2016-08-30 18:25:45 +0000
3+++ tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_flickable.py 2016-08-31 08:53:08 +0000
4@@ -301,7 +301,7 @@
5 start_x = stop_x = self.globalRect.x + (self.globalRect.width // 2)
6 # Start just a little under the top.
7 containers = self._get_containers()
8- top = _get_visible_container_top(containers) + 5
9+ top = _get_visible_container_top(containers) + units.gu(2)
10 bottom = _get_visible_container_bottom(containers)
11
12 start_y = top
13
14=== modified file 'tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_header.py'
15--- tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_header.py 2016-08-12 13:31:48 +0000
16+++ tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_header.py 2016-08-31 08:53:08 +0000
17@@ -19,6 +19,7 @@
18 from autopilot import logging as autopilot_logging
19 from autopilot.introspection import dbus
20
21+from ubuntuuitoolkit import units
22 from ubuntuuitoolkit._custom_proxy_objects import (
23 _common,
24 _tabbar
25@@ -55,7 +56,7 @@
26 # Workarounds https://bugs.launchpad.net/mir/+bug/1399690
27 rate = 10
28 start_x = stop_x = self.globalRect.x + self.globalRect.width // 2
29- start_y = top_container.globalRect.y + 5
30+ start_y = top_container.globalRect.y + units.gu(2)
31 stop_y = start_y + self.globalRect.height // rate * rate
32 self.pointing_device.drag(start_x, start_y, stop_x, stop_y, rate)
33 self.y.wait_for(0)

Subscribers

People subscribed via source and target branches