summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSamuel Colvin <samcolvin@gmail.com>2019-10-13 12:45:36 +0100
committerSerhiy Storchaka <storchaka@gmail.com>2019-10-13 14:45:36 +0300
commit793cb85437299a3da3d74fe65480d720af330cbb (patch)
treeaf1f297dba6baeb54bdb6838e840f79604d39b6d
parent140a7d1f3579e778656a6b6bfad72489e9870a4d (diff)
downloadcpython-git-793cb85437299a3da3d74fe65480d720af330cbb.tar.gz
bpo-38431: Fix __repr__ method of InitVar to work with typing objects. (GH-16702)
-rw-r--r--Lib/dataclasses.py7
-rw-r--r--Lib/test/test_dataclasses.py2
-rw-r--r--Misc/NEWS.d/next/Library/2019-10-10-16-53-00.bpo-38431.d5wzNp.rst1
3 files changed, 9 insertions, 1 deletions
diff --git a/Lib/dataclasses.py b/Lib/dataclasses.py
index 9135b07c9f..91c1f6f80f 100644
--- a/Lib/dataclasses.py
+++ b/Lib/dataclasses.py
@@ -206,7 +206,12 @@ class InitVar:
self.type = type
def __repr__(self):
- return f'dataclasses.InitVar[{self.type.__name__}]'
+ if isinstance(self.type, type):
+ type_name = self.type.__name__
+ else:
+ # typing objects, e.g. List[int]
+ type_name = repr(self.type)
+ return f'dataclasses.InitVar[{type_name}]'
def __class_getitem__(cls, type):
return InitVar(type)
diff --git a/Lib/test/test_dataclasses.py b/Lib/test/test_dataclasses.py
index 037bf4c221..238335e7d9 100644
--- a/Lib/test/test_dataclasses.py
+++ b/Lib/test/test_dataclasses.py
@@ -1102,6 +1102,8 @@ class TestCase(unittest.TestCase):
# Make sure the repr is correct.
self.assertEqual(repr(InitVar[int]), 'dataclasses.InitVar[int]')
+ self.assertEqual(repr(InitVar[List[int]]),
+ 'dataclasses.InitVar[typing.List[int]]')
def test_init_var_inheritance(self):
# Note that this deliberately tests that a dataclass need not
diff --git a/Misc/NEWS.d/next/Library/2019-10-10-16-53-00.bpo-38431.d5wzNp.rst b/Misc/NEWS.d/next/Library/2019-10-10-16-53-00.bpo-38431.d5wzNp.rst
new file mode 100644
index 0000000000..c2f860d804
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2019-10-10-16-53-00.bpo-38431.d5wzNp.rst
@@ -0,0 +1 @@
+Fix ``__repr__`` method for :class:`dataclasses.InitVar` to support typing objects, patch by Samuel Colvin.