Merge lp:~chris.gagnon/dropping-letters/autopilot-1.4 into lp:dropping-letters

Proposed by Chris Gagnon
Status: Merged
Approved by: Thomi Richards
Approved revision: 45
Merged at revision: 43
Proposed branch: lp:~chris.gagnon/dropping-letters/autopilot-1.4
Merge into: lp:dropping-letters
Diff against target: 114 lines (+24/-14)
4 files modified
debian/control (+1/-1)
tests/autopilot/dropping_letters_app/emulators.py (+13/-5)
tests/autopilot/dropping_letters_app/tests/__init__.py (+7/-3)
tests/autopilot/dropping_letters_app/tests/test_dropping_letters.py (+3/-5)
To merge this branch: bzr merge lp:~chris.gagnon/dropping-letters/autopilot-1.4
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Christopher Lee (community) Approve
Thomi Richards (community) Approve
Dropping Letters developers Pending
Review via email: mp+192870@code.launchpad.net

Commit message

update autopilot test to work in 1.4, do not merge until autopilot 1.4 is ready on builders/ci

Description of the change

update autopilot test to work in 1.4

To post a comment you must log in.
44. By Chris Gagnon

update debian control for libautopilot-14 >= 1.4 dependancy

Revision history for this message
Thomi Richards (thomir-deactivatedaccount) wrote :

LGTM

review: Approve
45. By Chris Gagnon

use base.get_launch_commmand from ubuntu-ui-toolkit to get qmlscene

Revision history for this message
Christopher Lee (veebers) wrote :

