[Lldb-commits] [lldb] 24fc317 - [lldb] Print the exception traceback when hitting cleanup errors
Raphael Isemann via lldb-commits
lldb-commits at lists.llvm.org
Mon Aug 17 00:54:13 PDT 2020
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(
More information about the lldb-commits
mailing list