[Tools] testbot/WineRunTask: TestLauncher's fake test result lines are wrong.

Francois Gouget fgouget at codeweavers.com
Tue Feb 21 11:02:31 CST 2017


TestLauncher uses the dll name instead of the test unit name on the fake
'tests skipped' and test summary lines it generates when a dll is
missing. Accept the latter to avoid a 'missing test summary line' error.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/bin/WineRunTask.pl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/testbot/bin/WineRunTask.pl b/testbot/bin/WineRunTask.pl
index 21ad8e2a..0b686f6d 100755
--- a/testbot/bin/WineRunTask.pl
+++ b/testbot/bin/WineRunTask.pl
@@ -509,7 +509,8 @@ if ($TA->GetFile($RptFileName, $FullLogFileName))
       {
         my ($Pid, $Unit, $Todo, $Failures) = ($1, $2, $3, $4);
 
-        if ($Unit eq $CurrentUnit)
+        # TestLauncher uses the wrong name in its test summary line
+        if ($Unit eq $CurrentUnit or $Unit eq $CurrentDll)
         {
           $CurrentPids{$Pid || 0} = 1;
           $SummaryFailures += $Failures;
-- 
2.11.0



More information about the wine-patches mailing list