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