Francois Gouget : testbot/TestWTBS: Adjust the TestUnits check to the wine test/module missions.

Alexandre Julliard julliard at winehq.org
Mon Mar 1 15:28:35 CST 2021


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Mon Mar  1 16:27:21 2021 +0100

testbot/TestWTBS: Adjust the TestUnits check to the wine test/module missions.

Don't complain that the test=test or test=module case is missing test
units if there was no mission of that type.

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

---

 testbot/tests/TestWTBS | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/testbot/tests/TestWTBS b/testbot/tests/TestWTBS
index 915eab6..e9fa98e 100755
--- a/testbot/tests/TestWTBS
+++ b/testbot/tests/TestWTBS
@@ -894,6 +894,7 @@ sub CheckTask($$$$)
         $MissionType .= ":$Test";
       }
       $ReportTypes{$ReportName} = $MissionType;
+      $TestUnits->{$MissionType} ||= {};
     }
   }
   my $TestType = ($TaskType eq "wine" and !$IsTestTask) ?
@@ -1203,6 +1204,9 @@ sub CheckJobTree($;$)
       my @MissionTypes = $Type eq "wine" ? ("wine:test", "wine:module") : ($Type);
       foreach my $MissionType (@MissionTypes)
       {
+        # Skip if there was no task running this type of mission
+        next if (!$TestUnits->{$MissionType});
+
         my $MissionInfo = $TestInfo->{$MissionType};
         next if (!CheckValue($MissionInfo->{TestUnits}));
 




More information about the wine-cvs mailing list