action #19542: avoid type_string taking "too long" - openQA Project (public) - openSUSE Project Management Tool

Actions

Copy link

action #19542

closed

avoid type_string taking "too long"

Added by RBrownSUSE over 7 years ago. Updated about 5 years ago.

Status:

Resolved

Priority:

Low

Assignee:

asmorodskyi

Category:

Feature requests

Target version:

Done

Start date:

2017-06-02

Due date:

% Done:

Estimated time:

Description

Tests like https://openqa.suse.de/tests/976761#step/register_system/18 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.

Actions

Copy link

Updated by riafarov over 7 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 distribution.pm 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?

Actions

Copy link

Updated by RBrownSUSE over 7 years ago

The issue with the specific test has it's own ticket (https://progress.opensuse.org/issues/19540 ) 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?

Actions

Copy link

Updated by coolo about 7 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