[uxtheme] Added test for IsAppThemed and grouped it with IsThemeActive

Paul Vriens Paul.Vriens at xs4all.nl
Fri Jun 16 09:02:47 CDT 2006


On Fri, 2006-06-16 at 15:25 +0200, Alexandre Julliard wrote:
> Paul Vriens <Paul.Vriens at xs4all.nl> writes:
> 
> > +    SetLastError(0xdeadbeef);
> > +    bAppThemed = pIsAppThemed();
> > +    todo_wine
> > +    {
> > +        ok( bAppThemed == FALSE, "Expected FALSE as this test executable is not (yet) themed.\n");
> > +        ok( GetLastError() == S_OK,
> > +            "Expected S_OK, got 0x%08lx\n",
> > +            GetLastError());
> > +    }
> 
> This won't work right if theming is disabled, since IsAppThemed
> returns FALSE in that case. Also note that GetLastError doesn't return
> an HRESULT, so ERROR_SUCCESS is a better name than S_OK.
> 
Well, tests (XP-enabled and W2K3-disabled) both return FALSE as our app
is not themed (I guess).

I'll do some rediffs and tests and start again. First patch will change
S_OK into ERROR_SUCCESS.

Cheers,

Paul.




More information about the wine-devel mailing list