From d3967bfdedd8a4a27804690043c1b8d4d8066334 Mon Sep 17 00:00:00 2001 From: Mitch Curtis Date: Wed, 18 Apr 2018 15:21:57 +0200 Subject: 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 --- tests/auto/controls/data/tst_popup.qml | 6 +++--- 1 file 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) } -- cgit v1.2.3