[prev in list] [next in list] [prev in thread] [next in thread]
List: wine-devel
Subject: [PATCH 2/6] shell32/tests: Don't test functions directly when reporting GetLastError()
From: André_Hentschel <nerv () dawncrow ! de>
Date: 2017-12-30 18:57:54
Message-ID: 05ef2674-7429-0115-ecf4-7020d9d5861e () dawncrow ! de
[Download RAW message or body]
Signed-off-by: André Hentschel <nerv@dawncrow.de>
---
dlls/shell32/tests/shelldispatch.c | 16 +++++++++++-----
1 file changed, 11 insertions(+), 5 deletions(-)
diff --git a/dlls/shell32/tests/shelldispatch.c b/dlls/shell32/tests/shelldispatch.c
index ce3b1a3..64a695d 100644
--- a/dlls/shell32/tests/shelldispatch.c
+++ b/dlls/shell32/tests/shelldispatch.c
@@ -387,6 +387,7 @@ static void test_items(void)
HANDLE file;
BSTR bstr;
char cstr[64];
+ BOOL ret;
int i;
r = CoCreateInstance(&CLSID_Shell, NULL, CLSCTX_INPROC_SERVER, \
&IID_IShellDispatch, (void**)&sd); @@ -397,7 +398,8 @@ static void test_items(void)
GetTempPathW(MAX_PATH, path);
GetCurrentDirectoryW(MAX_PATH, orig_dir);
SetCurrentDirectoryW(path);
- ok(CreateDirectoryW(winetestW, NULL), "CreateDirectory failed: %08x\n", \
GetLastError()); + ret = CreateDirectoryW(winetestW, NULL);
+ ok(ret, "CreateDirectory failed: %08x\n", GetLastError());
GetFullPathNameW(winetestW, MAX_PATH, path, NULL);
V_VT(&var) = VT_BSTR;
V_BSTR(&var) = SysAllocString(path);
@@ -673,7 +675,8 @@ static void test_items(void)
VariantClear(&str_index2);
/* delete the file in the subdirectory */
- ok(DeleteFileA(cstr), "file_defs[%d]: DeleteFile failed: %08x\n", i, \
GetLastError()); + ret = DeleteFileA(cstr);
+ ok(ret, "file_defs[%d]: DeleteFile failed: %08x\n", i, GetLastError());
/* test that getting an item object via a relative path fails */
strcpy(cstr, file_defs[i].name);
@@ -690,11 +693,13 @@ static void test_items(void)
VariantClear(&str_index2);
/* remove the directory */
- ok(RemoveDirectoryA(file_defs[i].name), "file_defs[%d]: RemoveDirectory \
failed: %08x\n", i, GetLastError()); + ret = \
RemoveDirectoryA(file_defs[i].name); + ok(ret, "file_defs[%d]: \
RemoveDirectory failed: %08x\n", i, GetLastError()); }
else
{
- ok(DeleteFileA(file_defs[i].name), "file_defs[%d]: DeleteFile failed: \
%08x\n", i, GetLastError()); + ret = DeleteFileA(file_defs[i].name);
+ ok(ret, "file_defs[%d]: DeleteFile failed: %08x\n", i, GetLastError());
}
/* test that the folder item is still accessible by integer index */
@@ -789,7 +794,8 @@ todo_wine
/* remove the temporary directory and restore the original working directory */
GetTempPathW(MAX_PATH, path);
SetCurrentDirectoryW(path);
- ok(RemoveDirectoryW(winetestW), "RemoveDirectory failed: %08x\n", \
GetLastError()); + ret = RemoveDirectoryW(winetestW);
+ ok(ret, "RemoveDirectory failed: %08x\n", GetLastError());
SetCurrentDirectoryW(orig_dir);
/* test that everything stops working after the directory has been removed */
--
2.7.4
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic