-
Notifications
You must be signed in to change notification settings - Fork 273
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
Handle java license popup during installation #16346
Conversation
e929af8
to
1ae5b5e
Compare
1ae5b5e
to
6f92eaa
Compare
I'd recommend to run verifications for all 4 test suite failing, just to be safe |
@@ -0,0 +1,46 @@ | |||
# SUSE's openQA tests | |||
# | |||
# Copyright 2021 SUSE LLC |
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.
2023 and rest of the files.
|
||
sub run { | ||
my $package_license_popup = $testapi::distri->get_accept_popup_controller(); | ||
$package_license_popup->wait_accept_popup({ |
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.
we don't need to wait for some action in progress to capture this popup, it appears immediately after clicking install, only accept should work.
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.
It doesn't work without the wait
In YaST maintenance job group, there has been a java license popup for tests using dud from generate_dud test suit. In this draft, a module running before confirm_installation can handle the popup.
There should be new Yaml schedules as the popup does not appear for all Service packs (same tests run for more than one SP)
VR: http://falafel.suse.cz/tests/904