]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
bpo-44524: Fix cryptic TypeError message when trying to subclass special forms in...
authorYurii Karabas <1998uriyyo@gmail.com>
Wed, 18 Aug 2021 19:08:32 +0000 (22:08 +0300)
committerGitHub <noreply@github.com>
Wed, 18 Aug 2021 19:08:32 +0000 (21:08 +0200)
This was a Python 3.9 regression.

Lib/test/test_typing.py
Lib/typing.py
Misc/NEWS.d/next/Library/2021-08-10-16-57-10.bpo-44524.dk9QX4.rst [new file with mode: 0644]

index 06141f8bbcb614fccad5afcce14d83f3c2733c18..d090c5f8818da16591f4ab785f4b8a713269ebac 100644 (file)
@@ -2424,6 +2424,22 @@ class GenericTests(BaseTestCase):
         self.assertEqual(c.from_b, 'b')
         self.assertEqual(c.from_c, 'c')
 
+    def test_subclass_special_form(self):
+        for obj in (
+            ClassVar[int],
+            Final[int],
+            Union[int, float],
+            Optional[int],
+            Literal[1, 2],
+            Concatenate[int, ParamSpec("P")],
+            TypeGuard[int],
+        ):
+            with self.subTest(msg=obj):
+                with self.assertRaisesRegex(
+                        TypeError, f'^{re.escape(f"Cannot subclass {obj!r}")}$'
+                ):
+                    class Foo(obj):
+                        pass
 
 class ClassVarTests(BaseTestCase):
 
index 51468aeaf5b75578e6a555b926034d85e30321a1..eeeb295b6c2c213a5b70fd0266ac202c876adc14 100644 (file)
@@ -1086,6 +1086,9 @@ class _GenericAlias(_BaseGenericAlias, _root=True):
         return operator.getitem, (origin, args)
 
     def __mro_entries__(self, bases):
+        if isinstance(self.__origin__, _SpecialForm):
+            raise TypeError(f"Cannot subclass {self!r}")
+
         if self._name:  # generic version of an ABC or built-in class
             return super().__mro_entries__(bases)
         if self.__origin__ is Generic:
diff --git a/Misc/NEWS.d/next/Library/2021-08-10-16-57-10.bpo-44524.dk9QX4.rst b/Misc/NEWS.d/next/Library/2021-08-10-16-57-10.bpo-44524.dk9QX4.rst
new file mode 100644 (file)
index 0000000..bc3659f
--- /dev/null
@@ -0,0 +1,2 @@
+Make exception message more useful when subclass from typing special form
+alias. Patch provided by Yurii Karabas.