Francois Gouget : winetest/dissect: Fix handling of skipped test unit.

Alexandre Julliard julliard at winehq.org
Tue Apr 20 16:26:03 CDT 2021


Module: tools
Branch: master
Commit: 42f389faf327f4634d9b1f147d9462062b5eb64c
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=42f389faf327f4634d9b1f147d9462062b5eb64c

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Apr 20 16:50:09 2021 +0200

winetest/dissect: Fix handling of skipped test unit.

close_test_unit() adds an entry in summary.txt for every test unit.
So the report parser must set $summary instead of manually adding a
redundant summary.txt entry.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 winetest/dissect | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/winetest/dissect b/winetest/dissect
index c58e47f..ce3329e 100755
--- a/winetest/dissect
+++ b/winetest/dissect
@@ -627,7 +627,7 @@ while ($line = <IN>) {
         if ($l_type eq "skipped")
         {
             add_test_line("skipped", "Skipped by user request");
-            print SUM "- $dll $unit skipped - - - $source\n";
+            $summary = "skipped - - -";
             mydie "too many test units skipped by user request (>$maxuserskips at $dll:$unit)" if ++$skipped_units > $maxuserskips;
             $rc = 0;
         }




More information about the wine-cvs mailing list