Paul Vriens : advapi32/tests: Fix a test failure on Vista+.

Alexandre Julliard julliard at winehq.org
Thu Nov 5 10:50:01 CST 2009


Module: wine
Branch: master
Commit: 6a5f67167871ce0f72f7471d4aaf993b8599abee
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=6a5f67167871ce0f72f7471d4aaf993b8599abee

Author: Paul Vriens <Paul.Vriens.Wine at gmail.com>
Date:   Thu Nov  5 11:33:43 2009 +0100

advapi32/tests: Fix a test failure on Vista+.

---

 dlls/advapi32/tests/eventlog.c |   20 ++++++++++++++++++--
 1 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/dlls/advapi32/tests/eventlog.c b/dlls/advapi32/tests/eventlog.c
index 3c932ea..702fc43 100644
--- a/dlls/advapi32/tests/eventlog.c
+++ b/dlls/advapi32/tests/eventlog.c
@@ -44,6 +44,9 @@ static void create_backup(const char *filename)
     handle = OpenEventLogA(NULL, "Application");
     BackupEventLogA(handle, filename);
     CloseEventLog(handle);
+
+    todo_wine
+    ok(GetFileAttributesA(filename) != INVALID_FILE_ATTRIBUTES, "Expected a backup file\n");
 }
 
 static void test_open_close(void)
@@ -561,6 +564,7 @@ static void test_clear(void)
     HANDLE handle;
     BOOL ret;
     const char backup[] = "backup.evt";
+    const char backup2[] = "backup2.evt";
 
     SetLastError(0xdeadbeef);
     ret = ClearEventLogA(NULL, NULL);
@@ -583,16 +587,28 @@ static void test_clear(void)
     SetLastError(0xdeadbeef);
     ret = ClearEventLogA(handle, backup);
     ok(!ret, "Expected failure\n");
-    ok(GetLastError() == ERROR_INVALID_HANDLE, "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError());
+    /* The eventlog service runs under an account that doesn't have the necessary
+     * permissions on the users home directory on a default Vista+ system.
+     */
+    ok(GetLastError() == ERROR_INVALID_HANDLE ||
+       GetLastError() == ERROR_ACCESS_DENIED, /* Vista+ */
+       "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError());
 
     /* Show that ClearEventLog only works for real eventlogs. */
     SetLastError(0xdeadbeef);
+    ret = ClearEventLogA(handle, backup2);
+    ok(!ret, "Expected failure\n");
+    ok(GetLastError() == ERROR_INVALID_HANDLE, "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError());
+    ok(GetFileAttributesA(backup2) == INVALID_FILE_ATTRIBUTES, "Expected no backup file\n");
+
+    SetLastError(0xdeadbeef);
     ret = ClearEventLogA(handle, NULL);
     ok(!ret, "Expected failure\n");
     ok(GetLastError() == ERROR_INVALID_HANDLE, "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError());
 
     CloseEventLog(handle);
-    DeleteFileA(backup);
+    todo_wine
+    ok(DeleteFileA(backup), "Could not delete the backup file\n");
 }
 
 START_TEST(eventlog)




More information about the wine-cvs mailing list