Test: remove QSKIP in tst_QCoreApplication::argc

Instead omit the whole test when Q_OS_WIN is defined since
the test is invalid on this platform.

Change-Id: Idb77df96b0c2a223cddbfffb4e24c6d1f5d33dfb
Reviewed-by: J-P Nurmi <jpnurmi@digia.com>

diff --git a/tests/auto/corelib/kernel/qcoreapplication/tst_qcoreapplication.cpp b/tests/auto/corelib/kernel/qcoreapplication/tst_qcoreapplication.cpp
index f662a88..dce7219 100644 (file)
--- a/tests/auto/corelib/kernel/qcoreapplication/tst_qcoreapplication.cpp
+++ b/tests/auto/corelib/kernel/qcoreapplication/tst_qcoreapplication.cpp
@@ -53,7 +53,9 @@ private slots:
     void sendEventsOnProcessEvents(); // this must be the first test
     void getSetCheck();
     void qAppName();
+#ifndef Q_OS_WIN
     void argc();
+#endif
     void postEvent();
     void removePostedEvents();
 #ifndef QT_NO_THREAD
@@ -125,11 +127,11 @@ void tst_QCoreApplication::qAppName()
     QCOMPARE(QCoreApplication::applicationName(), QString::fromLatin1("tst_qcoreapplication"));
 }
 
+// "QCoreApplication::arguments() always parses arguments from actual command line on Windows
+// making this test invalid."
+#ifndef Q_OS_WIN
 void tst_QCoreApplication::argc()
 {
-#ifdef Q_OS_WIN
-    QSKIP("QCoreApplication::arguments() always parses arguments from actual command line in Windows, making this test invalid.");
-#endif
     {
         int argc = 1;
         char *argv[] = { const_cast<char*>("tst_qcoreapplication") };
@@ -166,6 +168,7 @@ void tst_QCoreApplication::argc()
         QCOMPARE(app.arguments().count(), 1);
     }
 }
+#endif
 
 class EventGenerator : public QObject
 {