Francois Gouget : testbot: Tweak the disposition of patches when $OnlyCompletePatchSets is set.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: 88b98f0927a572a5f0f9ebef392bcc1347b0d498
URL:    https://gitlab.winehq.org/winehq/tools/-/commit/88b98f0927a572a5f0f9ebef392b...

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Thu Nov  3 03:13:50 2022 +0100

testbot: Tweak the disposition of patches when $OnlyCompletePatchSets is set.

This allows identifying patches which received special treatment due to
this setting.

---

 testbot/lib/WineTestBot/Patches.pm          | 5 ++++-
 testbot/lib/WineTestBot/PendingPatchSets.pm | 4 +++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/testbot/lib/WineTestBot/Patches.pm b/testbot/lib/WineTestBot/Patches.pm
index c0768bea..866a9a60 100644
--- a/testbot/lib/WineTestBot/Patches.pm
+++ b/testbot/lib/WineTestBot/Patches.pm
@@ -230,7 +230,10 @@ sub Submit($$;$)
     }
     else
     {
-      $self->Disposition(($SubjectInfo->{PartNo} ? "Part is not" : "Not") ." a Wine patch");
+      $self->Disposition(($OnlyCompletePatchSets ? "Series is not" :
+                          $SubjectInfo->{PartNo} ? "Part is not" :
+                          "Not")
+                         ." a Wine patch");
     }
     return undef;
   }
diff --git a/testbot/lib/WineTestBot/PendingPatchSets.pm b/testbot/lib/WineTestBot/PendingPatchSets.pm
index 070a71df..06ba8886 100644
--- a/testbot/lib/WineTestBot/PendingPatchSets.pm
+++ b/testbot/lib/WineTestBot/PendingPatchSets.pm
@@ -108,7 +108,9 @@ sub SubmitSubset($$$)
   {
     # Skip over the intermediate patches to instead
     # test the full patchset at once it is complete.
-    return undef;
+    $FinalPatch->Disposition("Only test full series");
+    my ($_ErrKey, $_ErrProperty, $ErrMessage) = $self->Save();
+    return $ErrMessage;
   }
 
   my ($CombinedFile, $CombinedFileName) = OpenNewFile("$DataDir/staging", "_patchset.diff");