The addition of " base.get_launch_commmand " looks good to me.

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/control'
--- debian/control 2013-09-26 17:30:10 +0000
+++ debian/control 2013-11-04 22:52:03 +0000
@@ -20,7 +20,7 @@
20Section: debug20Section: debug
21Priority: extra21Priority: extra
22Depends: ${misc:Depends},22Depends: ${misc:Depends},
23 libautopilot-qt,23 libautopilot-qt (>= 1.4),
24 dropping-letters (>= ${binary:Version}),24 dropping-letters (>= ${binary:Version}),
25 python-autopilot,25 python-autopilot,
26 ubuntu-ui-toolkit-autopilot,26 ubuntu-ui-toolkit-autopilot,
2727
=== modified file 'tests/autopilot/dropping_letters_app/emulators.py'
--- tests/autopilot/dropping_letters_app/emulators.py 2013-08-01 18:53:46 +0000
+++ tests/autopilot/dropping_letters_app/emulators.py 2013-11-04 22:52:03 +0000
@@ -32,16 +32,24 @@
32 return item32 return item
3333
34 def get_help_button(self):34 def get_help_button(self):
35 item = self.select_single('QQuickRectangle', objectName='introhelpbutton')35 item = self.select_single('QQuickRectangle',
36 objectName='introhelpbutton')
36 return item37 return item
3738
38 def get_help_popover(self):39 def get_help_popover(self):
39 #checking the popup seems to fail -- https://bugs.launchpad.net/autopilot/+bug/1195141?40 #checking the popup seems to fail
41 # -- https://bugs.launchpad.net/autopilot/+bug/1195141?
40 #we can't get the popup by objectname for ??42 #we can't get the popup by objectname for ??
41 #this works OK because at the moment, there is only one popup -- needs fixed43 #this works OK because at the moment, there is only one popup
44 # -- needs fixed
42 #item = self.select_single('Label', objectname='helpsheetpopoverlabel')45 #item = self.select_single('Label', objectname='helpsheetpopoverlabel')
43 #item = self.select_single('QQuickRectangle', objectname='helpsheetpopoverbox')46 #item = self.select_single('QQuickRectangle',
47 # objectname='helpsheetpopoverbox')
44 #item = self.select_single('Popover', objectname='helpsheetpopover')48 #item = self.select_single('Popover', objectname='helpsheetpopover')
4549
46 item = self.select_single('Popover')50 item = self.wait_select_single('Popover')
47 return item51 return item
52
53 def get_many_popover(self):
54 """get a list of open popovers"""
55 return self.select_many('Popover')
4856
=== modified file 'tests/autopilot/dropping_letters_app/tests/__init__.py'
--- tests/autopilot/dropping_letters_app/tests/__init__.py 2013-09-27 16:56:29 +0000
+++ tests/autopilot/dropping_letters_app/tests/__init__.py 2013-11-04 22:52:03 +0000
@@ -22,9 +22,13 @@
22from autopilot.platform import model22from autopilot.platform import model
23from autopilot.testcase import AutopilotTestCase23from autopilot.testcase import AutopilotTestCase
2424
25from ubuntuuitoolkit import emulators as toolkit_emulators
26from dropping_letters_app import emulators25from dropping_letters_app import emulators
2726
27from ubuntuuitoolkit import (
28 base,
29 emulators as toolkit_emulators
30)
31
2832
29class DroppingLettersTestCase(AutopilotTestCase):33class DroppingLettersTestCase(AutopilotTestCase):
3034
@@ -52,14 +56,14 @@
5256
53 def launch_test_local(self):57 def launch_test_local(self):
54 self.app = self.launch_test_application(58 self.app = self.launch_test_application(
55 "qmlscene",59 base.get_qmlscene_launch_command(),
56 self.local_location,60 self.local_location,
57 app_type='qt',61 app_type='qt',
58 emulator_base=toolkit_emulators.UbuntuUIToolkitEmulatorBase)62 emulator_base=toolkit_emulators.UbuntuUIToolkitEmulatorBase)
5963
60 def launch_test_installed(self):64 def launch_test_installed(self):
61 self.app = self.launch_test_application(65 self.app = self.launch_test_application(
62 "qmlscene",66 base.get_qmlscene_launch_command(),
63 self.installed_location,67 self.installed_location,
64 "--desktop_file_hint=/usr/share/applications/"68 "--desktop_file_hint=/usr/share/applications/"
65 "dropping-letters.desktop",69 "dropping-letters.desktop",
6670
=== modified file 'tests/autopilot/dropping_letters_app/tests/test_dropping_letters.py'
--- tests/autopilot/dropping_letters_app/tests/test_dropping_letters.py 2013-09-27 16:56:29 +0000
+++ tests/autopilot/dropping_letters_app/tests/test_dropping_letters.py 2013-11-04 22:52:03 +0000
@@ -19,7 +19,7 @@
19from __future__ import absolute_import19from __future__ import absolute_import
2020
21from autopilot.matchers import Eventually21from autopilot.matchers import Eventually
22from testtools.matchers import Equals, NotEquals, LessThan22from testtools.matchers import Equals, LessThan
2323
24from dropping_letters_app.tests import DroppingLettersTestCase24from dropping_letters_app.tests import DroppingLettersTestCase
25from time import sleep25from time import sleep
@@ -45,16 +45,14 @@
45 self.pointing_device.click_object(helpbutton)45 self.pointing_device.click_object(helpbutton)
4646
47 #check and ensure help popover appears47 #check and ensure help popover appears
48 self.assertThat(self.main_view.get_help_popover,
49 Eventually(NotEquals(None)))
50 helpPopover = self.main_view.get_help_popover()48 helpPopover = self.main_view.get_help_popover()
51 properties = helpPopover.get_properties()49 properties = helpPopover.get_properties()
52 self.assertTrue(properties["visible"])50 self.assertTrue(properties["visible"])
5351
54 #check and ensure help popover disappears52 #check and ensure help popover disappears
55 self.pointing_device.click_object(helpbutton)53 self.pointing_device.click_object(helpbutton)
56 self.assertThat(self.main_view.get_help_popover,54 self.assertThat(self.main_view.get_many_popover,
57 Eventually(Equals(None)))55 Eventually(Equals([])))
5856
59 def test_start_new_game(self):57 def test_start_new_game(self):
60 toolbar = self.main_view.open_toolbar()58 toolbar = self.main_view.open_toolbar()

Subscribers

People subscribed via source and target branches