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`.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8459#note_109918