Stabilize Graphics View tests

Reviewed-by: Jesper

1 files changed, 42 insertions, 28 deletions

diff --git a/tests/auto/qgraphicswidget/tst_qgraphicswidget.cpp b/tests/auto/qgraphicswidget/tst_qgraphicswidget.cpp
index 5296788..9045acf 100644
--- a/tests/auto/qgraphicswidget/tst_qgraphicswidget.cpp
+++ b/tests/auto/qgraphicswidget/tst_qgraphicswidget.cpp
@@ -230,7 +230,7 @@ public:
 
     int eventCount;
     Qt::LayoutDirection m_painterLayoutDirection;
-    
+
     void paint(QPainter *painter, const QStyleOptionGraphicsItem *option, QWidget *widget)
     {
         m_painterLayoutDirection = painter->layoutDirection();
@@ -276,15 +276,15 @@ class SizeHinter : public QGraphicsWidget
 {
 public:
     SizeHinter(QGraphicsItem *parent = 0, Qt::WindowFlags wFlags = 0,
-                const QSizeF &min = QSizeF(5,5), 
-                const QSizeF &pref = QSizeF(50, 50), 
-                const QSizeF &max = QSizeF(500, 500)) 
-        : QGraphicsWidget(parent, wFlags) 
+                const QSizeF &min = QSizeF(5,5),
+                const QSizeF &pref = QSizeF(50, 50),
+                const QSizeF &max = QSizeF(500, 500))
+        : QGraphicsWidget(parent, wFlags)
     {
         m_sizes[Qt::MinimumSize] = min;
         m_sizes[Qt::PreferredSize] = pref;
         m_sizes[Qt::MaximumSize] = max;
-        
+
     }
     void setSizeHint(Qt::SizeHint which, const QSizeF &newSizeHint)
     {
@@ -324,11 +324,11 @@ void tst_QGraphicsWidget::qgraphicswidget()
     QCOMPARE(widget.type(), (int)QGraphicsWidget::Type);
     QCOMPARE(widget.call_propertyChange(QString(), QVariant()), QVariant());
     widget.call_sizeHint(Qt::PreferredSize, QSizeF());
-    
+
     QGraphicsScene scene;
     QGraphicsWidget *parent = new QGraphicsWidget;
     SizeHinter *child = new SizeHinter(parent);
-    
+
     QCOMPARE(child->minimumSize(), QSizeF(5, 5));
 }
 
@@ -482,7 +482,7 @@ void tst_QGraphicsWidget::focusWidget2()
     QVERIFY(!widget->hasFocus());
     QVERIFY(!subWidget->hasFocus());
 
-    widget->setFocus();    
+    widget->setFocus();
 
     QVERIFY(widget->hasFocus());
     QCOMPARE(focusInSpy.count(), 1);
@@ -702,7 +702,7 @@ void tst_QGraphicsWidget::fontPropagationSceneChange()
     QFont font;
     font.setPointSize(47);
     scene.setFont(font);
-    
+
     QFont font2;
     font2.setPointSize(74);
     scene2.setFont(font2);
@@ -799,7 +799,9 @@ void tst_QGraphicsWidget::initStyleOption()
 #ifdef Q_WS_X11
     qt_x11_wait_for_window_manager(&view);
 #endif
-    QTest::qWait(250);
+    QApplication::setActiveWindow(&view);
+    QTest::qWait(25);
+    QTRY_COMPARE(QApplication::activeWindow(), &view);
 
     view.setAlignment(Qt::AlignTop | Qt::AlignLeft);
     SubQGraphicsWidget *widget = new SubQGraphicsWidget;
@@ -1132,6 +1134,9 @@ void tst_QGraphicsWidget::setTabOrder()
 #ifdef Q_WS_X11
     qt_x11_wait_for_window_manager(&view);
 #endif
+    QApplication::setActiveWindow(&view);
+    QTest::qWait(25);
+    QTRY_COMPARE(QApplication::activeWindow(), &view);
 
     QGraphicsWidget *lastItem = 0;
     QTest::ignoreMessage(QtWarningMsg, "QGraphicsWidget::setTabOrder(0, 0) is undefined");
@@ -1196,7 +1201,9 @@ void tst_QGraphicsWidget::setTabOrderAndReparent()
 #ifdef Q_WS_X11
     qt_x11_wait_for_window_manager(&view);
 #endif
-    QTest::qWait(250);
+    QApplication::setActiveWindow(&view);
+    QTest::qWait(25);
+    QTRY_COMPARE(QApplication::activeWindow(), &view);
 
     int i;
     QGraphicsWidget *w1, *w2, *w3, *w4;
@@ -1214,29 +1221,29 @@ void tst_QGraphicsWidget::setTabOrderAndReparent()
     }
 
     w1->setFocus();
-    QVERIFY(w1->hasFocus());
+    QTRY_VERIFY(w1->hasFocus());
     QVERIFY(compareFocusChain(&view, QList<QGraphicsItem*>() << w1 << w2 << w3));
 
     QGraphicsWidget *p = new QGraphicsWidget;
     p->setData(0, QLatin1String("parent"));
     p->setFocusPolicy(Qt::StrongFocus);
-    
+
     w1->setFocus();
     QVERIFY(compareFocusChain(&view, QList<QGraphicsItem*>() << w1 << w2 << w3));
-    
+
     w1->setParentItem(p);
     w2->setFocus();
     QVERIFY(compareFocusChain(&view, QList<QGraphicsItem*>() <<  w2 << w3));
