Merge lp:~elopio/notes-app/unskip_bug1288876 into lp:notes-app

Proposed by Leo Arias on 2014-03-25
Status: Needs review
Proposed branch: lp:~elopio/notes-app/unskip_bug1288876
Merge into: lp:notes-app
Diff against target: 15 lines (+0/-6)
1 file modified
tests/autopilot/notes_app/tests/test_expand_collapse.py (+0/-6)
To merge this branch: bzr merge lp:~elopio/notes-app/unskip_bug1288876
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve on 2014-03-25
Ubuntu Phablet Team 2014-03-25 Pending
Review via email: mp+212692@code.launchpad.net

Commit message

Removed the skip for the failing test due to bug #1288876.

To post a comment you must log in.
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Unmerged revisions

258. By Leo Arias on 2014-03-25

Removed the skip for the failing test due to bug #1288876.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/autopilot/notes_app/tests/test_expand_collapse.py'
2--- tests/autopilot/notes_app/tests/test_expand_collapse.py 2014-03-14 14:39:06 +0000
3+++ tests/autopilot/notes_app/tests/test_expand_collapse.py 2014-03-25 18:06:04 +0000
4@@ -64,12 +64,6 @@
5 self.assert_note_eventually_collapsed(second)
6
7 def test_collapse_header(self):
8- # Skipping the test for now because in order to release qt5.2 we want
9- # to match the results as much as possible to what we get in the
10- # current image. As the notes app is going to be deprecated, and this
11- # bug has a low impact on the user experience, it's not critical and
12- # won't be fixed before the qt5.2 release. -- elopio - 2014-03-14
13- self.skipTest('The test is failing because of bug #1288876')
14 first = self.main_window.get_notes()[0]
15 header = self.main_window.get_header()
16

Subscribers

People subscribed via source and target branches

to all changes: