summaryrefslogtreecommitdiffstats
path: root/Lib
diff options
context:
space:
mode:
authorNathaniel J. Smith <njs@pobox.com>2018-01-07 13:30:18 (GMT)
committerNick Coghlan <ncoghlan@gmail.com>2018-01-07 13:30:18 (GMT)
commite46a8af450210ee5c7f0459ad6beddbc626ae60f (patch)
tree7d8abe4ccbec09008bec52d603446599feb84fda /Lib
parentd327ae6ba157fb5118df28a86975c5ca523f05e2 (diff)
downloadcpython-e46a8af450210ee5c7f0459ad6beddbc626ae60f.zip
cpython-e46a8af450210ee5c7f0459ad6beddbc626ae60f.tar.gz
cpython-e46a8af450210ee5c7f0459ad6beddbc626ae60f.tar.bz2
bpo-30579: Allow TracebackType creation and tb_next mutation from Python (GH-4793)
Third party projects may wish to hide their own internal machinery in order to present more comprehensible tracebacks to end users (e.g. Jinja2 and Trio both do this). Previously such projects have had to rely on ctypes to do so: https://github.com/pallets/jinja/blob/fe3dadacdf4cf411d0a5b6bbd4d5234697a28af2/jinja2/debug.py#L345 https://github.com/python-trio/trio/blob/1e86b1aee8c0c759f6f239ae53a05d0d3963c629/trio/_core/_multierror.py#L296 This provides a Python level API for creating and modifying real Traceback objects, allowing tracebacks to be edited at runtime. Patch by Nathaniel Smith.
Diffstat (limited to 'Lib')
-rw-r--r--Lib/test/test_raise.py66
1 files changed, 66 insertions, 0 deletions
diff --git a/Lib/test/test_raise.py b/Lib/test/test_raise.py
index 103f608..c1ef154 100644
--- a/Lib/test/test_raise.py
+++ b/Lib/test/test_raise.py
@@ -228,6 +228,72 @@ class TestTraceback(unittest.TestCase):
self.fail("No exception raised")
+class TestTracebackType(unittest.TestCase):
+
+ def raiser(self):
+ raise ValueError
+
+ def test_attrs(self):
+ try:
+ self.raiser()
+ except Exception as exc:
+ tb = exc.__traceback__
+
+ self.assertIsInstance(tb.tb_next, types.TracebackType)
+ self.assertIs(tb.tb_frame, sys._getframe())
+ self.assertIsInstance(tb.tb_lasti, int)
+ self.assertIsInstance(tb.tb_lineno, int)
+
+ self.assertIs(tb.tb_next.tb_next, None)
+
+ # Invalid assignments
+ with self.assertRaises(TypeError):
+ del tb.tb_next
+
+ with self.assertRaises(TypeError):
+ tb.tb_next = "asdf"
+
+ # Loops
+ with self.assertRaises(ValueError):
+ tb.tb_next = tb
+
+ with self.assertRaises(ValueError):
+ tb.tb_next.tb_next = tb
+
+ # Valid assignments
+ tb.tb_next = None
+ self.assertIs(tb.tb_next, None)
+
+ new_tb = get_tb()
+ tb.tb_next = new_tb
+ self.assertIs(tb.tb_next, new_tb)
+
+ def test_constructor(self):
+ other_tb = get_tb()
+ frame = sys._getframe()
+
+ tb = types.TracebackType(other_tb, frame, 1, 2)
+ self.assertEqual(tb.tb_next, other_tb)
+ self.assertEqual(tb.tb_frame, frame)
+ self.assertEqual(tb.tb_lasti, 1)
+ self.assertEqual(tb.tb_lineno, 2)
+
+ tb = types.TracebackType(None, frame, 1, 2)
+ self.assertEqual(tb.tb_next, None)
+
+ with self.assertRaises(TypeError):
+ types.TracebackType("no", frame, 1, 2)
+
+ with self.assertRaises(TypeError):
+ types.TracebackType(other_tb, "no", 1, 2)
+
+ with self.assertRaises(TypeError):
+ types.TracebackType(other_tb, frame, "no", 2)
+
+ with self.assertRaises(TypeError):
+ types.TracebackType(other_tb, frame, 1, "nuh-uh")
+
+
class TestContext(unittest.TestCase):
def test_instance_context_instance_raise(self):
context = IndexError()