Francois Gouget : winetest/build-patterns: Fix detecting transitions in non-run-time failures.

[Alexandre Julliard]

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

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Mon May  3 14:45:11 2021 +0200

winetest/build-patterns: Fix detecting transitions in non-run-time failures.

Record the first failure type so changes can be detected later on.

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

---

 winetest/build-patterns | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/winetest/build-patterns b/winetest/build-patterns
index 460c17c..a84e2a7 100755
--- a/winetest/build-patterns
+++ b/winetest/build-patterns
@@ -480,12 +480,15 @@ foreach my $testname (keys %tests)
             }
             next if ($status eq "skipped"); # same as if WineTest was not run
 
-            # It is not an error if the dll is missing (or other similar
-            # error) _and_ it has always been so.
             my $failtype = fail_type($status);
-            next if ($testreport->{failtype} eq "" and $failtype ne "random");
-
-            if ($testreport->{failtype} ne $failtype)
+            if ($testreport->{failtype} eq "" and $failtype ne "random")
+            {
+                # It is not an error if the dll is missing (or other similar
+                # error) _and_ it has always been so. So just record failtype
+                # for now.
+                $testreport->{failtype} = $failtype;
+            }
+            elsif ($testreport->{failtype} ne $failtype)
             {
                 # Either there was no failure before; or the failure type
                 # changed in a way that cannot be explained by randomness; e.g.