action #64947
closed[desktop] remove workaround for boo#1086425 if possible / reduce duplicate handling / verify bug
0%
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
Updated by maritawerner almost 4 years ago
Update from the weekly qa status meeting: the desktop team is looking into that, issue is not forgotten
Updated by SLindoMansilla almost 4 years ago
- Is duplicate of action #62366: [functional][u][svirt-xen-pv] test fails in shutdown - Misfunction of workaround to login in GDM with unselected user item added
Updated by maritawerner almost 3 years ago
Ok, 1 year later, desktop teaodo you still have this on the radar?
Updated by zcjia over 2 years ago
- Status changed from Workable to In Progress
Checked the history of this issue's context, the bug referenced is no longer applicable because the design of GNOME has evolved.
Will check different SLE/TW versions to make sure the check can be removed safely.
Updated by zcjia over 2 years ago
- Status changed from In Progress to Resolved
PR https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/13009 merged.
Closing.
Final summary:
- The original bug was fixed.
- Our test code (gdm login) is used in various cases, we should use mouse to login as a reasonable fallback instead of softfail.