Francois Gouget : testbot: Fix the "Set doesn't affect tests" message.

Alexandre Julliard julliard at winehq.org
Thu May 31 14:31:28 CDT 2018


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

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Thu May 31 10:04:32 2018 +0200

testbot: Fix the "Set doesn't affect tests" message.

There is no need to rerun the tests for part N of a patch set if it
does not impact the tests: the results would be the same as those
already obtained from the combined part 1 .. part N patch.
So adjust the message to better convey this.

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

---

 testbot/lib/WineTestBot/Patches.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testbot/lib/WineTestBot/Patches.pm b/testbot/lib/WineTestBot/Patches.pm
index 71d735b..d5be1b3 100644
--- a/testbot/lib/WineTestBot/Patches.pm
+++ b/testbot/lib/WineTestBot/Patches.pm
@@ -137,7 +137,7 @@ sub Submit($$$)
   my $Impacts = GetPatchImpact("$DataDir/patches/" . $self->Id, undef, $PastImpacts);
   if (!$Impacts->{UnitCount})
   {
-    $self->Disposition(($IsSet ? "Set" : "Patch") .
+    $self->Disposition(($IsSet ? "Part" : "Patch") .
                        " doesn't affect tests");
     return undef;
   }




More information about the wine-cvs mailing list