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