-    
+
     w2->setParentItem(p);
     w3->setFocus();
     QVERIFY(compareFocusChain(&view, QList<QGraphicsItem*>() << w3));
     w3->setParentItem(p);
     QCOMPARE(scene.focusItem(), static_cast<QGraphicsItem*>(0));
-    
+
     scene.addItem(p);
     p->setFocus();
-    
+
     QVERIFY(compareFocusChain(&view, QList<QGraphicsItem*>() << p << w1 << w2 << w3));
     delete p;
 
@@ -1258,7 +1265,7 @@ void tst_QGraphicsWidget::setTabOrderAndReparent()
     QGraphicsWidget::setTabOrder(w1, w4);
     w1->setFocus();
     QVERIFY(compareFocusChain(&view, QList<QGraphicsItem*>() << w1 << w4 << w2 << w3));
-    
+
     p = new QGraphicsWidget;
     p->setData(0, QLatin1String("parent"));
     p->setFocusPolicy(Qt::StrongFocus);
@@ -1266,7 +1273,7 @@ void tst_QGraphicsWidget::setTabOrderAndReparent()
     w1->setParentItem(p);
     w2->setFocus();
     QVERIFY(compareFocusChain(&view, QList<QGraphicsItem*>() << w2 << w3));
-    
+
     scene.addItem(p);
     w2->setFocus();
     QVERIFY(compareFocusChain(&view, QList<QGraphicsItem*>() << w2 << w3 << p << w1 << w4));
@@ -1337,7 +1344,10 @@ void tst_QGraphicsWidget::verifyFocusChain()
 #ifdef Q_WS_X11
     qt_x11_wait_for_window_manager(&view);
 #endif
-    QTest::qWait(250);
+    QApplication::setActiveWindow(&view);
+    QTest::qWait(25);
+    QTRY_COMPARE(QApplication::activeWindow(), &view);
+
     {
         // parent/child focus
         SubQGraphicsWidget *w = new SubQGraphicsWidget(0, Qt::Window);
@@ -1412,7 +1422,7 @@ void tst_QGraphicsWidget::verifyFocusChain()
 #ifdef Q_WS_X11
         qt_x11_wait_for_window_manager(window);
 #endif
-        
+
         lineEdit->setFocus();
         QTest::qWait(250);
         QVERIFY(lineEdit->hasFocus());
@@ -1449,7 +1459,7 @@ void tst_QGraphicsWidget::verifyFocusChain()
         w1_4->setFocusPolicy(Qt::StrongFocus);
         w1_4->setData(0, "w1_4");
         w1_4->setGeometry(75,0,25, 25);
-        scene.addItem(w1_4);        
+        scene.addItem(w1_4);
         QVERIFY(w1_3->hasFocus());
         QTest::qWait(250);
         QVERIFY(compareFocusChain(view, QList<QGraphicsItem*>() << w1_3 << w1_4));
@@ -1459,7 +1469,7 @@ void tst_QGraphicsWidget::verifyFocusChain()
         // tabFocusFirst should now point to w1_3
         QTest::keyPress(QApplication::focusWidget(), Qt::Key_Tab);
         QTest::qWait(250);
-        QVERIFY(w1_3->hasFocus());        
+        QVERIFY(w1_3->hasFocus());
         QTest::qWait(250);
         QVERIFY(compareFocusChain(view, QList<QGraphicsItem*>() << w1_3 << w1_4));
         delete window;
@@ -1474,7 +1484,9 @@ void tst_QGraphicsWidget::updateFocusChainWhenChildDie()
 #ifdef Q_WS_X11
     qt_x11_wait_for_window_manager(&view);
 #endif
-    QTest::qWait(250);
+    QApplication::setActiveWindow(&view);
+    QTest::qWait(25);
+    QTRY_COMPARE(QApplication::activeWindow(), &view);
 
     // delete item in focus chain with no focus and verify chain
     SubQGraphicsWidget *parent = new SubQGraphicsWidget(0, Qt::Window);
@@ -1628,7 +1640,7 @@ void tst_QGraphicsWidget::setSizes_data()
                                  << (QVector<Inst>() << Inst(MinimumSize, QSizeF(5, 5)));
     QTest::newRow("unsetMaxSize")<< (QVector<Inst>() << Inst(Size, QSizeF(40, 40)) << Inst(MaximumSize, QSizeF(-1, -1)))
                                  << (QVector<Inst>() << Inst(MaximumSize, QSizeF(500, 500)));
-    QTest::newRow("unsetMinSize, expand size to minimumSizeHint") << (QVector<Inst>() 
+    QTest::newRow("unsetMinSize, expand size to minimumSizeHint") << (QVector<Inst>()
                                         << Inst(MinimumSize, QSize(0, 0))
                                         << Inst(Size, QSize(1,1))
                                         << Inst(MinimumSize, QSize(-1.0, -1.0))
@@ -1708,7 +1720,7 @@ void tst_QGraphicsWidget::setSizes()
     widget->setMaximumSize(max);
 
     QApplication::processEvents();
-    
+
     for (i = 0; i < compareInstructions.count(); ++i) {
         Inst input = compareInstructions.at(i);
         switch (input.first) {
@@ -2432,7 +2444,9 @@ void tst_QGraphicsWidget::task250119_shortcutContext()
     QGraphicsView view;
     view.setScene(&scene);
     view.show();
-    QTest::qWait(100);
+    QApplication::setActiveWindow(&view);
+    QTest::qWait(25);
+    QTRY_COMPARE(QApplication::activeWindow(), &view);
 
 
     // *** Event: ***