action #19542

avoid type_string taking "too long"

Added by RBrownSUSE about 6 years ago. Updated over 3 years ago.

Feature requests
Target version:
Start date:
Due date:
% Done:


Estimated time:


Tests like show that very long type_strings can be problematic

They will always be delicate to problems when they get 'too long' (such as 'over 200 characters'), and we should probably implement a log warning so if tests fail because of missing characters or keyboard buffers being full it the log reader can at least have a suggestion to trim the likely cause of failure and make their tests more resilient.


#1 Updated by riafarov about 6 years ago

Hi Richard,
In the failed test we actually call assert_script_run function. At first I was thinking about passing wait_screen_change parameter to it and then to underlying type_string call in script_run method in Do you think it's fine if we kill two birds with one stone, so we add warning when string is long and if it's over 200 characters we enable wait_screen_change option. Does it make sense?

#2 Updated by RBrownSUSE about 6 years ago

The issue with the specific test has it's own ticket ( ) and I've asked the maintainer involved to look at it :)

One of the suggestions I gave Wei Jiang was to use wait_screen_change :) I think it would be the best way also.

If you really want to do that work for him, I will not object, but please mail him to let him know you are helping because I mailed him directly to make him aware of the ticket.

With the test specific issue being dealt with in it's own ticket, this just leaves the WARN for long type_strings, hence this seperate [tools] ticket. Does that make sense?

#3 Updated by coolo over 5 years ago

  • Subject changed from [tools] WARN: when type_string is "too long" to avoid type_string taking "too long"
  • Category set to 136
  • Priority changed from Normal to Low
  • Target version set to Ready

I don't see any point in warning - if we think type_string shouldn't be used for too long, we should just inject something in type_string ourselves.

#4 Updated by okurz almost 4 years ago

  • Category changed from 136 to Feature requests

#5 Updated by okurz almost 4 years ago

  • Status changed from New to Resolved
  • Assignee set to asmorodskyi

#6 Updated by coolo over 3 years ago

  • Target version changed from Ready to Done

Also available in: Atom PDF