[1/2] tests: Add winetest_status() which indicates what to do with a test result.

Alexandre Julliard julliard at winehq.org
Fri Feb 5 09:26:36 CST 2016


Francois Gouget <fgouget at codeweavers.com> writes:

> winetest_status() takes into account not just the condition but also
> the todo level, debug level, whether to report successful tests; and
> returns the test status and whether to trace its result.
> This allows individual tests to do their own logging taking into account all these flags.
>
> Signed-off-by: Francois Gouget <fgouget at codeweavers.com>

Honestly, this thing is getting out of hand. Not only is this adding a
lot of complexity for a trivial difference in output, but it has
actually made the traces less readable IMO. Can we please get rid of the
fancy macros and go back to printing the relevant information in the
ok() calls, like all other tests do?  Pretty please?

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list