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
1=== modified file 'tests/autopilot/ubuntu_weather_app/tests/__init__.py'
2--- tests/autopilot/ubuntu_weather_app/tests/__init__.py 2014-09-04 16:21:49 +0000
3+++ tests/autopilot/ubuntu_weather_app/tests/__init__.py 2014-12-02 15:38:21 +0000
4@@ -25,22 +25,10 @@
5 base,
6 emulators as toolkit_emulators
7 )
8-from ubuntuuitoolkit._custom_proxy_objects import AppHeader
9 import ubuntu_weather_app
10
11 logger = logging.getLogger(__name__)
12
13-# -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
14-# Copyright 2013 Canonical
15-#
16-# This program is free software: you can redistribute it and/or modify it
17-# under the terms of the GNU General Public License version 3, as published
18-# by the Free Software Foundation.
19-#
20-# Authored by: Martin Borho <martin@borho.net>
21-
22-"""weather app autopilot tests and emulators - top level package."""
23-
24 import ubuntuuitoolkit
25
26
27@@ -125,24 +113,6 @@
28 return self.app.wait_select_single(MainView)
29
30
31-class AppHeader(AppHeader):
32- """AppHeader Autopilot custom proxy object."""
33-
34- def click_action_button(self, action_object_name):
35- """Click an action button of the header.
36-
37- :parameter object_name: The QML objectName property of the action
38- :raise ToolkitException: If there is no action button with that object
39- name.
40-
41- """
42- self._show_if_not_visible()
43-
44- button = self._get_action_button(action_object_name)
45- time.sleep(2)
46- self.pointing_device.click_object(button)
47-
48-
49 class Tabs(toolkit_emulators.UbuntuUIToolkitEmulatorBase):
50 """LocationTabs Autopilot emulator."""
51

Subscribers

People subscribed via source and target branches