action #104769
closedcoordination #93668: [epic] Redesign Installation test modules
Split welcome in gnome_http, gnome_smb & repo_inst to use libyui-rest-api
Description
We have a couple of checks to ensure that the install source matches boot parameters and linuxrc settings:
https://openqa.suse.de/tests/7922921#step/welcome/9
https://openqa.suse.de/tests/7922923#step/welcome/9
https://openqa.suse.de/tests/7924022#step/welcome/9
Corresponding code: https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/tests/installation/welcome.pm#L183-L196
AC1: Reuse existing test to automate, i.e.: installation/access_beta_distribution
, installation/product_selection/install_SLES
.
Once the split is down, we can proceed to create separate test suite for gnome_self_signed_https.
AC2: Create one new validation test that change virtual console and perform those checks
(so it doesn't require needles and can check the ret code from those two commands).
AC3: Check if that code is used by other team and if not, remove it.
AC4: Remove unnecessary settings/reduce number of extra settings.
Note: better doing it from scratch using ret code than trying to create lib to be used in existing code and in new test (even if it is repeated code, also could be tricky to find where is running because it is welcome and it is everywhere in many shapes).
Updated by JERiveraMoya almost 3 years ago
- Priority changed from Normal to High
- Target version set to Current
Updated by JERiveraMoya almost 3 years ago
- Tags deleted (
qe-yast-refinement) - Status changed from New to Workable
Updated by JERiveraMoya almost 3 years ago
- Subject changed from Split welcome in gnome_http & gnome_smb to use libyui-rest-api to Split welcome in gnome_http, gnome_smb & repo_inst to use libyui-rest-api
- Description updated (diff)
Updated by JRivrain almost 3 years ago
- Status changed from Workable to In Progress
Updated by JRivrain almost 3 years ago
PR https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/14235
I decided not to use different modules for validation of repos, as anyway we rely on variable INSTALL_SOURCE which is needed for grub, so I think it's okay to use it in that case, with a record_info to show what is expected.
Updated by JRivrain almost 3 years ago
- Status changed from In Progress to Feedback