monitor/dim_brightness test case is untested in AIO

Bug #1371443 reported by Jerry Kao
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox Provider - Base
Fix Released
Medium
Daniel Manrique

Bug Description

When test with AIO, monitor/dim_brightness test case is untested. The detail message is as following
job cannot be started: resource expression "dmi.product in ['Notebook','Laptop','Portable']" evaluates to false

Related branches

Jerry Kao (jerry.kao)
tags: added: ce-qa-concern
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

We have a few options here:

1) Test brightness on all of the machines
2) Try to detect if X exposes the brightness control interface
3) Try to detect if sysfs exposes the brightness control interface
4) Use DMI to check for "AIO" class systems (not sure if DMI supports that)
5) Work on manifests and derive this information from the manifest

affects: checkbox → plainbox-provider-checkbox
Changed in plainbox-provider-checkbox:
status: New → Triaged
Revision history for this message
Daniel Manrique (roadmr) wrote :

dmi.product is somewhat unreliable, another solution (and one which would take care of other issues too) is to come up with a dedicated form_factor resource and map all those disparate DMI types (pizza box is a valid dmi type) into form factors we care about.

Finally, this test should probably depend on having a built-in screen, not on the form factor itself, so we could also try to detect that (e.g. LVDS1 and eDP1 as seen by xrandr are usually built-in screens).

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

Jerry, could you please run these commands on your AIO and attach the resulting files? (Just so we have an idea of what their data looks like).

sudo dmidecode > dmidecode.log
xrandr -q > xrandr.log

Changed in plainbox-provider-checkbox:
importance: Undecided → Medium
Revision history for this message
Jerry Kao (jerry.kao) wrote :
Revision history for this message
Jerry Kao (jerry.kao) wrote :
Revision history for this message
Daniel Manrique (roadmr) wrote :

OK, the chassis is :

Handle 0x0003, DMI type 3, 25 bytes
Chassis Information
    Manufacturer: Hewlett-Packard
    Type: All In One
    Lock: Not Present

And the screen is eDP1:

Screen 0: minimum 8 x 8, current 1920 x 1080, maximum 16384 x 16384
eDP1 connected 1920x1080+0+0 509mm x 286mm
VGA1 disconnected
VIRTUAL1 disconnected

So the easy solution which we can implement very quickly is to modify monitor.txt.in and add "All in One" to the list of form factors:

plugin: user-interact-verify
id: monitor/dim_brightness
requires: dmi.product in ['Notebook','Laptop','Portable', 'All In One']

The other solutions, including manifests, requiring a hypothetical new display_type resource:

requires display_type.type == 'Built-in'

and verifying the kernel interfaces Zygmunt mentioned, would require a bit more work.

Daniel Manrique (roadmr)
Changed in plainbox-provider-checkbox:
status: Triaged → In Progress
assignee: nobody → Daniel Manrique (roadmr)
milestone: none → 0.16
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Committed
Changed in plainbox-provider-checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.