[PATCH 1/3] shell32/tests: Add more tests for FolderItems.
Alex Henrie
alexhenrie24 at gmail.com
Wed Jul 20 23:45:28 CDT 2016
Signed-off-by: Alex Henrie <alexhenrie24 at gmail.com>
---
dlls/shell32/tests/Makefile.in | 2 +-
dlls/shell32/tests/shelldispatch.c | 227 +++++++++++++++++++++++++++++++++++--
2 files changed, 219 insertions(+), 10 deletions(-)
diff --git a/dlls/shell32/tests/Makefile.in b/dlls/shell32/tests/Makefile.in
index cfe0c50..ef95b52 100644
--- a/dlls/shell32/tests/Makefile.in
+++ b/dlls/shell32/tests/Makefile.in
@@ -1,5 +1,5 @@
TESTDLL = shell32.dll
-IMPORTS = shell32 ole32 oleaut32 user32 advapi32
+IMPORTS = shell32 ole32 oleaut32 user32 advapi32 shlwapi
C_SRCS = \
appbar.c \
diff --git a/dlls/shell32/tests/shelldispatch.c b/dlls/shell32/tests/shelldispatch.c
index 70b6c36..315f3a5 100644
--- a/dlls/shell32/tests/shelldispatch.c
+++ b/dlls/shell32/tests/shelldispatch.c
@@ -310,19 +310,46 @@ static void test_namespace(void)
static void test_items(void)
{
- WCHAR wstr[MAX_PATH], orig_dir[MAX_PATH];
+ static const struct
+ {
+ char name[32];
+ enum
+ {
+ DIRECTORY,
+ SHORTCUT,
+ EMPTY,
+ RANDOM,
+ }
+ type;
+ }
+ file_defs[] =
+ {
+ { "00-Myfolder", DIRECTORY },
+ { "01-empty.lnk", EMPTY },
+ { "02-shortcut.lnk", SHORTCUT },
+ { "03-empty.bin", EMPTY },
+ { "04-random.bin", RANDOM },
+ };
+ WCHAR wstr[MAX_PATH], wstr2[MAX_PATH], orig_dir[MAX_PATH];
HRESULT r;
IShellDispatch *sd = NULL;
Folder *folder = NULL;
FolderItems *items = NULL;
FolderItems2 *items2 = NULL;
FolderItems3 *items3 = NULL;
- FolderItem *item = (FolderItem*)0xdeadbeef;
+ FolderItem *item = (FolderItem*)0xdeadbeef, *item2;
IDispatch *disp = NULL;
IUnknown *unk = NULL;
FolderItemVerbs *verbs = (FolderItemVerbs*)0xdeadbeef;
VARIANT var;
LONG lcount = -1;
+ HANDLE file;
+ DWORD dwcount;
+ IShellLinkA *sl = NULL;
+ IPersistFile *shortcut = NULL;
+ BSTR bstr, bstr2;
+ char buf[512];
+ int i, j;
r = CoCreateInstance(&CLSID_Shell, NULL, CLSCTX_INPROC_SERVER, &IID_IShellDispatch, (void**)&sd);
ok(SUCCEEDED(r), "CoCreateInstance failed: %08x\n", r);
@@ -345,13 +372,6 @@ static void test_items(void)
r = Folder_Items(folder, &items);
ok(r == S_OK, "Folder::Items failed: %08x\n", r);
ok(!!items, "items is null\n");
- r = FolderItems_QueryInterface(items, &IID_FolderItems2, (void**)&items2);
- ok(r == S_OK || broken(E_NOINTERFACE) /* xp and later */, "FolderItems::QueryInterface failed: %08x\n", r);
- ok(!!items2 || broken(!items2) /* xp and later */, "items2 is null\n");
- r = FolderItems_QueryInterface(items, &IID_FolderItems3, (void**)&items3);
- ok(r == S_OK, "FolderItems::QueryInterface failed: %08x\n", r);
- ok(!!items3, "items3 is null\n");
- Folder_Release(folder);
if (0) /* crashes on all versions of Windows */
r = FolderItems_get_Count(items, NULL);
@@ -373,6 +393,187 @@ todo_wine
ok(r == S_FALSE, "expected S_FALSE, got %08x\n", r);
ok(!item, "item is not null\n");
+ for (i = 0; i < sizeof(file_defs)/sizeof(file_defs[0]); i++)
+ {
+ switch (file_defs[i].type)
+ {
+ case DIRECTORY:
+ r = CreateDirectoryA(file_defs[i].name, NULL);
+ ok(r, "file_defs[%d]: CreateDirectory failed: %08x\n", i, GetLastError());
+ break;
+
+ case SHORTCUT:
+ sl = NULL;
+ r = CoCreateInstance(&CLSID_ShellLink, NULL, CLSCTX_INPROC_SERVER, &IID_IShellLinkA, (void**)&sl);
+ ok(SUCCEEDED(r), "file_defs[%d]: CoCreateInstance failed: %08x\n", i, r);
+ ok(!!sl, "sl is null\n");
+
+ shortcut = NULL;
+ r = IShellLinkA_QueryInterface(sl, &IID_IPersistFile, (void**)&shortcut);
+ ok(r == S_OK, "file_defs[%d]: IShellLink::QueryInterface failed: %08x\n", i, r);
+ ok(!!shortcut, "file_defs[%d]: shortcut is null\n", i);
+
+ MultiByteToWideChar(CP_ACP, 0, file_defs[i].name, -1, wstr, sizeof(wstr)/sizeof(WCHAR));
+ r = IPersistFile_Save(shortcut, wstr, FALSE);
+ ok(r == S_OK, "file_defs[%d]: IPersistFile::Save failed: %08x\n", i, r);
+
+ IPersistFile_Release(shortcut);
+ IShellLinkA_Release(sl);
+ break;
+
+ case EMPTY:
+ file = NULL;
+ file = CreateFileA(file_defs[i].name, 0, 0, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL);
+ ok(!!file, "file_defs[%d]: CreateFile failed: %08x\n", i, GetLastError());
+ CloseHandle(file);
+ break;
+
+ case RANDOM:
+ file = NULL;
+ file = CreateFileA(file_defs[i].name, GENERIC_WRITE, 0, NULL,
+ CREATE_ALWAYS, FILE_ATTRIBUTE_NORMAL, NULL);
+ ok(!!file, "file_defs[%d]: CreateFile failed: %08x\n", i, GetLastError());
+ for (j = 0; j < sizeof(buf); j++)
+ buf[j] = rand() % 256;
+ r = WriteFile(file, buf, sizeof(buf), &dwcount, NULL);
+ ok(r, "file_defs[%d]: WriteFile failed: %08x\n", i, GetLastError());
+ CloseHandle(file);
+ break;
+ }
+ }
+
+ lcount = -1;
+ r = FolderItems_get_Count(items, &lcount);
+todo_wine
+ ok(r == S_OK, "FolderItems::get_Count failed: %08x\n", r);
+todo_wine
+ ok(!lcount, "expected 0 files, got %d\n", lcount);
+ FolderItems_Release(items);
+
+ items = NULL;
+ r = Folder_Items(folder, &items);
+ ok(r == S_OK, "Folder::Items failed: %08x\n", r);
+ ok(!!items, "items is null\n");
+ r = FolderItems_QueryInterface(items, &IID_FolderItems2, (void**)&items2);
+ ok(r == S_OK || broken(E_NOINTERFACE) /* xp and later */, "FolderItems::QueryInterface failed: %08x\n", r);
+ ok(!!items2 || broken(!items2) /* xp and later */, "items2 is null\n");
+ r = FolderItems_QueryInterface(items, &IID_FolderItems3, (void**)&items3);
+ ok(r == S_OK, "FolderItems::QueryInterface failed: %08x\n", r);
+ ok(!!items3, "items3 is null\n");
+ Folder_Release(folder);
+
+ lcount = -1;
+ r = FolderItems_get_Count(items, &lcount);
+todo_wine
+ ok(r == S_OK, "FolderItems::get_Count failed: %08x\n", r);
+todo_wine
+ ok(lcount == sizeof(file_defs)/sizeof(file_defs[0]),
+ "expected %d files, got %d\n", sizeof(file_defs)/sizeof(file_defs[0]), lcount);
+
+ for (i = 0; i < sizeof(file_defs)/sizeof(file_defs[0]); i++)
+ {
+ V_I4(&var) = i;
+
+ item = NULL;
+ r = FolderItems_Item(items, var, &item);
+todo_wine
+ ok(r == S_OK, "file_defs[%d]: FolderItems::Item failed: %08x\n", i, r);
+todo_wine
+ ok(!!item, "file_defs[%d]: item is null\n", i);
+ if (!item) continue;
+
+ r = FolderItems_Item(items, var, &item2);
+ ok(r == S_OK, "file_defs[%d]: FolderItems::Item failed: %08x\n", i, r);
+ ok(item2 != item, "file_defs[%d]: item and item2 are the same\n", i);
+ FolderItem_Release(item2);
+
+ disp = NULL;
+ r = FolderItem_get_Application(item, &disp);
+ ok(r == S_OK, "file_defs[%d]: FolderItem::get_Application failed: %08x\n", i, r);
+ ok(!!disp, "file_defs[%d]: disp is null\n", i);
+ if (disp) IDispatch_Release(disp);
+
+ disp = NULL;
+ r = FolderItem_get_Parent(item, &disp);
+ ok(r == S_OK, "file_defs[%d]: FolderItem::get_Parent failed: %08x\n", i, r);
+ ok(!!disp, "file_defs[%d]: disp is null\n", i);
+ if (disp) IDispatch_Release(disp);
+
+ bstr = NULL;
+ r = FolderItem_get_Path(item, &bstr);
+ ok(r == S_OK, "file_defs[%d]: FolderItem::get_Path failed: %08x\n", i, r);
+ ok(!!bstr, "file_defs[%d]: bstr is null\n", i);
+
+ bstr2 = NULL;
+ r = FolderItem_get_Path(item, &bstr2);
+ ok(r == S_OK, "file_defs[%d]: FolderItem::get_Path failed: %08x\n", i, r);
+ ok(bstr2 != bstr, "file_defs[%d]: bstr and bstr2 are the same\n", i);
+ SysFreeString(bstr2);
+
+ if (bstr)
+ {
+ MultiByteToWideChar(CP_ACP, 0, file_defs[i].name, -1, wstr, MAX_PATH);
+ GetFullPathNameW(wstr, MAX_PATH, wstr2, NULL);
+ GetLongPathNameW(wstr2, wstr, MAX_PATH);
+ ok(!lstrcmpW(bstr, wstr),
+ "file_defs[%d]: expected %s, got %s\n", i, wine_dbgstr_w(wstr), wine_dbgstr_w(bstr));
+ SysFreeString(bstr);
+ }
+
+ disp = (IDispatch*)0xdeadbeef;
+ r = FolderItem_get_GetLink(item, &disp);
+ if (!lstrcmpA(PathFindExtensionA(file_defs[i].name), ".lnk"))
+ {
+ ok(r == S_OK, "file_defs[%d]: FolderItem::get_GetLink failed: %08x\n", i, r);
+ ok(!!disp, "file_defs[%d]: disp is null\n", i);
+ }
+ else
+ {
+ ok(r == E_NOTIMPL || broken(r == HRESULT_FROM_WIN32(ERROR_FILE_NOT_FOUND)) /* xp */,
+ "file_defs[%d]: expected E_NOTIMPL, got %08x\n", i, r);
+ ok(!disp, "file_defs[%d]: disp is not null\n", i);
+ }
+ if (disp) IDispatch_Release(disp);
+
+ disp = (IDispatch*)0xdeadbeef;
+ r = FolderItem_get_GetFolder(item, &disp);
+ if (file_defs[i].type == DIRECTORY)
+ {
+ ok(r == S_OK, "file_defs[%d]: FolderItem::get_GetFolder failed: %08x\n", i, r);
+ ok(!!disp, "file_defs[%d]: disp is null\n", i);
+ }
+ else
+ {
+ ok(r == E_FAIL || broken(r == HRESULT_FROM_WIN32(ERROR_FILE_NOT_FOUND)) /* xp */,
+ "file_defs[%d]: expected E_FAIL, got %08x\n", i, r);
+ ok(!disp, "file_defs[%d]: disp is not null\n", i);
+ }
+ if (disp) IDispatch_Release(disp);
+
+ lcount = -1;
+ r = FolderItem_get_Size(item, &lcount);
+ ok(r == S_OK, "file_defs[%d]: FolderItem::get_Size failed: %08x\n", i, r);
+ if (file_defs[i].type == DIRECTORY || file_defs[i].type == EMPTY)
+ ok(!lcount, "file_defs[%d]: got %d\n", i, lcount);
+ else
+ ok(lcount > 0, "file_defs[%d]: got %d\n", i, lcount);
+
+ verbs = NULL;
+ r = FolderItem_Verbs(item, &verbs);
+ ok(r == S_OK, "file_defs[%d]: FolderItem::Verbs failed: %08x\n", i, r);
+ ok(!!verbs, "file_defs[%d]: verbs is null\n", i);
+ if (verbs) FolderItemVerbs_Release(verbs);
+
+ FolderItem_Release(item);
+ }
+
+ V_I4(&var) = sizeof(file_defs)/sizeof(file_defs[0]);
+ item = NULL;
+ r = FolderItems_Item(items, var, &item);
+todo_wine
+ ok(r == S_FALSE, "expected S_FALSE, got %08x\n", r);
+ ok(!item, "item is not null\n");
+
if (0) /* crashes on xp */
{
r = FolderItems_get_Application(items, NULL);
@@ -422,12 +623,20 @@ todo_wine
ok(r == E_INVALIDARG, "expected E_INVALIDARG, got %08x\n", r);
}
+ verbs = (FolderItemVerbs*)0xdeadbeef;
r = FolderItems3_get_Verbs(items3, &verbs);
todo_wine
ok(r == S_FALSE, "expected S_FALSE, got %08x\n", r);
ok(!verbs, "verbs is not null\n");
}
+ for (i = 0; i < sizeof(file_defs)/sizeof(file_defs[0]); i++)
+ {
+ if (file_defs[i].type == DIRECTORY)
+ RemoveDirectoryA(file_defs[i].name);
+ else
+ DeleteFileA(file_defs[i].name);
+ }
GetTempPathW(MAX_PATH, wstr);
SetCurrentDirectoryW(wstr);
RemoveDirectoryW(winetestW);
--
2.9.0
More information about the wine-patches
mailing list