Merge lp:~nik90/ubuntu-clock-app/reenable-alarm-tests into lp:ubuntu-clock-app

Proposed by Nekhelesh Ramananthan
Status: Merged
Approved by: Nicholas Skaggs
Approved revision: 226
Merged at revision: 227
Proposed branch: lp:~nik90/ubuntu-clock-app/reenable-alarm-tests
Merge into: lp:ubuntu-clock-app
Diff against target: 47 lines (+3/-9)
2 files modified
debian/changelog (+2/-0)
tests/autopilot/ubuntu_clock_app/tests/test_alarm.py (+1/-9)
To merge this branch: bzr merge lp:~nik90/ubuntu-clock-app/reenable-alarm-tests
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Ubuntu Clock Developers Pending
Review via email: mp+253291@code.launchpad.net

Commit message

Re-enabled alarm tests on phone which were originally disabled due to upstream autopilot bug.

Description of the change

Re-enabled alarm tests on phone which were originally disabled due to upstream autopilot bug.

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)
226. By Nekhelesh Ramananthan

Fixed pyflakes error and updated copyright header

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 'debian/changelog'
2--- debian/changelog 2015-03-12 23:23:31 +0000
3+++ debian/changelog 2015-03-19 23:58:26 +0000
4@@ -12,6 +12,8 @@
5 * Fixed empty state description not wrapping (LP: #1428165)
6 * Fixed edit alarm crash issue in vivid (thanks to Zsombor) (LP: #1429273)
7 * Fixed strings not following gettext-style plural forms. (LP: #1431446)
8+ * Renabled create alarm tests on device which were disabled due to upstream bug
9+ in autopilot. (LP: #1426108)
10
11 [Brendan Donegan]
12 * Fixed AP failure by waiting for the bottom edge tip visible property to be true
13
14=== modified file 'tests/autopilot/ubuntu_clock_app/tests/test_alarm.py'
15--- tests/autopilot/ubuntu_clock_app/tests/test_alarm.py 2014-10-02 19:52:16 +0000
16+++ tests/autopilot/ubuntu_clock_app/tests/test_alarm.py 2015-03-19 23:58:26 +0000
17@@ -1,4 +1,4 @@
18-# Copyright (C) 2014 Canonical Ltd
19+# Copyright (C) 2014-2015 Canonical Ltd
20 #
21 # This file is part of Ubuntu Clock App
22 #
23@@ -19,12 +19,10 @@
24 from __future__ import absolute_import
25
26 import datetime
27-import unittest
28
29 from autopilot.matchers import Eventually
30 from testtools.matchers import Equals
31 from ubuntu_clock_app.tests import ClockAppTestCase
32-from autopilot.platform import model
33
34
35 class TestAlarm(ClockAppTestCase):
36@@ -71,12 +69,6 @@
37
38 self.page = self.main_view.open_alarm()
39
40- # TODO
41- # Due to bug https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1328600
42- # this test cannot be run on device, so until the bug is not fixed we are
43- # skipping the test if model not Desktop.
44- @unittest.skipIf(model() != 'Desktop',
45- "datepicker does not work correctly on device")
46 def test_add_recurring_type_alarm_must_add_to_alarm_list(self):
47 """Test to check if alarms are saved properly
48

Subscribers

People subscribed via source and target branches