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( _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
