From e89ef0ad2a299770a88ece8f7a316f7d3eb65c9f Mon Sep 17 00:00:00 2001 From: Serhiy Storchaka Date: Sat, 24 Jul 2021 10:40:08 +0300 Subject: bpo-44353: Expand NewType tests for complex __qualname__. (#27311) Make NewType pickleable by name. --- Lib/test/test_typing.py | 58 ++++++++++++++++++++++++++++++++++--------------- Lib/typing.py | 7 +++++- 2 files changed, 47 insertions(+), 18 deletions(-) diff --git a/Lib/test/test_typing.py b/Lib/test/test_typing.py index bfa5755..13ac395 100644 --- a/Lib/test/test_typing.py +++ b/Lib/test/test_typing.py @@ -3696,34 +3696,37 @@ class NewTypeTests: for f in self.module._cleanups: f() - def setUp(self): - sys.modules['typing'] = self.module + @classmethod + def setUpClass(cls): + sys.modules['typing'] = cls.module + global UserId + UserId = cls.module.NewType('UserId', int) + cls.UserName = cls.module.NewType(cls.__qualname__ + '.UserName', str) + + @classmethod + def tearDownClass(cls): + global UserId + del UserId + del cls.UserName + sys.modules['typing'] = typing def tearDown(self): self.cleanup() - sys.modules['typing'] = typing def test_basic(self): - UserId = self.module.NewType('UserId', int) - UserName = self.module.NewType('UserName', str) self.assertIsInstance(UserId(5), int) - self.assertIsInstance(UserName('Joe'), str) + self.assertIsInstance(self.UserName('Joe'), str) self.assertEqual(UserId(5) + 1, 6) def test_errors(self): - UserId = self.module.NewType('UserId', int) - UserName = self.module.NewType('UserName', str) with self.assertRaises(TypeError): issubclass(UserId, int) with self.assertRaises(TypeError): - class D(UserName): + class D(UserId): pass def test_or(self): - UserId = self.module.NewType('UserId', int) - UserName = self.module.NewType('UserName', str) - - for cls in (int, UserName): + for cls in (int, self.UserName): with self.subTest(cls=cls): self.assertEqual(UserId | cls, self.module.Union[UserId, cls]) self.assertEqual(cls | UserId, self.module.Union[cls, UserId]) @@ -3732,16 +3735,37 @@ class NewTypeTests: self.assertEqual(self.module.get_args(cls | UserId), (cls, UserId)) def test_special_attrs(self): - UserId = self.module.NewType('UserId', int) - self.assertEqual(UserId.__name__, 'UserId') self.assertEqual(UserId.__qualname__, 'UserId') self.assertEqual(UserId.__module__, __name__) + self.assertEqual(UserId.__supertype__, int) - def test_repr(self): - UserId = self.module.NewType('UserId', int) + UserName = self.UserName + self.assertEqual(UserName.__name__, 'UserName') + self.assertEqual(UserName.__qualname__, + self.__class__.__qualname__ + '.UserName') + self.assertEqual(UserName.__module__, __name__) + self.assertEqual(UserName.__supertype__, str) + def test_repr(self): self.assertEqual(repr(UserId), f'{__name__}.UserId') + self.assertEqual(repr(self.UserName), + f'{__name__}.{self.__class__.__qualname__}.UserName') + + def test_pickle(self): + UserAge = self.module.NewType('UserAge', float) + for proto in range(pickle.HIGHEST_PROTOCOL + 1): + pickled = pickle.dumps(UserId, proto) + loaded = pickle.loads(pickled) + self.assertIs(loaded, UserId) + + pickled = pickle.dumps(self.UserName, proto) + loaded = pickle.loads(pickled) + self.assertIs(loaded, self.UserName) + + with self.assertRaises(pickle.PicklingError): + pickle.dumps(UserAge, proto) + class NewTypePythonTests(NewTypeTests, BaseTestCase): module = py_typing diff --git a/Lib/typing.py b/Lib/typing.py index e618513..198837c 100644 --- a/Lib/typing.py +++ b/Lib/typing.py @@ -2385,14 +2385,19 @@ class NewType: __call__ = _idfunc def __init__(self, name, tp): - self.__name__ = name self.__qualname__ = name + if '.' in name: + name = name.rpartition('.')[-1] + self.__name__ = name self.__module__ = _callee(default='typing') self.__supertype__ = tp def __repr__(self): return f'{self.__module__}.{self.__qualname__}' + def __reduce__(self): + return self.__qualname__ + def __or__(self, other): return Union[self, other] -- cgit v0.12