New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement system role selection without shortcut usage #4674
Conversation
why again do you mention that there is no scenario where a "non-default" role is selected when you mention textmode+kvm_server_role? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- one word missing in comment
- please see the PR description comment
- please see comment in needle MR about typo
rest is fine. If you fixed all three feel free to directly merge both needle MR and this PR
tests/installation/system_role.pm
Outdated
@@ -26,8 +27,22 @@ my %role_hotkey = ( | |||
|
|||
sub change_system_role { | |||
my ($system_role) = @_; | |||
send_key 'alt-' . $role_hotkey{$system_role}; | |||
assert_screen "system-role-$system_role-selected"; | |||
# Since SLE 15 do not have shortcuts for system roles anymore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since SLE 15 we do not …
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
As a result of [bsc#1084674](https://bugzilla.suse.com/show_bug.cgi?id=1084674), now we cannot use shortkeys to select system role. Hence, click in case of x11 installation and using navigation keys for ncurses. See [poo#33562](https://progress.opensuse.org/issues/33562).
c58269f
to
79bf202
Compare
@okurz all points fixed, I'll merge once travis is done ;) |
@rwx788 when you merge this PR, could you also merge this MR: https://gitlab.suse.de/openqa/os-autoinst-needles-sles/merge_requests/780 If you want I can merge your needles as well. :) (NVM ... they're already merged) |
As a result of bsc#1084674,
now we cannot use shortkeys to select system role.
Hence, click in case of x11 installation and using navigation keys for ncurses.
See poo#33562.