Rotation test should be tested from normal to all orientation

Bug #1503167 reported by Po-Hsu Lin
6
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.get_current_screen()
    rotations = {'normal': xrandr.RR_ROTATE_0,
                 'right': xrandr.RR_ROTATE_90,
                 'inverted': xrandr.RR_ROTATE_180,
                 'left': xrandr.RR_ROTATE_270}
    rots_statuses = {}

    for rot in rotations:
        try:
            status = rotate_screen(rotations[rot])

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

Po-Hsu Lin (cypressyew)
affects: plainbox-provider-canonical-certification → plainbox-provider-checkbox
Po-Hsu Lin (cypressyew)
Changed in plainbox-provider-checkbox:
assignee: nobody → Po-Hsu Lin (cypressyew)
status: New → In Progress
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

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

Changed in plainbox-provider-checkbox:
importance: Undecided → Medium
milestone: none → 0.23
Po-Hsu Lin (cypressyew)
Changed in plainbox-provider-checkbox:
status: In Progress → Fix Committed
Pierre Equoy (pieq)
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.