Re: [PATCH v8 0/2] MR8459: urlmon: Add PersistentZoneIdentifier semi-stub

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/urlmon/tests/sec_mgr.c:
> - WCHAR tmp_file_path[MAX_PATH];
> - WCHAR *file_path;
> - HANDLE hfile;
> - DWORD i, zone;
> - LPWSTR file_name;
> - persist_file_test test;
> -
> - hres = IZoneIdentifier_Remove(zone_id_save);
> - ok(hres == S_OK,
> -
>    "Unexpected Remove result: 0x%08lx, expected result: 0x%08lx\n", hres, S_OK);
> -
> - if (!GetTempPathW(sizeof(tmp_dir), tmp_dir))
> - {
> -
>    skip("GetTempPathW failed with: 0x%08lx\n", GetLastError());
> -
>    return;
> - }

`GetTempPathW` is not expected to fail. You may verify that with `ok()`, but there is no need for `skip()`. It's similar for `CreateDirectoryW`.