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
Slow type and run very long command in scripted way #3161
Conversation
looks good, but I wouldn't make it arch specific. The type_string is already slower on power, so switching to current_script for all archs would be more consistent in reviewing/error handling |
8578a46
to
31cbcd8
Compare
Ok, hope that it would not spent long time on checking dozen repo urls with multiple addons registered :) |
lib/utils.pm
Outdated
@@ -66,6 +66,7 @@ our @EXPORT = qw( | |||
service_action | |||
assert_gui_app | |||
install_all_from_repo | |||
slow_run_scripted_commnad |
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.
s/comnad/command/, or maybe even better name: run_scripted_command_slow
to sound similar to type_string_slow
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.
that sounds good. naming is hard to me sometimes :)
It aims to avoid 'key event queue full' issue. For the detail, see: https://progress.opensuse.org/issues/19540
31cbcd8
to
85661ef
Compare
LGTM |
I assume https://openqa.suse.de/tests/1037812#step/zypper_lr/11 is fixed by #3204 ? |
Yes, see: https://openqa.suse.de/tests/1038285#step/repos_check/10 |
It aims to avoid 'key event queue full' issue:
https://openqa.suse.de/tests/1020607
For the detail, see:
https://progress.opensuse.org/issues/19540
Verified run:
http://147.2.207.208/tests/603