Francois Gouget : testbot: Better check for the "No build or test VM" disposition.

[Alexandre Julliard]

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

Author: Francois Gouget <fgouget(a)codeweavers.com>
Date:   Tue Sep 25 16:58:15 2018 +0200

testbot: Better check for the "No build or test VM" disposition.

Use a direct test instead of an indirect one.
Also the $Impacts->{UnitCount} field has been renamed.

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)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 469fedc..96e4a0d 100644
--- a/testbot/lib/WineTestBot/Patches.pm
+++ b/testbot/lib/WineTestBot/Patches.pm
@@ -266,7 +266,7 @@ sub Submit($$$)
   if ($NewJob->Steps->IsEmpty())
   {
     # This may be a Wine patch but there is no suitable VM to test it!
-    if ($Impacts->{UnitCount})
+    if ($BuildVMs->IsEmpty() and $WineVMs->IsEmpty())
     {
       $self->Disposition("No build or test VM!");
     }