From f465a8d658f2b5ac0d9d5134812bdf7a7ec4f422 Mon Sep 17 00:00:00 2001 From: Francois Gouget Date: Sat, 30 Nov 2002 19:02:53 +0000 Subject: [PATCH] GetDiskFreeSpace(all NULLs) crashes on NT4 -> removed. GetDiskFreeSpace("." | "..") fails on Win98 and NT4 but succeeds on XP -> removed. Skip the GetDiskFreeSpace tests on floppy drives because NT pops up a MessageBox if no floppy is present. Fix the error code checks to take into account the differences between Win9x, NT4 and XP. Skip the unicode tests if the first call returns ERROR_CALL_NOT_IMPLEMENTED. Whenever a test fails, print sufficient information (typically return code and GetLastError()!) to let the developer figure out what is wrong. --- dlls/kernel/tests/drive.c | 89 +++++++++++++++++++++------------------ 1 file changed, 48 insertions(+), 41 deletions(-) diff --git a/dlls/kernel/tests/drive.c b/dlls/kernel/tests/drive.c index a6cc42b2e7b..45f23d4f314 100644 --- a/dlls/kernel/tests/drive.c +++ b/dlls/kernel/tests/drive.c @@ -38,7 +38,7 @@ void test_GetDriveTypeA(void) if (!(logical_drives & 1)) ok(type == DRIVE_NO_ROOT_DIR, - "GetDriveTypeA should return DRIVE_NO_ROOT_DIR for a not existing drive %c: but not %u", + "GetDriveTypeA should return DRIVE_NO_ROOT_DIR for inexistant drive %c: but not %u", drive[0], type); logical_drives >>= 1; @@ -57,11 +57,16 @@ void test_GetDriveTypeW(void) for (drive[0] = 'A'; drive[0] <= 'Z'; drive[0]++) { type = GetDriveTypeW(drive); + if (type == DRIVE_UNKNOWN && GetLastError()==ERROR_CALL_NOT_IMPLEMENTED) + { + /* Must be Win9x which doesn't support the Unicode functions */ + return; + } ok(type > 0 && type <= 6, "not a valid drive %c: type %u", drive[0], type); if (!(logical_drives & 1)) ok(type == DRIVE_NO_ROOT_DIR, - "GetDriveTypeW should return DRIVE_NO_ROOT_DIR for a not existing drive %c: but not %u", + "GetDriveTypeW should return DRIVE_NO_ROOT_DIR for inexistant drive %c: but not %u", drive[0], type); logical_drives >>= 1; @@ -75,14 +80,13 @@ void test_GetDiskFreeSpaceA(void) char drive[] = "?:\\"; DWORD logical_drives; - ret = GetDiskFreeSpaceA(NULL, NULL, NULL, NULL, NULL); - ok(ret, "GetDiskFreeSpaceA error %ld", GetLastError()); - ret = GetDiskFreeSpaceA(NULL, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); ok(ret, "GetDiskFreeSpaceA error %ld", GetLastError()); ret = GetDiskFreeSpaceA("", §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - ok(!ret && GetLastError() == ERROR_PATH_NOT_FOUND, "GetDiskFreeSpaceA should return ERROR_PATH_NOT_FOUND for \"\""); + ok(!ret && (GetLastError() == ERROR_PATH_NOT_FOUND || GetLastError() == ERROR_INVALID_NAME), + "GetDiskFreeSpaceA(\"\"): ret=%d GetLastError=%ld", + ret, GetLastError()); ret = GetDiskFreeSpaceA("\\", §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); ok(ret, "GetDiskFreeSpaceA error %ld", GetLastError()); @@ -90,25 +94,27 @@ void test_GetDiskFreeSpaceA(void) ret = GetDiskFreeSpaceA("/", §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); ok(ret, "GetDiskFreeSpaceA error %ld", GetLastError()); - ret = GetDiskFreeSpaceA(".", §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - ok(!ret && GetLastError() == ERROR_INVALID_NAME, "GetDiskFreeSpaceA should return ERROR_INVALID_NAME for \".\""); - - ret = GetDiskFreeSpaceA("..", §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - ok(!ret && GetLastError() == ERROR_INVALID_NAME, "GetDiskFreeSpaceA should return ERROR_INVALID_NAME for \"..\""); - logical_drives = GetLogicalDrives(); ok(logical_drives != 0, "GetLogicalDrives error %ld", GetLastError()); for (drive[0] = 'A'; drive[0] <= 'Z'; drive[0]++) { - ret = GetDiskFreeSpaceA(drive, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - if (!(logical_drives & 1)) - ok(!ret && GetLastError() == ERROR_PATH_NOT_FOUND, - "GetDiskFreeSpaceA should return ERROR_PATH_NOT_FOUND for a not existing drive %c", - drive[0]); - else - ok(ret, "GetDiskFreeSpaceA error %ld", GetLastError()); - + /* Skip floppy drives because NT pops up a MessageBox if no + * floppy is present + */ + if (GetDriveTypeA(drive)!=DRIVE_REMOVABLE) + { + ret = GetDiskFreeSpaceA(drive, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); + if (!(logical_drives & 1)) + ok(!ret && (GetLastError() == ERROR_PATH_NOT_FOUND || GetLastError() == ERROR_INVALID_DRIVE), + "GetDiskFreeSpaceA(%s): ret=%d GetLastError=%ld", + drive, ret, GetLastError()); + else + ok(ret || + (!ret && (GetLastError() == ERROR_NOT_READY || GetLastError() == ERROR_INVALID_DRIVE)), + "GetDiskFreeSpaceA(%s): ret=%d GetLastError=%ld", + drive, ret, GetLastError()); + } logical_drives >>= 1; } } @@ -122,43 +128,44 @@ void test_GetDiskFreeSpaceW(void) static const WCHAR empty_pathW[] = { 0 }; static const WCHAR root_pathW[] = { '\\', 0 }; static const WCHAR unix_style_root_pathW[] = { '/', 0 }; - static const WCHAR cur_dirW[] = { '.', 0 }; - static const WCHAR upper_dirW[] = { '.','.', 0 }; - - ret = GetDiskFreeSpaceW(NULL, NULL, NULL, NULL, NULL); - ok(ret, "GetDiskFreeSpaceW error %ld", GetLastError()); ret = GetDiskFreeSpaceW(NULL, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); + if (ret == 0 && GetLastError()==ERROR_CALL_NOT_IMPLEMENTED) + { + /* Must be Win9x which doesn't support the Unicode functions */ + return; + } ok(ret, "GetDiskFreeSpaceW error %ld", GetLastError()); ret = GetDiskFreeSpaceW(empty_pathW, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - ok(!ret && GetLastError() == ERROR_PATH_NOT_FOUND, "GetDiskFreeSpaceW should return ERROR_PATH_NOT_FOUND for \"\""); + ok(!ret && GetLastError() == ERROR_PATH_NOT_FOUND, + "GetDiskFreeSpaceW(\"\"): ret=%d GetLastError=%ld", + ret, GetLastError()); ret = GetDiskFreeSpaceW(root_pathW, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - ok(ret, "GetDiskFreeSpaceW error %ld", GetLastError()); + ok(ret, "GetDiskFreeSpaceW(\"\") error %ld", GetLastError()); ret = GetDiskFreeSpaceW(unix_style_root_pathW, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); ok(ret, "GetDiskFreeSpaceW error %ld", GetLastError()); - ret = GetDiskFreeSpaceW(cur_dirW, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - ok(!ret && GetLastError() == ERROR_INVALID_NAME, "GetDiskFreeSpaceW should return ERROR_INVALID_NAME for \".\""); - - ret = GetDiskFreeSpaceW(upper_dirW, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - ok(!ret && GetLastError() == ERROR_INVALID_NAME, "GetDiskFreeSpaceW should return ERROR_INVALID_NAME for \"..\""); - logical_drives = GetLogicalDrives(); ok(logical_drives != 0, "GetLogicalDrives error %ld", GetLastError()); for (drive[0] = 'A'; drive[0] <= 'Z'; drive[0]++) { - ret = GetDiskFreeSpaceW(drive, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); - if (!(logical_drives & 1)) - ok(!ret && GetLastError() == ERROR_PATH_NOT_FOUND, - "GetDiskFreeSpaceW should return ERROR_PATH_NOT_FOUND for a not existing drive %c", - drive[0]); - else - ok(ret, "GetDiskFreeSpaceW error %ld", GetLastError()); - + /* Skip floppy drives because NT4 pops up a MessageBox if no floppy is present */ + if (GetDriveTypeW(drive)!=DRIVE_REMOVABLE) + { + ret = GetDiskFreeSpaceW(drive, §ors_per_cluster, &bytes_per_sector, &free_clusters, &total_clusters); + if (!(logical_drives & 1)) + ok(!ret && GetLastError() == ERROR_PATH_NOT_FOUND, + "GetDiskFreeSpaceW(%c): ret=%d GetLastError=%ld", + drive[0], ret, GetLastError()); + else + ok(ret || GetLastError() == ERROR_NOT_READY, + "GetDiskFreeSpaceW(%c): ret=%d GetLastError=%ld", + drive[0], ret, GetLastError()); + } logical_drives >>= 1; } }