Actions
action #64947
closed[desktop] remove workaround for boo#1086425 if possible / reduce duplicate handling / verify bug
Start date:
2020-03-28
Due date:
% Done:
0%
Estimated time:
Difficulty:
Description
Observation¶
https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/lib/x11test.pm#L35
is still using a workaround for bgo#657996 / https://bugzilla.suse.com/show_bug.cgi?id=1086425
with clicking to select the user name but the bug is reported as RESOLVED FIXED and also the workaround in code does not detect the bug being present.
Tis for example is used in https://openqa.suse.de/tests/4038867#step/change_password/48 . Also there seems to be a record_soft_failure
in lib/x11utils referencing https://bugzilla.suse.com/show_bug.cgi?id=1086425 that should also be handled.
Acceptance criteria¶
- AC1: Bug workaround is removed or only in place for product versions that need it with proper bug condition detection
- AC2: Bug is set to VERIFIED FIXED after verification
- AC3: No duplicate handling for bgo#657996 / https://bugzilla.suse.com/show_bug.cgi?id=1086425 in os-autoinst-distri-opensuse
Suggestions¶
- Remove https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/lib/x11test.pm#L34 and L35 and check if tests still work on all potentially affected versions, e.g. Tumbleweed, Leap, SLE15-SP2 as well as older versions
- If clicking on username is not necessary anymore in any version remove the code and set bug to VERIFIED FIXED
- If workaround still necessary for affected versions then add bug condition detection
- Reduce duplication with lib/x11utils to ensure the bug condition detection and workaround is controlled from a single code location
Further details¶
Always latest result in this scenario: latest
Actions