From: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Date: Tue, 2 Jan 2018 08:20:12 +0000 (-0800)
Subject: bpo-32478: Add tests for 'break' and 'return' inside 'finally' clause. (GH-5078)... 
X-Git-Tag: v3.6.5rc1~172
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=aea95c2fbdec05dddd8b833a74eeb7ec8dc69706;p=python

bpo-32478: Add tests for 'break' and 'return' inside 'finally' clause. (GH-5078) (#5083)

(cherry picked from commit 7cc42c356b0dc5ad9eaa9392789e84bd4aa1c7de)
---

diff --git a/Lib/test/test_grammar.py b/Lib/test/test_grammar.py
index 65e26bfd38..62f3251f05 100644
--- a/Lib/test/test_grammar.py
+++ b/Lib/test/test_grammar.py
@@ -805,6 +805,80 @@ class GrammarTests(unittest.TestCase):
         x = g2()
         check_syntax_error(self, "class foo:return 1")
 
+    def test_break_in_finally(self):
+        count = 0
+        while count < 2:
+            count += 1
+            try:
+                pass
+            finally:
+                break
+        self.assertEqual(count, 1)
+
+        count = 0
+        while count < 2:
+            count += 1
+            try:
+                continue
+            finally:
+                break
+        self.assertEqual(count, 1)
+
+        count = 0
+        while count < 2:
+            count += 1
+            try:
+                1/0
+            finally:
+                break
+        self.assertEqual(count, 1)
+
+        for count in [0, 1]:
+            self.assertEqual(count, 0)
+            try:
+                pass
+            finally:
+                break
+        self.assertEqual(count, 0)
+
+        for count in [0, 1]:
+            self.assertEqual(count, 0)
+            try:
+                continue
+            finally:
+                break
+        self.assertEqual(count, 0)
+
+        for count in [0, 1]:
+            self.assertEqual(count, 0)
+            try:
+                1/0
+            finally:
+                break
+        self.assertEqual(count, 0)
+
+    def test_return_in_finally(self):
+        def g1():
+            try:
+                pass
+            finally:
+                return 1
+        self.assertEqual(g1(), 1)
+
+        def g2():
+            try:
+                return 2
+            finally:
+                return 3
+        self.assertEqual(g2(), 3)
+
+        def g3():
+            try:
+                1/0
+            finally:
+                return 4
+        self.assertEqual(g3(), 4)
+
     def test_yield(self):
         # Allowed as standalone statement
         def g(): yield 1