[tools] testbot/LogUtils: Add support for winedbg's unhandled exception lines.

Francois Gouget fgouget at codeweavers.com
Tue Feb 9 19:16:01 CST 2021


They replace the test framework's "unhandled exception" lines we get on
Windows but specify neither the test unit nor the process pid. So just
assign them to the current test unit.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=50538
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
For the following wtbsuite tests:
WTBS Simple test crash (serialui:confdlg).
WTBS Crash in a child process (kernel32:toolhelp).

 testbot/lib/WineTestBot/LogUtils.pm | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index 69299da66..9164b47ff 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -631,6 +631,13 @@ sub ParseWineTestReport($$$)
       _CheckUnit($LogInfo, $Cur, $Unit, "unhandled exception");
       $Cur->{LineFailures}++;
     }
+    elsif ($Line =~ /Unhandled exception: .* in .* code /)
+    {
+      # This also replaces a test summary line. The pid is unknown so use 0.
+      $Cur->{Pids}->{0} = 1;
+      $Cur->{SummaryFailures}++;
+      $Cur->{LineFailures}++;
+    }
     elsif (($Cur->{Unit} ne "" and
             $Line =~ /($Cur->{UnitsRE}):\d+: unhandled exception [0-9a-fA-F]{8} in child process /) or
            $Line =~ /^([_.a-z0-9]+):\d+: unhandled exception [0-9a-fA-F]{8} in child process /)
-- 
2.20.1




More information about the wine-devel mailing list