Merge lp:~bladernr/checkbox/1053010 into lp:checkbox

Proposed by Jeff Lane 
Status: Merged
Merged at revision: 1682
Proposed branch: lp:~bladernr/checkbox/1053010
Merge into: lp:checkbox
Diff against target: 78 lines (+21/-30) (has conflicts)
2 files modified
debian/changelog (+7/-0)
jobs/bluetooth.txt.in (+14/-30)
Text conflict in debian/changelog
To merge this branch: bzr merge lp:~bladernr/checkbox/1053010
Reviewer Review Type Date Requested Status
Marc Tardif (community) Approve
Review via email: mp+125278@code.launchpad.net

Description of the change

combined the bluetooth/mouse and bluetooth/keyboard tests into a single bluetooth/HID test to allow the tester to choose which device to use, rather than have a test that is often skipped.

To post a comment you must log in.
lp:~bladernr/checkbox/1053010 updated
1676. By Jeff Lane 

made a change to re-introduce keyboard_test as an option

Revision history for this message
Marc Tardif (cr3) wrote :

Awesome, but I had to add [FEATURE] to your changelog entry.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-09-19 14:35:12 +0000
3+++ debian/changelog 2012-09-19 18:25:22 +0000
4@@ -22,6 +22,9 @@
5 connections. This is necessary for adequate QA testing (LP: #1042425)
6 * scripts/graphics_driver: fixed a bug causing the hybrid check to throw an
7 exception on hybrid systems (LP: #1048058)
8+ * jobs/bluetooth.txt.in: removed the bluetooth/keyboard job and modified
9+ bluetooth/mouse to be bluetooth/HID and allow the tester to choose a
10+ device to use. (LP: #1053010)
11
12 [Daniel Manrique]
13 * Bumped to 0.14.6 to keep changelog size sane and fix a small mishap in the
14@@ -114,7 +117,11 @@
15 the 'device' argument (bus type) and require at least one value
16 * [FEATURE] scripts/removable_storage_watcher: add support for debugging
17
18+<<<<<<< TREE
19 -- Jeff Lane <jeff@ubuntu.com> Wed, 19 Sep 2012 10:33:17 -0400
20+=======
21+ -- Jeff Lane <jeff@ubuntu.com> Wed, 19 Sep 2012 13:09:08 -0400
22+>>>>>>> MERGE-SOURCE
23
24 checkbox (0.14.5) quantal; urgency=low
25
26
27=== modified file 'jobs/bluetooth.txt.in'
28--- jobs/bluetooth.txt.in 2012-01-31 19:35:41 +0000
29+++ jobs/bluetooth.txt.in 2012-09-19 18:25:22 +0000
30@@ -59,34 +59,18 @@
31 Did you hear the sound you recorded in the bluetooth
32
33 plugin: manual
34-name: bluetooth/keyboard
35+name: bluetooth/HID
36+depends: bluetooth/detect-output
37 command: keyboard_test
38-depends: bluetooth/detect-output
39-_description:
40- PURPOSE:
41- This test will check that you can use a bluetooth keyboard
42- STEPS:
43- 1. Enable the bluetooth keyboard
44- 2. Click on the bluetooth icon in the menu bar
45- 3. Select 'Setup new device'
46- 4. Look for the device in the list and select it
47- 5. Click "Test"
48- 6. Enter some text
49- VERIFICATION:
50- Were you able to enter some text with the bluetooth keyboard?
51-
52-plugin: manual
53-name: bluetooth/mouse
54-depends: bluetooth/detect-output
55-_description:
56- PURPOSE:
57- This test will check that you can use a bluetooth mouse
58- STEPS:
59- 1. Enable the bluetooth mouse
60- 2. Click on the bluetooth icon in the menu bar
61- 3. Select 'Setup new device'
62- 4. Look for the device in the list and select it
63- 5. Move the mouse around the screen
64- 6. Perform some single/double/right click operations
65- VERIFICATION:
66- Did the mouse work as expected?
67+_description:
68+ PURPOSE:
69+ This test will check that you can use a BlueTooth HID device
70+ STEPS:
71+ 1. Enable either a BT mouse or keyboard
72+ 2. Click on the bluetooth icon in the menu bar
73+ 3. Select 'Setup new device'
74+ 4. Look for the device in the list and select it
75+ 5. For mice, perform actions such as moving the pointer, right and left button clicks and double clicks
76+ 6. For keyboards, click the Test button to lauch a small tool. Enter some text into the tool and close it.
77+ VERIFICATION:
78+ Did the device work as expected?

Subscribers

People subscribed via source and target branches