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@codeweavers.com --- This caused skipped tests to appear as '0 failures' in the index pages instead of 'skipped'. That's because close_test_unit() was adding an extra 'not-skipped' entry in summary.txt. --- winetest/dissect | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/winetest/dissect b/winetest/dissect index c58e47f57..ce3329ea6 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; }