Test: remove QSKIP from tst_QProcessEnvironment::putenv
authorCaroline Chao <caroline.chao@digia.com>
Tue, 16 Oct 2012 10:59:06 +0000 (12:59 +0200)
committerThe Qt Project <gerrit-noreply@qt-project.org>
Tue, 16 Oct 2012 13:29:15 +0000 (15:29 +0200)
Instead omit the whole test when Q_OS_WINCE is defined.

Change-Id: I19e35b837709c92e0202c6a96d113367bc6c92c2
Reviewed-by: J-P Nurmi <jpnurmi@digia.com>
tests/auto/corelib/io/qprocessenvironment/tst_qprocessenvironment.cpp

index eb3f25d..3826bfc 100644 (file)
@@ -57,7 +57,9 @@ private slots:
 
     void caseSensitivity();
     void systemEnvironment();
+#ifndef Q_OS_WINCE
     void putenv();
+#endif
 };
 
 void tst_QProcessEnvironment::operator_eq()
@@ -276,11 +278,10 @@ void tst_QProcessEnvironment::systemEnvironment()
 #endif
 }
 
+#ifndef Q_OS_WINCE
+//Windows CE has no environment
 void tst_QProcessEnvironment::putenv()
 {
-#ifdef Q_OS_WINCE
-    QSKIP("Windows CE has no environment");
-#else
     static const char envname[] = "WE_RE_SETTING_THIS_ENVIRONMENT_VARIABLE";
     static bool testRan = false;
 
@@ -311,8 +312,8 @@ void tst_QProcessEnvironment::putenv()
     QVERIFY(eAfter.contains(lower));
     QCOMPARE(eAfter.value(lower), QString("Hello, World"));
 # endif
-#endif
 }
+#endif
 
 QTEST_MAIN(tst_QProcessEnvironment)