QA » openQA Project » openQA Tests

Since the fail appeared with similarity level 47 I would suppose to set the level to 45 and check if this has unwanted side effects

Created PR:
https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/5500
This should avoid wasting time by waiting for blinking cursors

Next I will try to get wait_still_screen to tell us when it timed out so we see these situations better

Regarding the first suggestion we should not change wait_still_screen itself but try to check the return values when it is really critical
Also we have Log Entries so not sure, how to continue here

PR merged, great! Yes, you are right in all your points. We could go with changing "wait_still_screen" to "assert_still_screen" by default and only use "wait_still_screen" or a hypothetical "check_still_screen" only in cases we want to explicitly check the return value. I already tried this with a new function assert_screen_change which is part of os-autoinst already doing the equivalent for wait_screen_change but I think it does not even work properly. When you want you can simply close the ticket or create new feature requests on the openQA issue tracker to have assert_screen_change and assert_still_screen functions with proper unit tests in os-autoinst. As a ticket first but if you are motivated we can also bring it into our backlog and work on that.

Created https://progress.opensuse.org/issues/39116 as a followup
Will add an output of the real similarity level to the logs and then close this

To print the last similarity level I would have to change the scope of $sim out of the loop