user32/tests: Add some tests for dialog owner disabled state. (v3)

Alexandre Julliard julliard at winehq.org
Mon Apr 11 23:34:03 CDT 2016


Dmitry Timoshkov <dmitry at baikal.ru> writes:

> Alexandre Julliard <julliard at winehq.org> wrote:
>
>> > Regarding any possible additional tests or moving some logic around:
>> > as the patch subject says these tests don't pretend to be perfect or
>> > exhaustive set of tests, they simply test some things I was interested
>> > in, and as always with every test there are things that could be added
>> > or changed.
>> 
>> Of course, but could you please fix the few things that Jacek mentioned,
>> and update the todos that succeed now?  Thanks.
>
> I can resend with just todos removed if that's ok, since changing the things
> that Jacek pointed out (adding the tests after EndDialog, moving the tests
> into another place or removing a test that  makes sure that window state has
> not really changed) has no influence on the things being actually tested.

Maybe it would still be testing the same thing, but moving the tests out
of WM_INITDIALOG would make things clearer. Avoiding changing the window
creation of existing tests would be a good thing too.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list