summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2018-10-28 13:43:03 +0200
committerGitHub <noreply@github.com>2018-10-28 13:43:03 +0200
commit6015cc50bc38b9e920ce4986ee10658eaa14f561 (patch)
treeeeae07dc8d901b5295b10c11c98a96205415bade
parent913876d824d969f8c7431e8a9d4610a9a11a786e (diff)
downloadcpython-git-6015cc50bc38b9e920ce4986ee10658eaa14f561.tar.gz
bpo-32892: Support subclasses of base types in isinstance checks for AST constants. (GH-9934)
Some projects (e.g. Chameleon) create ast.Str containing an instance of the str subclass.
-rw-r--r--Lib/ast.py2
-rw-r--r--Lib/test/test_ast.py4
2 files changed, 5 insertions, 1 deletions
diff --git a/Lib/ast.py b/Lib/ast.py
index de3df1473e..4c8c7795ff 100644
--- a/Lib/ast.py
+++ b/Lib/ast.py
@@ -346,7 +346,7 @@ class _ABC(type):
except AttributeError:
return False
else:
- return type(value) in _const_types[cls]
+ return isinstance(value, _const_types[cls])
return type.__instancecheck__(cls, inst)
def _new(cls, *args, **kwargs):
diff --git a/Lib/test/test_ast.py b/Lib/test/test_ast.py
index 10be02eee0..4bbdc3b82d 100644
--- a/Lib/test/test_ast.py
+++ b/Lib/test/test_ast.py
@@ -399,6 +399,10 @@ class AST_Tests(unittest.TestCase):
self.assertFalse(isinstance(ast.Constant(), ast.NameConstant))
self.assertFalse(isinstance(ast.Constant(), ast.Ellipsis))
+ class S(str): pass
+ self.assertTrue(isinstance(ast.Constant(S('42')), ast.Str))
+ self.assertFalse(isinstance(ast.Constant(S('42')), ast.Num))
+
def test_subclasses(self):
class N(ast.Num):
def __init__(self, *args, **kwargs):