Jacek Caban (@jacek) commented about dlls/urlmon/tests/zone_id.c:
+ const save_file_test save = { NULL, FALSE, E_INVALIDARG }; + IUnknown *unk; + IZoneIdentifier *zone_id; + IPersistFile *persist_file; + HRESULT hres; + DWORD zone; + CLSID clsid; + LPWSTR file_name; + + hres = CoCreateInstance(&CLSID_PersistentZoneIdentifier, NULL, + CLSCTX_INPROC_SERVER, &IID_IUnknown, (void**)&unk); + if (hres != S_OK) + { + skip("%s failed to obtain IUnknown iface: 0x%08lx\n", uninit_tests, hres); + return; + } Are there any Windows versions that fail to create the zone identifier? If not, this should use `ok()` instead. If there are, it should be `win_skip()` to avoid unintentionally accepting such skips on Wine. The same applies to all other `skip()` calls in the patch.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/8459#note_109716