Merge lp:~om26er/ubuntu-calculator-app/pointing_device into lp:~ubuntu-calculator-dev/ubuntu-calculator-app/old_trunk

Proposed by Omer Akram
Status: Merged
Approved by: Gustavo Pichorim Boiko
Approved revision: 172
Merged at revision: 205
Proposed branch: lp:~om26er/ubuntu-calculator-app/pointing_device
Merge into: lp:~ubuntu-calculator-dev/ubuntu-calculator-app/old_trunk
Diff against target: 33 lines (+1/-8)
1 file modified
tests/autopilot/ubuntu_calculator_app/tests/__init__.py (+1/-8)
To merge this branch: bzr merge lp:~om26er/ubuntu-calculator-app/pointing_device
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Gustavo Pichorim Boiko (community) Approve
Review via email: mp+188303@code.launchpad.net

Commit message

create input device from the uitoolkit emulators

Description of the change

toolkit emulators have a function that creates the input device, just inherit it over instead of repeating the code.

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Gustavo Pichorim Boiko (boiko) wrote :

Code looks good.

review: Approve
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
171. By Omer Akram

merge trunk

172. By Omer Akram

fix pyflakes warnings

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_calculator_app/tests/__init__.py'
2--- tests/autopilot/ubuntu_calculator_app/tests/__init__.py 2013-11-05 00:41:03 +0000
3+++ tests/autopilot/ubuntu_calculator_app/tests/__init__.py 2014-01-06 12:15:47 +0000
4@@ -11,8 +11,6 @@
5 import shutil
6 import logging
7
8-from autopilot.input import Mouse, Touch, Pointer
9-from autopilot.platform import model
10 from autopilot.testcase import AutopilotTestCase
11 from ubuntuuitoolkit import (
12 base,
13@@ -30,11 +28,6 @@
14 app.
15
16 """
17- if model() == 'Desktop':
18- scenarios = [('with mouse', dict(input_device_class=Mouse))]
19- else:
20- scenarios = [('with touch', dict(input_device_class=Touch))]
21-
22 local_location = "../../ubuntu-calculator-app.qml"
23 installed_location = "/usr/share/ubuntu-calculator-app/" \
24 "ubuntu-calculator-app.qml"
25@@ -44,7 +37,7 @@
26 backup_dir = sqlite_dir + ".backup"
27
28 def setUp(self):
29- self.pointing_device = Pointer(self.input_device_class.create())
30+ self.pointing_device = toolkit_emulators.get_pointing_device()
31 super(CalculatorTestCase, self).setUp()
32 self.temp_move_sqlite_db()
33 self.addCleanup(self.restore_sqlite_db)

Subscribers

People subscribed via source and target branches