Merge lp:~jeffmarcom/checkbox/accelerometer_test_fix into lp:checkbox

Proposed by Jeff Marcom
Status: Merged
Approved by: Daniel Manrique
Approved revision: 1956
Merged at revision: 1956
Proposed branch: lp:~jeffmarcom/checkbox/accelerometer_test_fix
Merge into: lp:checkbox
Diff against target: 27 lines (+5/-1)
2 files modified
debian/changelog (+4/-1)
jobs/input.txt.in (+1/-0)
To merge this branch: bzr merge lp:~jeffmarcom/checkbox/accelerometer_test_fix
Reviewer Review Type Date Requested Status
Daniel Manrique (community) Approve
Review via email: mp+151089@code.launchpad.net

Description of the change

Added simple check for accelerometer test to only run on hardware with hdaps and hp_accel modules loaded and working.

Now the job will be skipped on machines that do not have any accelerometer hardware at all.

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

Awesome, thanks!

review: Approve
Revision history for this message
Ara Pulido (ara) wrote :

What happens with systems with accelerometer but no working drivers?

Revision history for this message
Daniel Manrique (roadmr) wrote :

This case would require a "please check your computer's specifications. Is the computer supposed to have an accelerometer?" test. We do something similar with the multi-touch tests. The logic can be a bit crazy :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2013-02-27 22:27:39 +0000
+++ debian/changelog 2013-02-28 20:26:21 +0000
@@ -8,7 +8,10 @@
8 [ Daniel Manrique ]8 [ Daniel Manrique ]
9 * Added pipefail option to a few jobs using ansi_parser (LP: #1131598)9 * Added pipefail option to a few jobs using ansi_parser (LP: #1131598)
1010
11 -- Brendan Donegan <brendan.donegan@canonical.com> Mon, 18 Feb 2013 14:35:27 +000011 [ Jeff Marcom ]
12 * jobs/input.txt.in Added job requirement for accelerometer test (LP: #1135832)
13
14 -- Jeff Marcom <jeff.marcom@ubuntu.com> Mon, 18 Feb 2013 14:35:27 +0000
1215
13checkbox (0.15.3) raring; urgency=low16checkbox (0.15.3) raring; urgency=low
1417
1518
=== modified file 'jobs/input.txt.in'
--- jobs/input.txt.in 2013-01-31 23:30:18 +0000
+++ jobs/input.txt.in 2013-02-28 20:26:21 +0000
@@ -44,6 +44,7 @@
44plugin: user-interact44plugin: user-interact
45name: input/accelerometer45name: input/accelerometer
46user: root46user: root
47requires: module.name in ['hdaps', 'hp_accel']
47command: accelerometer_test -m48command: accelerometer_test -m
48_description:49_description:
49 PURPOSE:50 PURPOSE:

Subscribers

People subscribed via source and target branches