action #54425

openQA Project - action #55703: send_keys() doesn't guarantee that the application will handle modifiers+normal_key properly

[functional][u] hexchat: ctrl-a mis-interpreted as 'a' since we switched to QEMUVGA "std"

Added by dimstar 9 months ago. Updated 4 months ago.

Status:ResolvedStart date:18/07/2019
Priority:NormalDue date:
Assignee:dimstar% Done:

0%

Category:Bugs in existing testsEstimated time:42.00 hours
Target version:SUSE QA tests - Milestone 28
Difficulty:
Duration:

Description

Observation

openQA test in scenario opensuse-Tumbleweed-DVD-x86_64-gnome@64bit fails in
hexchat

Acceptance criteria

  • AC: The failure rate has been reduced to the minimum possible.

Suggestions

  • Use a prepared configuration file to skip typing connection information.

Reproducible

Fails since (at least) Build 20190702

Expected result

Last good: 20190701 (or more recent)

Further details

Always latest result in this scenario: latest

After clicking the window, we are supposedly pressing ctrl-a, followed by delete - but looking at the screenshot, 'ctrl' was missed, 'a' typed (and 'delete as no effect at the end of the word)

History

#1 Updated by SLindoMansilla 9 months ago

  • Subject changed from hexchat: ctrl-a mis-interpreted as 'a' to [functional][u] hexchat: ctrl-a mis-interpreted as 'a'
  • Priority changed from Normal to High

#2 Updated by SLindoMansilla 8 months ago

  • Description updated (diff)

#3 Updated by SLindoMansilla 8 months ago

  • Parent task set to #55703

#4 Updated by dimstar 7 months ago

A proposal: https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/8382

avoids using ctrl-a, but rather sends 'home' and then 'delete' until the input box is empty

#5 Updated by coolo 7 months ago

IMO this fails since GNOME tests runs under wayland (i.e. we removed the cirrus setting)

#6 Updated by okurz 7 months ago

  • Subject changed from [functional][u] hexchat: ctrl-a mis-interpreted as 'a' to [functional][u] hexchat: ctrl-a mis-interpreted as 'a' since we switched to QEMUVGA "std"

yes, this is correct.

#7 Updated by SLindoMansilla 7 months ago

  • Description updated (diff)

PR was merged

#8 Updated by SLindoMansilla 7 months ago

  • Status changed from New to Workable
  • Priority changed from High to Normal
  • Target version set to Milestone 28
  • Estimated time set to 42.00

#9 Updated by SLindoMansilla 5 months ago

Should be fixed by: https://github.com/os-autoinst/os-autoinst/pull/1260/files

To resolve, provide verification runs, set a proper value for VNC_TYPING_LIMIT if necessary.

#10 Updated by okurz 4 months ago

  • Project changed from openQA Project to openQA Tests
  • Category set to Bugs in existing tests
  • Status changed from Workable to Resolved
  • Assignee set to dimstar

Actually this has been resolved with DimStar's change in tests: fe4a4b7b7 , https://openqa.opensuse.org/tests/1107110#next_previous shows that this is stable now for 7 jobs in a row

Also available in: Atom PDF