Francois Gouget : testbot: Rename the patched test units list to PatchedUnits to avoid confusion.

Alexandre Julliard julliard at winehq.org
Tue Nov 6 14:00:18 CST 2018


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Nov  6 20:06:32 2018 +0100

testbot: Rename the patched test units list to PatchedUnits to avoid confusion.

It is not a list of all the module's test units.

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

---

 testbot/bin/build/WineTest.pl         | 4 ++--
 testbot/lib/WineTestBot/PatchUtils.pm | 6 +++---
 testbot/lib/WineTestBot/Patches.pm    | 4 ++--
 testbot/web/Submit.pl                 | 2 +-
 4 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/testbot/bin/build/WineTest.pl b/testbot/bin/build/WineTest.pl
index 516b18e..6891faa 100755
--- a/testbot/bin/build/WineTest.pl
+++ b/testbot/bin/build/WineTest.pl
@@ -144,9 +144,9 @@ sub TestPatch($$)
       }
       else
       {
-        foreach my $Unit (sort keys %{$TestInfo->{Units}})
+        foreach my $PatchedUnit (sort keys %{$TestInfo->{PatchedUnits}})
         {
-          push @TestList, "$Module:$Unit";
+          push @TestList, "$Module:$PatchedUnit";
         }
       }
     }
diff --git a/testbot/lib/WineTestBot/PatchUtils.pm b/testbot/lib/WineTestBot/PatchUtils.pm
index 0e17a70..6009264 100644
--- a/testbot/lib/WineTestBot/PatchUtils.pm
+++ b/testbot/lib/WineTestBot/PatchUtils.pm
@@ -405,7 +405,7 @@ sub GetPatchImpacts($)
       }
     }
 
-    $TestInfo->{Units} = {};
+    $TestInfo->{PatchedUnits} = {};
     foreach my $File (keys %{$TestInfo->{Files}})
     {
       my $Base = $File;
@@ -418,7 +418,7 @@ sub GetPatchImpacts($)
 
       if ($TestInfo->{All} or $TestInfo->{Files}->{$File})
       {
-        $TestInfo->{Units}->{$Base} = 1;
+        $TestInfo->{PatchedUnits}->{$Base} = 1;
         $Impacts->{ModuleUnitCount}++;
       }
       elsif ($TestInfo->{PatchedModule})
@@ -428,7 +428,7 @@ sub GetPatchImpacts($)
       }
     }
 
-    $TestInfo->{UnitCount} = scalar(keys %{$TestInfo->{Units}});
+    $TestInfo->{UnitCount} = scalar(keys %{$TestInfo->{PatchedUnits}});
     $Impacts->{TestUnitCount} += $TestInfo->{UnitCount};
   }
 
diff --git a/testbot/lib/WineTestBot/Patches.pm b/testbot/lib/WineTestBot/Patches.pm
index e621e88..8a22d3a 100644
--- a/testbot/lib/WineTestBot/Patches.pm
+++ b/testbot/lib/WineTestBot/Patches.pm
@@ -181,7 +181,7 @@ sub Submit($$$)
     foreach my $Module (sort keys %{$Impacts->{Tests}})
     {
       my $TestInfo = $Impacts->{Tests}->{$Module};
-      foreach my $Unit (sort keys %{$TestInfo->{Units}})
+      foreach my $PatchedUnit (sort keys %{$TestInfo->{PatchedUnits}})
       {
         foreach my $Bits ("32", "64")
         {
@@ -236,7 +236,7 @@ sub Submit($$$)
               $Task->VM($VM);
               $Task->Timeout($SingleTimeout);
               $Task->Missions($TaskMissions->{Statement});
-              $Task->CmdLineArg($Unit);
+              $Task->CmdLineArg($PatchedUnit);
             }
           }
         }
diff --git a/testbot/web/Submit.pl b/testbot/web/Submit.pl
index bcb5458..85a12a0 100644
--- a/testbot/web/Submit.pl
+++ b/testbot/web/Submit.pl
@@ -611,7 +611,7 @@ sub DetermineFileType($$)
         {
           $FileType = "patch";
           $ExeBase = $TestInfo->{ExeBase};
-          $TestUnit = (keys %{$TestInfo->{Units}})[0];
+          $TestUnit = (keys %{$TestInfo->{PatchedUnits}})[0];
           last;
         }
       }




More information about the wine-cvs mailing list