Remove insignificant_test for tst_qwindow on Mac OS X

tst_QWindow::positioning() still fails on Mac OS X, and it does so in
several places. Skip this test for now as it causes isActive() to fail
as well. With positioning() QSKIP()ed, all the other test functions
pass:

Totals: 19 passed, 0 failed, 1 skipped
********* Finished testing of tst_QWindow *********

Task-number: QTBUG-23059
Change-Id: I58d036120c0121f515813cd20955ab3b82f81fe1
Reviewed-by: Morten Johan Sørvig <morten.sorvig@nokia.com>

diff --git a/tests/auto/gui/kernel/qwindow/qwindow.pro b/tests/auto/gui/kernel/qwindow/qwindow.pro
index 363f7dd..e419a10 100644 (file)
--- a/tests/auto/gui/kernel/qwindow/qwindow.pro
+++ b/tests/auto/gui/kernel/qwindow/qwindow.pro
@@ -4,6 +4,3 @@ TARGET = tst_qwindow
 QT += core-private gui-private testlib
 
 SOURCES  += tst_qwindow.cpp
-
-mac: CONFIG += insignificant_test # QTBUG-23059
-

diff --git a/tests/auto/gui/kernel/qwindow/tst_qwindow.cpp b/tests/auto/gui/kernel/qwindow/tst_qwindow.cpp
index 55a9a39..73f70fb 100644 (file)
--- a/tests/auto/gui/kernel/qwindow/tst_qwindow.cpp
+++ b/tests/auto/gui/kernel/qwindow/tst_qwindow.cpp
@@ -162,6 +162,12 @@ void tst_QWindow::eventOrderOnShow()
 
 void tst_QWindow::positioning()
 {
+#ifdef Q_OS_MAC
+    // the fullscreen animation delay on OS X Lion also causes failures in
+    // the isActive() test below, so it's best to just skip it for now
+    QSKIP("Multiple failures in this test on Mac OS X, see QTBUG-23059");
+#endif
+
     // Some platforms enforce minimum widths for windows, which can cause extra resize
     // events, so set the width to suitably large value to avoid those.
     QRect geometry(80, 80, 300, 40);
@@ -233,9 +239,6 @@ void tst_QWindow::isExposed()
     window.hide();
 
     QCoreApplication::processEvents();
-#ifdef Q_OS_MAC
-    QEXPECT_FAIL("", "This test fails on Mac OS X, see QTBUG-23059", Abort);
-#endif
     QTRY_VERIFY(window.received(QEvent::Expose) > 1);
     QTRY_VERIFY(!window.isExposed());
 }