uxtheme/tests: Don't test for themed app

André Hentschel nerv at dawncrow.de
Mon Apr 1 11:38:34 CDT 2013


win8 seems to always report IsAppThemed()==TRUE
---
 dlls/uxtheme/tests/system.c |   14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/dlls/uxtheme/tests/system.c b/dlls/uxtheme/tests/system.c
index 8dd413b..9acbaf6 100644
--- a/dlls/uxtheme/tests/system.c
+++ b/dlls/uxtheme/tests/system.c
@@ -84,18 +84,14 @@ static void test_IsThemed(void)
     SetLastError(0xdeadbeef);
     bThemeActive = pIsThemeActive();
     trace("Theming is %s\n", (bThemeActive) ? "active" : "inactive");
+    ok( GetLastError() == ERROR_SUCCESS ||
+        broken(GetLastError() == ERROR_FILE_NOT_FOUND /* Win2k8 */),
+        "Expected ERROR_SUCCESS, got 0x%08x\n",
+        GetLastError());
 
-    /* This test is not themed */
     SetLastError(0xdeadbeef);
     bAppThemed = pIsAppThemed();
-
-    if (bThemeActive)
-        todo_wine
-            ok( bAppThemed == FALSE, "Expected FALSE as this test executable is not (yet) themed.\n");
-    else
-        /* Although Wine currently returns FALSE, the logic behind it is wrong. It is not a todo_wine though in the testing sense */
-        ok( bAppThemed == FALSE, "Expected FALSE as this test executable is not (yet) themed.\n");
-
+    trace("Test executable is %s\n", (bAppThemed) ? "themed" : "not themed");
     ok( GetLastError() == ERROR_SUCCESS,
         "Expected ERROR_SUCCESS, got 0x%08x\n",
         GetLastError());
-- 
1.7.10.4



More information about the wine-patches mailing list