]> granicus.if.org Git - python/commitdiff
Added more tests for issue #27122.
authorSerhiy Storchaka <storchaka@gmail.com>
Mon, 20 Jun 2016 02:29:54 +0000 (05:29 +0300)
committerSerhiy Storchaka <storchaka@gmail.com>
Mon, 20 Jun 2016 02:29:54 +0000 (05:29 +0300)
Lib/test/test_contextlib.py

index a246c43187f7c0390b654938e4536e9890aab737..516403ef655b9ed0c61f23b9963793e70ef000ae 100644 (file)
@@ -763,8 +763,9 @@ class TestExitStack(unittest.TestCase):
         self.assertIs(stack._exit_callbacks[-1], cm)
 
     def test_dont_reraise_RuntimeError(self):
-        """https://bugs.python.org/issue27122"""
+        # https://bugs.python.org/issue27122
         class UniqueException(Exception): pass
+        class UniqueRuntimeError(RuntimeError): pass
 
         @contextmanager
         def second():
@@ -780,15 +781,20 @@ class TestExitStack(unittest.TestCase):
             except Exception as exc:
                 raise exc
 
-        # The RuntimeError should be caught by second()'s exception
+        # The UniqueRuntimeError should be caught by second()'s exception
         # handler which chain raised a new UniqueException.
         with self.assertRaises(UniqueException) as err_ctx:
             with ExitStack() as es_ctx:
                 es_ctx.enter_context(second())
                 es_ctx.enter_context(first())
-                raise RuntimeError("please no infinite loop.")
-
-        self.assertEqual(err_ctx.exception.args[0], "new exception")
+                raise UniqueRuntimeError("please no infinite loop.")
+
+        exc = err_ctx.exception
+        self.assertIsInstance(exc, UniqueException)
+        self.assertIsInstance(exc.__context__, UniqueRuntimeError)
+        self.assertIsNone(exc.__context__.__context__)
+        self.assertIsNone(exc.__context__.__cause__)
+        self.assertIs(exc.__cause__, exc.__context__)
 
 
 class TestRedirectStream: