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