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.


Reply via email to