[PATCH] wine/test.h: Increment correct counter for tests in todo blocks

Nikolay Sivov bunglehead at gmail.com
Thu Sep 21 11:28:22 CDT 2017


On 21.09.2017 19:11, Fabian Maurer wrote:
> Tests succeeding in todo blocks are counted as failures,
> and failures as success. This patch fixes that.
> 
> Signed-off-by: Fabian Maurer <dark.shadow4 at web.de>
> ---
>  include/wine/test.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/include/wine/test.h b/include/wine/test.h
> index 4ec66d5391..f0ee617d5f 100644
> --- a/include/wine/test.h
> +++ b/include/wine/test.h
> @@ -311,7 +311,7 @@ int winetest_vok( int condition, const char *msg, __winetest_va_list args )
>              printf( "%s:%d: Test succeeded inside todo block: ",
>                      data->current_file, data->current_line );
>              vprintf(msg, args);
> -            InterlockedIncrement(&todo_failures);
> +            InterlockedIncrement(&todo_successes);
>              return 0;
>          }
>          else
> @@ -322,7 +322,7 @@ int winetest_vok( int condition, const char *msg, __winetest_va_list args )
>                          data->current_file, data->current_line );
>                  vprintf(msg, args);
>              }
> -            InterlockedIncrement(&todo_successes);
> +            InterlockedIncrement(&todo_failures);
>              return 1;
>          }
>      }
> 

This is correct. If test passes but is marked as todo it is a failure.



More information about the wine-devel mailing list