aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMitch Curtis <mitch.curtis@qt.io>2018-04-18 15:21:57 +0200
committerMitch Curtis <mitch.curtis@qt.io>2018-04-18 13:49:16 +0000
commitd3967bfdedd8a4a27804690043c1b8d4d8066334 (patch)
treed8401237edefbec78f97f3c0f9f5b719a20bc6b8
parentf722ebc09a6d0336bd3bc86ef7e621932adb5064 (diff)
Popup::test_shortcut(): use "A" instead of "Tab" for shortcut
The recently merged stabilization patch showed that the shortcut isn't delivered, and the logs show some focus stuff going on before it fails. If the flakiness is related to focus handling as a result of the tab press, we can try a different key. Change-Id: I6094f11de9d938fd18ad5af8b39b0a1489443638 Reviewed-by: J-P Nurmi <jpnurmi@qt.io>
-rw-r--r--tests/auto/controls/data/tst_popup.qml6
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/auto/controls/data/tst_popup.qml b/tests/auto/controls/data/tst_popup.qml
index 2a6d585c..559e3b59 100644
--- a/tests/auto/controls/data/tst_popup.qml
+++ b/tests/auto/controls/data/tst_popup.qml
@@ -1287,7 +1287,7 @@ TestCase {
Shortcut {
id: shortcut
- sequence: "Tab"
+ sequence: "A"
onActivated: popup.visible = !popup.visible
}
}
@@ -1305,11 +1305,11 @@ TestCase {
verify(shortcutActivatedSpy.valid)
waitForRendering(window.contentItem)
- keyClick(Qt.Key_Tab)
+ keyClick(Qt.Key_A)
compare(shortcutActivatedSpy.count, 1)
tryCompare(control, "visible", true)
- keyClick(Qt.Key_Tab)
+ keyClick(Qt.Key_A)
compare(shortcutActivatedSpy.count, 2)
tryCompare(control, "visible", false)
}