Merge lp:~brendan-donegan/ubuntu-weather-app/remove_app_header_override into lp:ubuntu-weather-app/obsolete.trunk

Proposed by Brendan Donegan
Status: Merged
Approved by: Alan Pope 🍺🐧🐱 πŸ¦„
Approved revision: 390
Merged at revision: 391
Proposed branch: lp:~brendan-donegan/ubuntu-weather-app/remove_app_header_override
Merge into: lp:ubuntu-weather-app/obsolete.trunk
Diff against target: 50 lines (+0/-30)
1 file modified
tests/autopilot/ubuntu_weather_app/tests/__init__.py (+0/-30)
To merge this branch: bzr merge lp:~brendan-donegan/ubuntu-weather-app/remove_app_header_override
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Leo Arias (community) Approve
Review via email: mp+243409@code.launchpad.net

Commit message

Remove overridden AppHeader since the version in ubuntuuitoolkit works just fine. Also remove the redundant second copyright notice.

Description of the change

The overridden AppHeader custom proxy object was using a function that was removed from the UI toolkit helper (_show_if_not_visible). Further investigation found that the override is no longer necessary so I propose to remove it altogether. Running the tests with this modification means that they all pass (previously there were 5 failures).

There was also a stray copyright header that was added by mistake so removed that as well.

To post a comment you must log in.
Revision history for this message
Leo Arias (elopio) wrote :

Looks good, and if you say that the test pass, then +1.

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/autopilot/ubuntu_weather_app/tests/__init__.py'
--- tests/autopilot/ubuntu_weather_app/tests/__init__.py 2014-09-04 16:21:49 +0000
+++ tests/autopilot/ubuntu_weather_app/tests/__init__.py 2014-12-02 15:38:21 +0000
@@ -25,22 +25,10 @@
25 base,25 base,
26 emulators as toolkit_emulators26 emulators as toolkit_emulators
27)27)
28from ubuntuuitoolkit._custom_proxy_objects import AppHeader
29import ubuntu_weather_app28import ubuntu_weather_app
3029
31logger = logging.getLogger(__name__)30logger = logging.getLogger(__name__)
3231
33# -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
34# Copyright 2013 Canonical
35#
36# This program is free software: you can redistribute it and/or modify it
37# under the terms of the GNU General Public License version 3, as published
38# by the Free Software Foundation.
39#
40# Authored by: Martin Borho <martin@borho.net>
41
42"""weather app autopilot tests and emulators - top level package."""
43
44import ubuntuuitoolkit32import ubuntuuitoolkit
4533
4634
@@ -125,24 +113,6 @@
125 return self.app.wait_select_single(MainView)113 return self.app.wait_select_single(MainView)
126114
127115
128class AppHeader(AppHeader):
129 """AppHeader Autopilot custom proxy object."""
130
131 def click_action_button(self, action_object_name):
132 """Click an action button of the header.
133
134 :parameter object_name: The QML objectName property of the action
135 :raise ToolkitException: If there is no action button with that object
136 name.
137
138 """
139 self._show_if_not_visible()
140
141 button = self._get_action_button(action_object_name)
142 time.sleep(2)
143 self.pointing_device.click_object(button)
144
145
146class Tabs(toolkit_emulators.UbuntuUIToolkitEmulatorBase):116class Tabs(toolkit_emulators.UbuntuUIToolkitEmulatorBase):
147 """LocationTabs Autopilot emulator."""117 """LocationTabs Autopilot emulator."""
148118

Subscribers

People subscribed via source and target branches