Alexandre Julliard : kernel32/tests: Print some more details on directory test failures.

Alexandre Julliard julliard at winehq.org
Tue Feb 3 09:13:36 CST 2009


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

Author: Alexandre Julliard <julliard at winehq.org>
Date:   Tue Feb  3 15:01:41 2009 +0100

kernel32/tests: Print some more details on directory test failures.

---

 dlls/kernel32/tests/directory.c |   25 ++++++++++++++++---------
 1 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/dlls/kernel32/tests/directory.c b/dlls/kernel32/tests/directory.c
index 47e7701..2c943d1 100644
--- a/dlls/kernel32/tests/directory.c
+++ b/dlls/kernel32/tests/directory.c
@@ -339,27 +339,32 @@ static void test_CreateDirectoryW(void)
     ret = CreateDirectoryW(NULL, NULL);
     if (!ret && GetLastError()==ERROR_CALL_NOT_IMPLEMENTED)
         return;
-    ok(ret == FALSE && GetLastError() == ERROR_PATH_NOT_FOUND, "should not create NULL path\n");
+    ok(ret == FALSE && GetLastError() == ERROR_PATH_NOT_FOUND,
+       "should not create NULL path ret %u err %u\n", ret, GetLastError());
 
     ret = CreateDirectoryW(empty_strW, NULL);
-    ok(ret == FALSE && GetLastError() == ERROR_PATH_NOT_FOUND, "should not create empty path\n");
+    ok(ret == FALSE && GetLastError() == ERROR_PATH_NOT_FOUND,
+       "should not create empty path ret %u err %u\n", ret, GetLastError());
 
     ret = GetSystemDirectoryW(tmpdir, MAX_PATH);
     ok(ret < MAX_PATH, "System directory should fit into MAX_PATH\n");
 
     ret = SetCurrentDirectoryW(tmpdir);
-    ok(ret == TRUE, "could not chdir to the System directory\n");
+    ok(ret == TRUE, "could not chdir to the System directory ret %u err %u\n", ret, GetLastError());
 
     ret = CreateDirectoryW(dotW, NULL);
-    ok(ret == FALSE && GetLastError() == ERROR_ALREADY_EXISTS, "should not create existing path\n");
+    ok(ret == FALSE && GetLastError() == ERROR_ALREADY_EXISTS,
+       "should not create existing path ret %u err %u\n", ret, GetLastError());
 
     ret = CreateDirectoryW(dotdotW, NULL);
-    ok(ret == FALSE && GetLastError() == ERROR_ALREADY_EXISTS, "should not create existing path\n");
+    ok(ret == FALSE && GetLastError() == ERROR_ALREADY_EXISTS,
+       "should not create existing path ret %u err %u\n", ret, GetLastError());
 
     GetTempPathW(MAX_PATH, tmpdir);
     tmpdir[3] = 0; /* truncate the path */
     ret = CreateDirectoryW(tmpdir, NULL);
-    ok(ret == FALSE && GetLastError() == ERROR_ACCESS_DENIED, "should deny access to the drive root\n");
+    ok(ret == FALSE && GetLastError() == ERROR_ACCESS_DENIED,
+       "should deny access to the drive root ret %u err %u\n", ret, GetLastError());
 
     GetTempPathW(MAX_PATH, tmpdir);
     lstrcatW(tmpdir, tmp_dir_name);
@@ -367,7 +372,8 @@ static void test_CreateDirectoryW(void)
     ok(ret == TRUE, "CreateDirectoryW should always succeed\n");
 
     ret = CreateDirectoryW(tmpdir, NULL);
-    ok(ret == FALSE && GetLastError() == ERROR_ALREADY_EXISTS, "should not create existing path\n");
+    ok(ret == FALSE && GetLastError() == ERROR_ALREADY_EXISTS,
+       "should not create existing path ret %u err %u\n", ret, GetLastError());
 
     ret = RemoveDirectoryW(tmpdir);
     ok(ret == TRUE, "RemoveDirectoryW should always succeed\n");
@@ -391,8 +397,9 @@ static void test_CreateDirectoryW(void)
     lstrcatW(tmpdir, slashW);
     lstrcatW(tmpdir, tmp_dir_name);
     ret = CreateDirectoryW(tmpdir, NULL);
-    ok(ret == FALSE && GetLastError() == ERROR_PATH_NOT_FOUND, 
-      "CreateDirectoryW with multiple nonexistent directories in path should fail\n");
+    ok(ret == FALSE && GetLastError() == ERROR_PATH_NOT_FOUND,
+      "CreateDirectoryW with multiple nonexistent directories in path should fail ret %u err %u\n",
+       ret, GetLastError());
     ret = RemoveDirectoryW(tmpdir);
 }
 




More information about the wine-cvs mailing list