Alexandre Julliard julliard@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.