Merge lp:~brendan-donegan/checkbox/bug886129_touch_as_mouse into lp:checkbox

Proposed by Brendan Donegan
Status: Merged
Merged at revision: 1124
Proposed branch: lp:~brendan-donegan/checkbox/bug886129_touch_as_mouse
Merge into: lp:checkbox
Diff against target: 36 lines (+7/-5)
2 files modified
debian/changelog (+2/-0)
jobs/input.txt.in (+5/-5)
To merge this branch: bzr merge lp:~brendan-donegan/checkbox/bug886129_touch_as_mouse
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+82264@code.launchpad.net

Description of the change

This change very simply adds 'device.category == TOUCH' as an accepted requirement for running the input/mouse test. At the moment some people with multitouch touchpads and touchscreens are getting the input devices category skipped, causing their system to get one star on the Ubuntu Friendly site.

I also took the opportunity to generalise the language used by the test to say 'pointing device' instead of 'mouse' specifically. If that's too controversial I can revert it.

This fix is for a bug that has been milestoned for the first Oneiric SRU so it would be good to get it merged ASAP

To post a comment you must log in.
Revision history for this message
Daniel Manrique (roadmr) wrote :

This looks OK, thanks! merging...

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2011-11-14 20:53:56 +0000
+++ debian/changelog 2011-11-15 10:24:24 +0000
@@ -40,6 +40,8 @@
40 (LP: #887964)40 (LP: #887964)
41 * Guard against trying to write files to logical partitions on USB sticks41 * Guard against trying to write files to logical partitions on USB sticks
42 (which will obviously fail) in usb_test (LP: #887049)42 (which will obviously fail) in usb_test (LP: #887049)
43 * Allow input/mouse test to run if a TOUCH device is present
44 (LP: #886129)
4345
44 [ Javier Collado ]46 [ Javier Collado ]
45 * Broken job dependencies fixed (LP: #888447)47 * Broken job dependencies fixed (LP: #888447)
4648
=== modified file 'jobs/input.txt.in'
--- jobs/input.txt.in 2011-10-07 18:59:33 +0000
+++ jobs/input.txt.in 2011-11-15 10:24:24 +0000
@@ -1,14 +1,14 @@
1plugin: manual1plugin: manual
2name: input/mouse2name: input/mouse
3requires: device.category == 'MOUSE'3requires: device.category == 'MOUSE' or device.category == 'TOUCH'
4_description:4_description:
5 PURPOSE:5 PURPOSE:
6 This test will test your mouse device6 This test will test your pointing device
7 STEPS:7 STEPS:
8 1. Move the mouse8 1. Move the cursor using the pointing device or touch the screen.
9 2. Perform some single/double/right click operations9 2. Perform some single/double/right click operations.
10 VERIFICATION:10 VERIFICATION:
11 Did the mouse work as expected??11 Did the pointing device work as expected?
1212
13plugin: manual13plugin: manual
14name: input/keyboard14name: input/keyboard

Subscribers

People subscribed via source and target branches