Re: [PATCH v4 0/2] MR7539: combase : clsid_from_string_reg() add read of CurVer key value.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/ole32/tests/compobj.c:
>  ok(IsEqualCLSID(&clsid, &CLSID_StdFont), "clsid wasn't equal to CLSID_StdFont\n");
>
>  /* test some failure cases */
> - StringFromGUID2(&CLSID_non_existent, clsidstr, ARRAYSIZE(clsidstr));
> - if (RegCreateKeyExW(HKEY_CLASSES_ROOT, L"MyApp.DocumentTest\CurVer", 0, NULL, 0, KEY_ALL_ACCESS, NULL, &hKey, NULL) == ERROR_SUCCESS) {
> -
>    RegSetValueExW(hKey, NULL, 0, REG_SZ, (const BYTE *)L"MyApp.DocumentTest.1", (wcslen(L"MyApp.DocumentTest.1") + 1) * sizeof(WCHAR));
> -
>    RegCloseKey(hKey);
> - }
> - if (RegCreateKeyExW(HKEY_CLASSES_ROOT, L"MyApp.DocumentTest.1\CLSID", 0, NULL, 0, KEY_ALL_ACCESS, NULL, &hKey, NULL) == ERROR_SUCCESS) {
> -
>    RegSetValueExW(hKey, NULL, 0, REG_SZ, (const BYTE *)clsidstr, (wcslen(clsidstr) + 1) * sizeof(WCHAR));
> -
>    RegCloseKey(hKey);
> - }
> - hr = CLSIDFromProgID(L"MyApp.DocumentTest", &clsid);
> - ok(hr == S_OK, "CLSIDFromProgID failed with error 0x%08lx\n", hr);
> - ok(IsEqualCLSID(&clsid, &CLSID_non_existent), "got wrong clsid %s\n", wine_dbgstr_guid(&clsid));

This needs a test for when both CurVer and CLSID are present. The test should clean up entries it creates.