diff options
author | Dimitrios Apostolou <dimitrios.apostolou@qt.io> | 2019-07-18 19:02:40 +0200 |
---|---|---|
committer | Dimitrios Apostolou <dimitrios.apostolou@qt.io> | 2019-07-22 12:08:22 +0200 |
commit | 1d06eb3f8215b67c5061ee3a076df405724ff7ee (patch) | |
tree | fcb3f958c698f69f1cf0714753c2cd3cec3cb13d /tests/auto/qquickmenubar/tst_qquickmenubar.cpp | |
parent | ce7c431fb23157fa5125d1102a594de045818a72 (diff) |
Fix Flaky tests
These tests where failing sometimes in our CI, because centerOnScreen()
was invoked which asks the window manager to move the window, but does
not wait for the event's completion. The solution is to do all that
*before* the initial qWaitForWindowActive().
Fixes: QTBUG-73128
Change-Id: I06c1f85d5ff9657dccab50f29084f7624cd7e194
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
Diffstat (limited to 'tests/auto/qquickmenubar/tst_qquickmenubar.cpp')
-rw-r--r-- | tests/auto/qquickmenubar/tst_qquickmenubar.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/auto/qquickmenubar/tst_qquickmenubar.cpp b/tests/auto/qquickmenubar/tst_qquickmenubar.cpp index 19d67eac..9a22d26f 100644 --- a/tests/auto/qquickmenubar/tst_qquickmenubar.cpp +++ b/tests/auto/qquickmenubar/tst_qquickmenubar.cpp @@ -85,10 +85,10 @@ void tst_qquickmenubar::mouse() QScopedPointer<QQuickApplicationWindow> window(qobject_cast<QQuickApplicationWindow *>(engine.rootObjects().value(0))); QVERIFY(window); - QVERIFY(QTest::qWaitForWindowActive(window.data())); centerOnScreen(window.data()); moveMouseAway(window.data()); + QVERIFY(QTest::qWaitForWindowActive(window.data())); QQuickMenuBar *menuBar = window->property("header").value<QQuickMenuBar *>(); QVERIFY(menuBar); @@ -224,10 +224,10 @@ void tst_qquickmenubar::keys() QScopedPointer<QQuickApplicationWindow> window(qobject_cast<QQuickApplicationWindow *>(engine.rootObjects().value(0))); QVERIFY(window); - QVERIFY(QTest::qWaitForWindowActive(window.data())); centerOnScreen(window.data()); moveMouseAway(window.data()); + QVERIFY(QTest::qWaitForWindowActive(window.data())); QQuickMenuBar *menuBar = window->property("header").value<QQuickMenuBar *>(); QVERIFY(menuBar); @@ -412,10 +412,10 @@ void tst_qquickmenubar::mnemonics() QScopedPointer<QQuickApplicationWindow> window(qobject_cast<QQuickApplicationWindow *>(engine.rootObjects().value(0))); QVERIFY(window); - QVERIFY(QTest::qWaitForWindowActive(window.data())); centerOnScreen(window.data()); moveMouseAway(window.data()); + QVERIFY(QTest::qWaitForWindowActive(window.data())); QQuickMenuBar *menuBar = window->property("header").value<QQuickMenuBar *>(); QVERIFY(menuBar); |