[tools]winetest: fix child-result coloring and counting

[Paul Vriens]

On 01/04/2010 08:50 PM, André Hentschel wrote:
> You can see wrong coloring e.g. here http://test.winehq.org/data/f74e312bf81032c46bd2ce080a5f6a33c7ac3ee3/wine_ah-910-64-nv/rpcrt4:server.html
> Also the counting is wrong, that unit test is shown in the summary with 2 errors, but only has 1
> This patch fixes the issue. The totalize of those variables is already made in the "offline" software
> ---
>   winetest/dissect |    6 +++---
>   1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/winetest/dissect b/winetest/dissect
> index 16cc118..ce90bda 100755
> --- a/winetest/dissect
> +++ b/winetest/dissect
> @@ -252,9 +252,9 @@ while (<IN>) {
>       } elsif (/^(.*$unit.*: (\d+) tests executed \((\d+) marked as todo, (\d+) failures?\), (\d+) skipped\.)\r?$/) {
>           $lines++;
>           $total += $2;
> -        $todo += $3;
> -        $failed += $4;
> -        $skipped += $5;
> +        $todo = $3;
> +        $failed = $4;
> +        $skipped = $5;
>           chomp;
>           s/\r+$//;
>           my $class = "test result";

Hi André,

That log also shows "server: 1 failures in child process". I'm not a 
100% sure but from the looks of it this is that one/1 failure mentioned 
in that last line:

server: 120 tests executed (1 marked as todo, 1 failure), 0 skipped.

If I'm correct it does mean that we have 2 failures and 1 todo.

(cc-ing AJ as he wrote that winetest_wait_child_process() thingy).

-- 
Cheers,

Paul.