[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