On Tue Apr 29 16:24:50 2025 +0000, Bernhard Übelacker wrote:
Thanks for looking at it, so would be early leaving test_FreeConsole like in this [testbot run](https://testbot.winehq.org/JobDetails.pl?Key=158110) the way to go? Or would you like the few tests above the `WaitForSingleObject(unbound_output` still to be run?
actually no for the ctrl handlers test at the bottom of the function
logic of the test is: set ctrl-handler, freeconsole, ctrl handlers should have been removed by FreeConsole (there's a symetrical test for AllocConsole)
Zeb's patch introduced a couple of console creation/deletion in between setting / testing the ctrl handlers, weakening the test (ie fixing only allocconsole would let the test pass)
now we'd bail out for Win7 while it should (could at least) be tested
doesn't look simple to mix both tests, so perhaps splitting them is the way to go