On 4/11/2014 11:26, Dmitry Timoshkov wrote:
- hr = IRegistrationInfo_get_Description(reginfo, &bstr);
+todo_wine
- ok(hr == S_OK, "get_Description error %#x\n", hr);
+if (hr == S_OK) +{
- ok(bstr && !lstrcmpW(bstr, Task1), "expected Task1, got %s\n", wine_dbgstr_w(bstr));
It's redundant to check for 'bstr' being not NULL, lstrcmpW test will fail in this case too. For other tests when it returns NULL bstr it would be cleaner to set pointer to some non-zero value to test that each method actually resets it to NULL.