[Lldb-commits] [lldb] 24fc317 - [lldb] Print the exception traceback when hitting cleanup errors

[Raphael Isemann via lldb-commits]

Author: Raphael Isemann
Date: 2020-08-17T09:53:52+02:00
New Revision: 24fc3177c1767535aeb494511a13dabf9f6de647

URL: https://github.com/llvm/llvm-project/commit/24fc3177c1767535aeb494511a13dabf9f6de647
DIFF: https://github.com/llvm/llvm-project/commit/24fc3177c1767535aeb494511a13dabf9f6de647.diff

LOG: [lldb] Print the exception traceback when hitting cleanup errors

Right now if the test suite encounters a cleanup error it just prints "CLEANUP
ERROR:" but not any additional information.

This patch just prints the exception that caused the cleanup error. This should
make debugging the failing tests for D83865 easier (and seems in general nice to
have).

Reviewed By: labath

Differential Revision: https://reviews.llvm.org/D83874

Added: 
    

Modified: 
    lldb/packages/Python/lldbsuite/test/test_result.py

Removed: 
    


################################################################################
diff  --git a/lldb/packages/Python/lldbsuite/test/test_result.py b/lldb/packages/Python/lldbsuite/test/test_result.py
index 54affca86057..cab446d95e6d 100644
--- a/lldb/packages/Python/lldbsuite/test/test_result.py
+++ b/lldb/packages/Python/lldbsuite/test/test_result.py
@@ -9,6 +9,7 @@
 
 # System modules
 import os
+import traceback
 
 # Third-party modules
 import unittest2
@@ -228,8 +229,8 @@ def addCleanupError(self, test, err):
         if method:
             method()
         self.stream.write(
-            "CLEANUP ERROR: LLDB (%s) :: %s\n" %
-            (self._config_string(test), str(test)))
+            "CLEANUP ERROR: LLDB (%s) :: %s\n%s\n" %
+            (self._config_string(test), str(test), traceback.format_exc()))
 
     def addFailure(self, test, err):
         if (self.checkExclusion(