[1/3] taskschd/tests: Add some tests for task registration info.
Dmitry Timoshkov
dmitry at baikal.ru
Fri Apr 11 02:56:34 CDT 2014
Nikolay Sivov <bunglehead at gmail.com> wrote:
> 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.
I had an impression that when __WINESRC__ is defined lstrcmpW() gets replaced
by an inline implementation which doesn't have a SEH wrapper and crashes, but
it looks like that's not the case. Thanks.
> 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.
--
Dmitry.
More information about the wine-devel
mailing list