Merge lp:~om26er/notes-app/skip into lp:notes-app

Proposed by Omer Akram on 2013-09-27
Status: Rejected
Rejected by: Olivier Tilloy on 2013-10-07
Proposed branch: lp:~om26er/notes-app/skip
Merge into: lp:notes-app
Diff against target: 21 lines (+3/-1)
1 file modified
tests/autopilot/notes_app/tests/ (+3/-1)
To merge this branch: bzr merge lp:~om26er/notes-app/skip
Reviewer Review Type Date Requested Status
Olivier Tilloy Disapprove on 2013-09-27
PS Jenkins bot continuous-integration Approve on 2013-09-27
Bill Filler 2013-09-27 Pending
Review via email:

Commit message

Skip problematic tests on nexus 4.

Description of the change

Skip problematic tests on nexus 4. There is still on module test_images that cannot be skipped because the way those tests are written makes it really hard to disable them, python does not like that.

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

This shouldn’t be needed, let’s fix the app itself.

review: Disapprove
Olivier Tilloy (osomon) wrote :

On top of this, I’ve observed the failed CI runs of a few MRs for this specific problem, and it seems to affect all tests randomly, not just the tests in the test_parts module. So we’d need to skip every single test for this "solution" to be effective, which would render the autopilot tests pointless. We need to find a real fix.

Unmerged revisions

186. By Omer Akram on 2013-09-27

skip problematic tests on nexus 4, spent more than 6hours on them. Couldn't fix. I reported a bug for the notes-app which I think is also related.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/autopilot/notes_app/tests/'
--- tests/autopilot/notes_app/tests/ 2013-07-29 20:53:21 +0000
+++ tests/autopilot/notes_app/tests/ 2013-09-27 03:45:52 +0000
@@ -9,15 +9,17 @@
10from __future__ import absolute_import10from __future__ import absolute_import
12from testtools.matchers import Equals
13from autopilot.matchers import Eventually12from autopilot.matchers import Eventually
14from autopilot.platform import model13from autopilot.platform import model
14from testtools import skipIf
15from testtools.matchers import Equals
16from notes_app.tests import NotesTestCaseBaseWithHTTPServer, DatabaseMixin17from notes_app.tests import NotesTestCaseBaseWithHTTPServer, DatabaseMixin
18import sqlite319import sqlite3
22@skipIf(model() == 'Nexus 4', "unstable tests on mako due to a race")
21class TestFocus(NotesTestCaseBaseWithHTTPServer, DatabaseMixin):23class TestFocus(NotesTestCaseBaseWithHTTPServer, DatabaseMixin):
22 """Tests focusing notes"""24 """Tests focusing notes"""


People subscribed via source and target branches