Merge lp:~brian-murray/xpresser/xpresser-right-click into lp:~niemeyer/xpresser/trunk

Proposed by Brian Murray
Status: Merged
Merged at revision: 7
Proposed branch: lp:~brian-murray/xpresser/xpresser-right-click
Merge into: lp:~niemeyer/xpresser/trunk
Diff against target: 52 lines (+15/-3)
2 files modified
xpresser/tests/test_xutils.py (+12/-3)
xpresser/xutils.py (+3/-0)
To merge this branch: bzr merge lp:~brian-murray/xpresser/xpresser-right-click
Reviewer Review Type Date Requested Status
Gustavo Niemeyer Approve
Review via email: mp+40154@code.launchpad.net
To post a comment you must log in.
8. By Brian Murray

add in a test for right clicking

Revision history for this message
Brian Murray (brian-murray) wrote :

I've added a test_right_click to test_xutils. I ran the test after setting right_click to MOUSE_B1C and it failed. After restoring it to MOUSE_B3C the test passed so I think its good to go.

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

That's been merged, thanks Brian.

I'm going to add an additional method to the Xpresser class as well, so that we can use xp.right_click() with it.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'xpresser/tests/test_xutils.py'
2--- xpresser/tests/test_xutils.py 2010-05-18 20:27:10 +0000
3+++ xpresser/tests/test_xutils.py 2010-11-05 17:52:54 +0000
4@@ -149,16 +149,20 @@
5 self.window = self.create_button_window()
6 self.button = self.window.get_child()
7 self.button_clicked = False
8+ self.button_rclicked = False
9 self.button_hovered = False
10
11- def clicked(widget):
12+ def clicked(widget, event):
13+ if event.button == 1:
14+ self.button_clicked = True
15+ elif event.button == 3:
16+ self.button_rclicked = True
17 self.window.destroy()
18- self.button_clicked = True
19
20 def entered(widget):
21 self.button_hovered = True
22
23- self.button.connect("clicked", clicked)
24+ self.button.connect("button_press_event", clicked)
25 self.button.connect("enter", entered)
26
27 self.flush_gtk()
28@@ -176,6 +180,11 @@
29 self.flush_gtk()
30 self.assertTrue(self.button_clicked)
31
32+ def test_right_click(self):
33+ xutils.right_click(*self.get_button_center())
34+ self.flush_gtk()
35+ self.assertTrue(self.button_rclicked)
36+
37 def test_hover(self):
38 xutils.hover(*self.get_button_center())
39 self.flush_gtk()
40
41=== modified file 'xpresser/xutils.py'
42--- xpresser/xutils.py 2010-05-18 20:27:10 +0000
43+++ xpresser/xutils.py 2010-11-05 17:52:54 +0000
44@@ -31,6 +31,9 @@
45 def click(x, y):
46 pyatspi.Registry.generateMouseEvent(x, y, pyatspi.MOUSE_B1C)
47
48+def right_click(x, y):
49+ pyatspi.Registry.generateMouseEvent(x, y, pyatspi.MOUSE_B3C)
50+
51 def hover(x, y):
52 pyatspi.Registry.generateMouseEvent(x, y, pyatspi.MOUSE_ABS)
53

Subscribers

People subscribed via source and target branches