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

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/urlmon/tests/zone_id.c:
> -
>    hres = IPersistFile_Load(persist_file_load, file_path, test.load.mode);
> -
>    ok(hres == test.load.hres,
> -
>        "%s unexpected load result: 0x%08lx, expected result: 0x%08lx\n",
> -
>        file_tests, hres, test.load.hres);
> -
> -
>    hres = IPersistFile_Load(persist_file_save, file_path, test.load.mode);
> -
> -
>    hres = IPersistFile_IsDirty(persist_file_load);
> -
>    ok(hres == test.is_dirty_after_load,
> -
>        "%s unexpected change object state after load: 0x%08lx, expected state: 0x%08lx\n",
> -
>        file_tests, hres, test.is_dirty_after_load);
> -
> -
>    hres = IZoneIdentifier_GetId(zone_id_load, &zone);
> -
> -
>    /* GetId checks after expected Load falure shouldn't be marked as todo_wine */
> -
>    if (test.get.hres == __HRESULT_FROM_WIN32(ERROR_NOT_FOUND))

You could use `todo_wine_if(test.get.hres != __HRESULT_FROM_WIN32(ERROR_NOT_FOUND))`.