Rotation test should be tested from normal to all orientation
Bug #1503167 reported by
Po-Hsu Lin
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox Provider - Base |
Fix Released
|
Medium
|
Po-Hsu Lin |
Bug Description
Currently we are testing the display rotation with:
def main():
screen = xrandr.
rotations = {'normal': xrandr.RR_ROTATE_0,
rots_statuses = {}
for rot in rotations:
try:
status = rotate_
I think we should test it explicitly with:
normal -> right
normal -> inverted
normal -> left
So that we will be able to capture issues like: bug 1446979 and bug 1443345.
And it's more close to what a normal user will do on their system.
Related branches
lp:~cypressyew/checkbox/rotation-from-normal
- Sylvain Pineau: Approve
-
Diff: 22 lines (+6/-6)1 file modifiedproviders/plainbox-provider-checkbox/bin/rotation_test_using_dbus (+6/-6)
affects: | plainbox-provider-canonical-certification → plainbox-provider-checkbox |
Changed in plainbox-provider-checkbox: | |
assignee: | nobody → Po-Hsu Lin (cypressyew) |
status: | New → In Progress |
Changed in plainbox-provider-checkbox: | |
importance: | Undecided → Medium |
milestone: | none → 0.23 |
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.
With Pierre's new rotation script (with dbus) it's super easy to do this.
Just need to change the orientation index from
for i in 2 4 8 1
to
for i in 2 1 4 1 8 1