[PATCH resend] user32/tests: Fix broadcast test failures on Windows 10

Alex Henrie alexhenrie24 at gmail.com
Fri Apr 27 13:22:44 CDT 2018


On Fri, Apr 27, 2018 at 11:38 AM Alexandre Julliard <julliard at winehq.org>
wrote:

> Alex Henrie <alexhenrie24 at gmail.com> writes:

> > What is the right way to fix these test failures? Do you think the
> > pre-win10 behavior should be marked as broken?

> I'm not sure this privilege dropping is meaningful at all. I feel it
> would be more useful to test more behaviors of that function under
> normal privileges, with hopefully consistent results across platforms.

You're right. I just tested it and to my surprise, these tests behave the
same on all testbot platforms with or without the call to
AdjustTokenPrivileges: https://testbot.winehq.org/JobDetails.pl?Key=38082

That makes me think that these are essentially just more invalid parameter
tests like the ones my patch already removes. In that light, would it be
acceptable to simply delete the test_noprivileges function and encourage
anyone working on message broadcasting to write more meaningful tests in
the future?

-Alex



More information about the wine-devel mailing list