Test: remove QSKIP in tst_QLocale::emptyCtor
authorCaroline Chao <caroline.chao@digia.com>
Thu, 18 Oct 2012 13:19:11 +0000 (15:19 +0200)
committerThe Qt Project <gerrit-noreply@qt-project.org>
Fri, 19 Oct 2012 07:41:36 +0000 (09:41 +0200)
Instead omit the whole test when appropriate.

Change-Id: Iebd569676cc7b2f8fe4a1d272107e092410d397b
Reviewed-by: Caroline Chao <caroline.chao@digia.com>
tests/auto/corelib/tools/qlocale/tst_qlocale.cpp

index af9f03c..a8aded5 100644 (file)
@@ -95,7 +95,9 @@ private slots:
 #endif
 
     void ctor();
+#if !defined(Q_OS_WINCE) && !defined(QT_NO_PROCESS)
     void emptyCtor();
+#endif
     void unixLocaleName();
     void double_conversion_data();
     void double_conversion();
@@ -374,6 +376,10 @@ void tst_QLocale::ctor()
 #undef TEST_CTOR
 }
 
+#if !defined(Q_OS_WINCE) && !defined(QT_NO_PROCESS)
+// Not when Q_OS_WINCE is defined because the test uses unsupported
+// Windows CE QProcess functionality (std streams, env)
+// Also Qt needs to be compiled without QT_NO_PROCESS
 static inline bool runSysApp(const QString &binary,
                              const QStringList &env,
                              QString *output,
@@ -424,12 +430,6 @@ static inline bool runSysAppTest(const QString &binary,
 
 void tst_QLocale::emptyCtor()
 {
-#if defined(Q_OS_WINCE)
-    QSKIP("Uses unsupported Windows CE QProcess functionality (std streams, env)");
-#endif
-#if defined(QT_NO_PROCESS)
-    QSKIP("Qt was compiled with QT_NO_PROCESS");
-#else
 #define TEST_CTOR(req_lc, exp_str) \
     { \
     /* Test constructor without arguments. Needs separate process */ \
@@ -492,8 +492,8 @@ void tst_QLocale::emptyCtor()
     TEST_CTOR("123456", defaultLoc.toLatin1());
 
 #undef TEST_CTOR
-#endif
 }
+#endif
 
 void tst_QLocale::unixLocaleName()
 {