Unity

Merge lp:~brandontschaefer/unity/ap-ibus-shorten-timer into lp:unity

  1. ap-ibus-shorten-timer
  2. Merge into trunk
Proposed by Brandon Schaefer on 2012-04-25
Status: Merged
Approved by: Thomi Richards on 2012-04-25
Approved revision: 2336
Merged at revision: 2337
Proposed branch: lp:~brandontschaefer/unity/ap-ibus-shorten-timer
Merge into: lp:unity
Diff against target: 19 lines (+2/-2)
1 file modified
tests/autopilot/autopilot/tests/test_ibus.py (+2/-2)
To merge this branch: bzr merge lp:~brandontschaefer/unity/ap-ibus-shorten-timer
Reviewer Review Type Date Requested Status
Thomi Richards (community) 2012-04-25 Approve on 2012-04-25
Review via email: mp+103407@code.launchpad.net

Commit Message

Shorten the press/release time of the Ctrl+Space for the IBus AP tests.

Description of the Change

=== Problem ===
In the IBus test when pressing Ctrl+Space and holding it long starts to toggle ibus off/on.
On the local test it was fine, but jenkins must have ran it little slowing causing all the failures.

=== Fix ===
Shorten press/release of the timer.

To post a comment you must log in.
Thomi Richards (thomir) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/autopilot/autopilot/tests/test_ibus.py'
2--- tests/autopilot/autopilot/tests/test_ibus.py 2012-04-20 08:11:26 +0000
3+++ tests/autopilot/autopilot/tests/test_ibus.py 2012-04-25 04:44:18 +0000
4@@ -48,13 +48,13 @@
5 def activate_ibus(self, widget):
6 """Activate IBus, and wait till it's actived on 'widget'"""
7 self.assertThat(widget.im_active, Equals(False))
8- self.keyboard.press_and_release('Ctrl+Space')
9+ self.keyboard.press_and_release('Ctrl+Space', 0.05)
10 self.assertThat(widget.im_active, Eventually(Equals(True)))
11
12 def deactivate_ibus(self, widget):
13 """Deactivate ibus, and wait till it's inactive on 'widget'"""
14 self.assertThat(widget.im_active, Equals(True))
15- self.keyboard.press_and_release('Ctrl+Space')
16+ self.keyboard.press_and_release('Ctrl+Space', 0.05)
17 self.assertThat(widget.im_active, Eventually(Equals(False)))
18
19 def do_dash_test_with_engine(self):