[1/3] taskschd/tests: Add some tests for task registration info.

Nikolay Sivov bunglehead at gmail.com
Fri Apr 11 03:03:06 CDT 2014


On 4/11/2014 11:56, Dmitry Timoshkov wrote:
>> 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.
> That's a matter of taste.
Not really. You don't know if it's changed or not after the call if you 
don't know what initial value was.
For series of calls like these:

---

+    hr = IRegistrationInfo_get_Documentation(reginfo, &bstr);
+todo_wine
+    ok(hr == S_OK, "get_Documentation error %#x\n", hr);
+if (hr == S_OK)
+    ok(!bstr, "expected NULL, got %s\n", wine_dbgstr_w(bstr));
+    hr = IRegistrationInfo_get_URI(reginfo, &bstr);
+todo_wine
+    ok(hr == S_OK, "get_URI error %#x\n", hr);
+if (hr == S_OK)
+    ok(!bstr, "expected NULL, got %s\n", wine_dbgstr_w(bstr));
---

there's no way to tell did get_URI set it to NULL or not.





More information about the wine-devel mailing list