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:
> -
>    hres, S_OK);
> -
> - hres = IZoneIdentifier_GetId(zone_id, &zone);
> - ok(hres == HRESULT_FROM_WIN32(ERROR_NOT_FOUND),
> -
>    "Unexpected GetId result after Remove: 0x%08lx, expected result: 0x%08lx\n",
> -
>    hres, HRESULT_FROM_WIN32(ERROR_NOT_FOUND));
> - ok(zone == URLZONE_UNTRUSTED,
> -
>    "Unexpected GetId zone after Remove: 0x%08lx, expected zone: 0x%08x\n",
> -
>    zone, URLZONE_UNTRUSTED);
> -
>
> +}
> -
>
> +typedef struct _get_cur_file_test {
> - LPCWSTR name;
> - HRESULT hres;
>
> +} get_cur_file_test;

It's similar to `IsDirty`, if it always returns `E_NOTIMPL` there is no need to specify that for each test, you could just change the test to always check for `E_NOTIMPL`.