Merge lp:~boiko/messaging-app/rtm-test_strings_escaped into lp:messaging-app/rtm-14.09

Proposed by Gustavo Pichorim Boiko
Status: Merged
Approved by: Gustavo Pichorim Boiko
Approved revision: 206
Merged at revision: 205
Proposed branch: lp:~boiko/messaging-app/rtm-test_strings_escaped
Merge into: lp:messaging-app/rtm-14.09
Prerequisite: lp:~tiagosh/messaging-app/rtm-14.09-fix-1376793
Diff against target: 17 lines (+8/-0)
1 file modified
tests/autopilot/messaging_app/tests/test_messaging.py (+8/-0)
To merge this branch: bzr merge lp:~boiko/messaging-app/rtm-test_strings_escaped
Reviewer Review Type Date Requested Status
Ubuntu Phablet Team Pending
Review via email: mp+247522@code.launchpad.net

This proposal supersedes a proposal from 2015-01-24.

Commit message

Test that search terms are properly escaped.

Description of the change

Test that search terms are properly escaped.

To post a comment you must log in.
206. By Gustavo Pichorim Boiko

Make pep8 happy.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/autopilot/messaging_app/tests/test_messaging.py'
2--- tests/autopilot/messaging_app/tests/test_messaging.py 2015-01-24 20:32:42 +0000
3+++ tests/autopilot/messaging_app/tests/test_messaging.py 2015-01-24 20:32:42 +0000
4@@ -339,6 +339,14 @@
5 self.assertThat(
6 lambda: count_visible_threads(threads), Eventually(Equals(5)))
7
8+ text_field.clear()
9+ text_field.write('%')
10+ # as we are testing for items NOT to appear, there is no other way
11+ # other than sleeping for awhile before checking if the threads are
12+ # visible
13+ time.sleep(5)
14+ self.assertThat(count_visible_threads(threads), Equals(0))
15+
16
17 class MessagingTestCaseWithExistingThread(BaseMessagingTestCase):
18

Subscribers

People subscribed via source and target branches