#14179: add tests for re.compile. Patch by Florian Mladitsch.

2 files changed, 11 insertions, 0 deletions

diff --git a/Lib/test/test_re.py b/Lib/test/test_re.py
index b19834cae6..6f6d014cb1 100644
--- a/Lib/test/test_re.py
+++ b/Lib/test/test_re.py
@@ -783,6 +783,16 @@ class ReTests(unittest.TestCase):
         self.assertRaises(TypeError, re.finditer, "a", {})
         self.assertRaises(OverflowError, _sre.compile, "abc", 0, [long_overflow])
 
+    def test_compile(self):
+        # Test return value when given string and pattern as parameter
+        pattern = re.compile('random pattern')
+        self.assertIsInstance(pattern, re._pattern_type)
+        same_pattern = re.compile(pattern)
+        self.assertIsInstance(same_pattern, re._pattern_type)
+        self.assertIs(same_pattern, pattern)
+        # Test behaviour when not given a string or pattern as parameter
+        self.assertRaises(TypeError, re.compile, 0)
+
 def run_re_tests():
     from test.re_tests import tests, SUCCEED, FAIL, SYNTAX_ERROR
     if verbose:
diff --git a/Misc/ACKS b/Misc/ACKS
index b51de26d22..dc6d749190 100644
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -577,6 +577,7 @@ Roman Milner
 Andrii V. Mishkovskyi
 Dustin J. Mitchell
 Dom Mitchell
+Florian Mladitsch
 Doug Moen
 The Dragon De Monsyne
 Skip Montanaro