Merge lp:~bladernr/checkbox/1116681-fix-multi-head-description into lp:checkbox

Proposed by Jeff Lane 
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 2102
Merged at revision: 2103
Proposed branch: lp:~bladernr/checkbox/1116681-fix-multi-head-description
Merge into: lp:checkbox
Diff against target: 12 lines (+1/-1)
1 file modified
jobs/monitor.txt.in (+1/-1)
To merge this branch: bzr merge lp:~bladernr/checkbox/1116681-fix-multi-head-description
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+162461@code.launchpad.net

Description of the change

change the requires line for the multi-head monitor test to avoid "not in" syntax to work around a bug in how checkbox parses requirement statements.

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'jobs/monitor.txt.in'
2--- jobs/monitor.txt.in 2013-04-18 09:10:10 +0000
3+++ jobs/monitor.txt.in 2013-05-03 20:22:26 +0000
4@@ -66,7 +66,7 @@
5
6 plugin: manual
7 name: monitor/multi-head
8-requires: dmi.product not in ['Notebook','Laptop','Portable']
9+requires: dmi.product in ['Desktop','Low Profile Desktop','Tower','Mini Tower']
10 _description:
11 PURPOSE:
12 This test verifies that multi-monitor output works on your desktop system. This is NOT the same test as the external monitor tests you would run on your laptop. If your video card does not support multiple monitors, please skip this test. You will need two monitors to perform this test.

Subscribers

People subscribed via source and target branches