Project

General

Profile

Actions

action #64947

closed

[desktop] remove workaround for boo#1086425 if possible / reduce duplicate handling / verify bug

Added by okurz about 4 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Bugs in existing tests
Target version:
-
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


Related issues 1 (0 open1 closed)

Is duplicate of openQA Tests - action #62366: [functional][u][svirt-xen-pv] test fails in shutdown - Misfunction of workaround to login in GDM with unselected user itemRejectedszarate2020-01-20

Actions
Actions #1

Updated by zcjia almost 4 years ago

  • Assignee set to zcjia

I'll look into this.

Actions #2

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

Actions #3

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
Actions #4

Updated by maritawerner almost 3 years ago

Ok, 1 year later, desktop teaodo you still have this on the radar?

Actions #5

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.

Actions #6

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:

  1. The original bug was fixed.
  2. Our test code (gdm login) is used in various cases, we should use mouse to login as a reasonable fallback instead of softfail.
Actions

Also available in: Atom PDF