-
Notifications
You must be signed in to change notification settings - Fork 270
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
Add autoyast keyboard layout test #6833
Add autoyast keyboard layout test #6833
Conversation
586dd80
to
ea76776
Compare
ea76776
to
5e50289
Compare
5e50289
to
888aa94
Compare
888aa94
to
f679ee8
Compare
7361dd7
to
534fe66
Compare
All suggestions addressed. |
Don't merge before the needles are merged |
tests/locale/keymap_or_locale.pm
Outdated
|
||
if (check_var('DESKTOP', 'textmode')) { | ||
assert_screen([qw(linux-login cleared-console)]); | ||
verify_default_keymap_textmode($keystrokes, "${expected}_keymap"); | ||
return if verify_default_keymap_textmode($keystrokes, "${expected}_keymap"); |
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.
I'm not sure if this is clear, this condition is weird to me. I would split the method into two and make it explicit that if it's us
we don't run further checks.
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.
I created a function for that to make it more clear. VR updated.
534fe66
to
b439eb8
Compare
b439eb8
to
eb4ddf1
Compare
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.
main.pm
changes are ugly, but there is no other way as of now. We will address that with new scheduling approach.
I was not able to merge needles, because gitlab is down, so please, do not enable test suite yet =) |
For me is up, should I merge them myself? |
@jknphy , please, do so. I've checked those needles and they are fine ;) |
Add autoyast keyboard layout test
Requires new test suite
autoyast_keyboard_layout
similar toautoyast_mini
except for the following params:AUTOYAST=autoyast_sle15/autoyast_keyboard_layout.xml
INSTALL_KEYBOARD_LAYOUT=cz
AUTOYAST_CONFIRM=