action #108380
closedcoordination #108290: [Epic] Improve installation test modules where libyui REST API is not applicable
Replace select_patterns when selecting only YaST Development pattern
Description
We will work on this one after https://progress.opensuse.org/issues/108293
Motivation¶
Execution time of this test is huge, it iterates a list that sometimes is quite long, https://openqa.suse.de/tests/8291661#step/select_patterns/127 checking items one by one. It is an old module using shortcuts and needles and quite generic.
Acceptance criteria¶
AC1: Use alternative path to select that pattern.
AC2: Map the screen using POM but use needles/shortcuts instead of libyui REST API calls.
Further information¶
- There are simple ways to achieve the same, we can do the following:
- Focus on the list (alt-tab)
- Send alt-end to go to the end of the list as it is the last pattern
Select pattern 'YaST Development':
- Click on item with a needle
- send alt-+
- Test assume that this pattern to be selected is at the end of the list, worth to add on the description of the the new test.
- We might try to avoid the complexity of having a controller or a distri provider for now, just implement the Pages for simplicity. In future we can think about if we really want more layers.
Updated by JERiveraMoya almost 3 years ago
- Tags deleted (
qe-yast-refinement) - Description updated (diff)
- Status changed from New to Workable
Updated by leli about 2 years ago
- Status changed from Workable to In Progress
- Assignee set to leli
Updated by leli about 2 years ago
Created branch, run job to verify: https://openqa.nue.suse.com/tests/9707628#step/select_patterns/8
Updated by leli about 2 years ago
Updated by JERiveraMoya about 2 years ago
- Related to action #108293: Replace select_patterns when unselecting all patterns except for "base" and "minimal added
Updated by leli about 2 years ago
PR https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/15693 already merged, while we still need MR to remove 'default' from PATTERNS, and another PR to remove code of the removal of default.
Updated by leli about 2 years ago
We do not need to file MR since the setting is in the test suite of select_modules_and_patterns+registration, and I have removed 'default' pattern from test suite. This is the PR https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/15813 to remove the not needed code line to remove 'default'.