Barry Warsaw | 906569d | 2002-04-23 22:48:42 +0000 | [diff] [blame] | 1 | # Tests some corner cases with isinstance() and issubclass(). While these |
| 2 | # tests use new style classes and properties, they actually do whitebox |
| 3 | # testing of error conditions uncovered when using extension types. |
| 4 | |
| 5 | import unittest |
Brett Cannon | 4f65331 | 2004-03-20 22:52:14 +0000 | [diff] [blame] | 6 | import sys |
Maggie Moss | 1b4552c | 2020-09-09 13:23:24 -0700 | [diff] [blame] | 7 | import typing |
Barry Warsaw | 906569d | 2002-04-23 22:48:42 +0000 | [diff] [blame] | 8 | |
| 9 | |
| 10 | |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 11 | class TestIsInstanceExceptions(unittest.TestCase): |
Barry Warsaw | 906569d | 2002-04-23 22:48:42 +0000 | [diff] [blame] | 12 | # Test to make sure that an AttributeError when accessing the instance's |
| 13 | # class's bases is masked. This was actually a bug in Python 2.2 and |
| 14 | # 2.2.1 where the exception wasn't caught but it also wasn't being cleared |
| 15 | # (leading to an "undetected error" in the debug build). Set up is, |
| 16 | # isinstance(inst, cls) where: |
| 17 | # |
Ezio Melotti | 3f5db39 | 2013-01-27 06:20:14 +0200 | [diff] [blame] | 18 | # - cls isn't a type, or a tuple |
Barry Warsaw | 906569d | 2002-04-23 22:48:42 +0000 | [diff] [blame] | 19 | # - cls has a __bases__ attribute |
| 20 | # - inst has a __class__ attribute |
| 21 | # - inst.__class__ as no __bases__ attribute |
| 22 | # |
| 23 | # Sounds complicated, I know, but this mimics a situation where an |
| 24 | # extension type raises an AttributeError when its __bases__ attribute is |
| 25 | # gotten. In that case, isinstance() should return False. |
| 26 | def test_class_has_no_bases(self): |
| 27 | class I(object): |
| 28 | def getclass(self): |
| 29 | # This must return an object that has no __bases__ attribute |
| 30 | return None |
| 31 | __class__ = property(getclass) |
| 32 | |
| 33 | class C(object): |
| 34 | def getbases(self): |
| 35 | return () |
| 36 | __bases__ = property(getbases) |
| 37 | |
| 38 | self.assertEqual(False, isinstance(I(), C())) |
| 39 | |
| 40 | # Like above except that inst.__class__.__bases__ raises an exception |
| 41 | # other than AttributeError |
| 42 | def test_bases_raises_other_than_attribute_error(self): |
| 43 | class E(object): |
| 44 | def getbases(self): |
| 45 | raise RuntimeError |
| 46 | __bases__ = property(getbases) |
| 47 | |
| 48 | class I(object): |
| 49 | def getclass(self): |
| 50 | return E() |
| 51 | __class__ = property(getclass) |
| 52 | |
| 53 | class C(object): |
| 54 | def getbases(self): |
| 55 | return () |
| 56 | __bases__ = property(getbases) |
| 57 | |
| 58 | self.assertRaises(RuntimeError, isinstance, I(), C()) |
| 59 | |
| 60 | # Here's a situation where getattr(cls, '__bases__') raises an exception. |
| 61 | # If that exception is not AttributeError, it should not get masked |
| 62 | def test_dont_mask_non_attribute_error(self): |
| 63 | class I: pass |
| 64 | |
| 65 | class C(object): |
| 66 | def getbases(self): |
| 67 | raise RuntimeError |
| 68 | __bases__ = property(getbases) |
| 69 | |
| 70 | self.assertRaises(RuntimeError, isinstance, I(), C()) |
| 71 | |
| 72 | # Like above, except that getattr(cls, '__bases__') raises an |
| 73 | # AttributeError, which /should/ get masked as a TypeError |
| 74 | def test_mask_attribute_error(self): |
| 75 | class I: pass |
| 76 | |
| 77 | class C(object): |
| 78 | def getbases(self): |
| 79 | raise AttributeError |
| 80 | __bases__ = property(getbases) |
| 81 | |
| 82 | self.assertRaises(TypeError, isinstance, I(), C()) |
| 83 | |
R. David Murray | 6bb9989 | 2010-11-20 16:33:30 +0000 | [diff] [blame] | 84 | # check that we don't mask non AttributeErrors |
| 85 | # see: http://bugs.python.org/issue1574217 |
| 86 | def test_isinstance_dont_mask_non_attribute_error(self): |
| 87 | class C(object): |
| 88 | def getclass(self): |
Benjamin Peterson | 5c41787 | 2010-11-20 17:22:13 +0000 | [diff] [blame] | 89 | raise RuntimeError |
| 90 | __class__ = property(getclass) |
R. David Murray | 6bb9989 | 2010-11-20 16:33:30 +0000 | [diff] [blame] | 91 | |
Benjamin Peterson | 5c41787 | 2010-11-20 17:22:13 +0000 | [diff] [blame] | 92 | c = C() |
R. David Murray | 6bb9989 | 2010-11-20 16:33:30 +0000 | [diff] [blame] | 93 | self.assertRaises(RuntimeError, isinstance, c, bool) |
| 94 | |
| 95 | # test another code path |
| 96 | class D: pass |
| 97 | self.assertRaises(RuntimeError, isinstance, c, D) |
Barry Warsaw | 906569d | 2002-04-23 22:48:42 +0000 | [diff] [blame] | 98 | |
| 99 | |
| 100 | # These tests are similar to above, but tickle certain code paths in |
| 101 | # issubclass() instead of isinstance() -- really PyObject_IsSubclass() |
| 102 | # vs. PyObject_IsInstance(). |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 103 | class TestIsSubclassExceptions(unittest.TestCase): |
Barry Warsaw | 906569d | 2002-04-23 22:48:42 +0000 | [diff] [blame] | 104 | def test_dont_mask_non_attribute_error(self): |
| 105 | class C(object): |
| 106 | def getbases(self): |
| 107 | raise RuntimeError |
| 108 | __bases__ = property(getbases) |
| 109 | |
| 110 | class S(C): pass |
| 111 | |
| 112 | self.assertRaises(RuntimeError, issubclass, C(), S()) |
| 113 | |
| 114 | def test_mask_attribute_error(self): |
| 115 | class C(object): |
| 116 | def getbases(self): |
| 117 | raise AttributeError |
| 118 | __bases__ = property(getbases) |
| 119 | |
| 120 | class S(C): pass |
Tim Peters | 8ac1495 | 2002-05-23 15:15:30 +0000 | [diff] [blame] | 121 | |
Barry Warsaw | 906569d | 2002-04-23 22:48:42 +0000 | [diff] [blame] | 122 | self.assertRaises(TypeError, issubclass, C(), S()) |
| 123 | |
| 124 | # Like above, but test the second branch, where the __bases__ of the |
| 125 | # second arg (the cls arg) is tested. This means the first arg must |
| 126 | # return a valid __bases__, and it's okay for it to be a normal -- |
| 127 | # unrelated by inheritance -- class. |
| 128 | def test_dont_mask_non_attribute_error_in_cls_arg(self): |
| 129 | class B: pass |
| 130 | |
| 131 | class C(object): |
| 132 | def getbases(self): |
| 133 | raise RuntimeError |
| 134 | __bases__ = property(getbases) |
| 135 | |
| 136 | self.assertRaises(RuntimeError, issubclass, B, C()) |
| 137 | |
| 138 | def test_mask_attribute_error_in_cls_arg(self): |
| 139 | class B: pass |
| 140 | |
| 141 | class C(object): |
| 142 | def getbases(self): |
| 143 | raise AttributeError |
| 144 | __bases__ = property(getbases) |
| 145 | |
| 146 | self.assertRaises(TypeError, issubclass, B, C()) |
| 147 | |
| 148 | |
| 149 | |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 150 | # meta classes for creating abstract classes and instances |
| 151 | class AbstractClass(object): |
| 152 | def __init__(self, bases): |
| 153 | self.bases = bases |
| 154 | |
| 155 | def getbases(self): |
| 156 | return self.bases |
| 157 | __bases__ = property(getbases) |
| 158 | |
| 159 | def __call__(self): |
| 160 | return AbstractInstance(self) |
| 161 | |
| 162 | class AbstractInstance(object): |
| 163 | def __init__(self, klass): |
| 164 | self.klass = klass |
| 165 | |
| 166 | def getclass(self): |
| 167 | return self.klass |
| 168 | __class__ = property(getclass) |
| 169 | |
| 170 | # abstract classes |
| 171 | AbstractSuper = AbstractClass(bases=()) |
| 172 | |
| 173 | AbstractChild = AbstractClass(bases=(AbstractSuper,)) |
| 174 | |
| 175 | # normal classes |
| 176 | class Super: |
| 177 | pass |
| 178 | |
| 179 | class Child(Super): |
| 180 | pass |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 181 | |
| 182 | class TestIsInstanceIsSubclass(unittest.TestCase): |
| 183 | # Tests to ensure that isinstance and issubclass work on abstract |
| 184 | # classes and instances. Before the 2.2 release, TypeErrors were |
| 185 | # raised when boolean values should have been returned. The bug was |
| 186 | # triggered by mixing 'normal' classes and instances were with |
| 187 | # 'abstract' classes and instances. This case tries to test all |
| 188 | # combinations. |
| 189 | |
| 190 | def test_isinstance_normal(self): |
Tim Peters | 8ac1495 | 2002-05-23 15:15:30 +0000 | [diff] [blame] | 191 | # normal instances |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 192 | self.assertEqual(True, isinstance(Super(), Super)) |
| 193 | self.assertEqual(False, isinstance(Super(), Child)) |
| 194 | self.assertEqual(False, isinstance(Super(), AbstractSuper)) |
| 195 | self.assertEqual(False, isinstance(Super(), AbstractChild)) |
| 196 | |
| 197 | self.assertEqual(True, isinstance(Child(), Super)) |
| 198 | self.assertEqual(False, isinstance(Child(), AbstractSuper)) |
| 199 | |
| 200 | def test_isinstance_abstract(self): |
Tim Peters | 8ac1495 | 2002-05-23 15:15:30 +0000 | [diff] [blame] | 201 | # abstract instances |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 202 | self.assertEqual(True, isinstance(AbstractSuper(), AbstractSuper)) |
| 203 | self.assertEqual(False, isinstance(AbstractSuper(), AbstractChild)) |
| 204 | self.assertEqual(False, isinstance(AbstractSuper(), Super)) |
| 205 | self.assertEqual(False, isinstance(AbstractSuper(), Child)) |
| 206 | |
| 207 | self.assertEqual(True, isinstance(AbstractChild(), AbstractChild)) |
| 208 | self.assertEqual(True, isinstance(AbstractChild(), AbstractSuper)) |
| 209 | self.assertEqual(False, isinstance(AbstractChild(), Super)) |
| 210 | self.assertEqual(False, isinstance(AbstractChild(), Child)) |
Tim Peters | 8ac1495 | 2002-05-23 15:15:30 +0000 | [diff] [blame] | 211 | |
Maggie Moss | 1b4552c | 2020-09-09 13:23:24 -0700 | [diff] [blame] | 212 | def test_isinstance_with_or_union(self): |
| 213 | self.assertTrue(isinstance(Super(), Super | int)) |
| 214 | self.assertFalse(isinstance(None, str | int)) |
| 215 | self.assertTrue(isinstance(3, str | int)) |
| 216 | self.assertTrue(isinstance("", str | int)) |
| 217 | self.assertTrue(isinstance([], typing.List | typing.Tuple)) |
| 218 | self.assertTrue(isinstance(2, typing.List | int)) |
| 219 | self.assertFalse(isinstance(2, typing.List | typing.Tuple)) |
| 220 | self.assertTrue(isinstance(None, int | None)) |
| 221 | self.assertFalse(isinstance(3.14, int | str)) |
| 222 | with self.assertRaises(TypeError): |
| 223 | isinstance(2, list[int]) |
| 224 | with self.assertRaises(TypeError): |
| 225 | isinstance(2, list[int] | int) |
| 226 | with self.assertRaises(TypeError): |
| 227 | isinstance(2, int | str | list[int] | float) |
| 228 | |
| 229 | |
| 230 | |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 231 | def test_subclass_normal(self): |
| 232 | # normal classes |
| 233 | self.assertEqual(True, issubclass(Super, Super)) |
| 234 | self.assertEqual(False, issubclass(Super, AbstractSuper)) |
| 235 | self.assertEqual(False, issubclass(Super, Child)) |
| 236 | |
| 237 | self.assertEqual(True, issubclass(Child, Child)) |
| 238 | self.assertEqual(True, issubclass(Child, Super)) |
| 239 | self.assertEqual(False, issubclass(Child, AbstractSuper)) |
Maggie Moss | 1b4552c | 2020-09-09 13:23:24 -0700 | [diff] [blame] | 240 | self.assertTrue(issubclass(typing.List, typing.List|typing.Tuple)) |
| 241 | self.assertFalse(issubclass(int, typing.List|typing.Tuple)) |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 242 | |
| 243 | def test_subclass_abstract(self): |
| 244 | # abstract classes |
| 245 | self.assertEqual(True, issubclass(AbstractSuper, AbstractSuper)) |
| 246 | self.assertEqual(False, issubclass(AbstractSuper, AbstractChild)) |
| 247 | self.assertEqual(False, issubclass(AbstractSuper, Child)) |
| 248 | |
| 249 | self.assertEqual(True, issubclass(AbstractChild, AbstractChild)) |
| 250 | self.assertEqual(True, issubclass(AbstractChild, AbstractSuper)) |
| 251 | self.assertEqual(False, issubclass(AbstractChild, Super)) |
| 252 | self.assertEqual(False, issubclass(AbstractChild, Child)) |
Tim Peters | 8ac1495 | 2002-05-23 15:15:30 +0000 | [diff] [blame] | 253 | |
Walter Dörwald | d9a6ad3 | 2002-12-12 16:41:44 +0000 | [diff] [blame] | 254 | def test_subclass_tuple(self): |
| 255 | # test with a tuple as the second argument classes |
| 256 | self.assertEqual(True, issubclass(Child, (Child,))) |
| 257 | self.assertEqual(True, issubclass(Child, (Super,))) |
| 258 | self.assertEqual(False, issubclass(Super, (Child,))) |
| 259 | self.assertEqual(True, issubclass(Super, (Child, Super))) |
| 260 | self.assertEqual(False, issubclass(Child, ())) |
Walter Dörwald | 7e5c6a0 | 2002-12-12 19:14:08 +0000 | [diff] [blame] | 261 | self.assertEqual(True, issubclass(Super, (Child, (Super,)))) |
| 262 | |
Guido van Rossum | e2a383d | 2007-01-15 16:59:06 +0000 | [diff] [blame] | 263 | self.assertEqual(True, issubclass(int, (int, (float, int)))) |
Jim Fasarakis-Hilliard | 094909a | 2017-05-02 20:17:18 +0300 | [diff] [blame] | 264 | self.assertEqual(True, issubclass(str, (str, (Child, str)))) |
Walter Dörwald | d9a6ad3 | 2002-12-12 16:41:44 +0000 | [diff] [blame] | 265 | |
Brett Cannon | 4f65331 | 2004-03-20 22:52:14 +0000 | [diff] [blame] | 266 | def test_subclass_recursion_limit(self): |
Yury Selivanov | f488fb4 | 2015-07-03 01:04:23 -0400 | [diff] [blame] | 267 | # make sure that issubclass raises RecursionError before the C stack is |
Brett Cannon | 4f65331 | 2004-03-20 22:52:14 +0000 | [diff] [blame] | 268 | # blown |
Yury Selivanov | f488fb4 | 2015-07-03 01:04:23 -0400 | [diff] [blame] | 269 | self.assertRaises(RecursionError, blowstack, issubclass, str, str) |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 270 | |
Brett Cannon | 4f65331 | 2004-03-20 22:52:14 +0000 | [diff] [blame] | 271 | def test_isinstance_recursion_limit(self): |
Yury Selivanov | f488fb4 | 2015-07-03 01:04:23 -0400 | [diff] [blame] | 272 | # make sure that issubclass raises RecursionError before the C stack is |
Tim Peters | 27f8836 | 2004-07-08 04:22:35 +0000 | [diff] [blame] | 273 | # blown |
Yury Selivanov | f488fb4 | 2015-07-03 01:04:23 -0400 | [diff] [blame] | 274 | self.assertRaises(RecursionError, blowstack, isinstance, '', str) |
Brett Cannon | 4f65331 | 2004-03-20 22:52:14 +0000 | [diff] [blame] | 275 | |
Maggie Moss | 1b4552c | 2020-09-09 13:23:24 -0700 | [diff] [blame] | 276 | def test_subclass_with_union(self): |
| 277 | self.assertTrue(issubclass(int, int | float | int)) |
| 278 | self.assertTrue(issubclass(str, str | Child | str)) |
| 279 | self.assertFalse(issubclass(dict, float|str)) |
| 280 | self.assertFalse(issubclass(object, float|str)) |
| 281 | with self.assertRaises(TypeError): |
| 282 | issubclass(2, Child | Super) |
| 283 | with self.assertRaises(TypeError): |
| 284 | issubclass(int, list[int] | Child) |
| 285 | |
Yonatan Goldschmidt | 1c56f8f | 2020-02-22 15:11:48 +0200 | [diff] [blame] | 286 | def test_issubclass_refcount_handling(self): |
| 287 | # bpo-39382: abstract_issubclass() didn't hold item reference while |
| 288 | # peeking in the bases tuple, in the single inheritance case. |
| 289 | class A: |
| 290 | @property |
| 291 | def __bases__(self): |
| 292 | return (int, ) |
| 293 | |
| 294 | class B: |
| 295 | def __init__(self): |
| 296 | # setting this here increases the chances of exhibiting the bug, |
| 297 | # probably due to memory layout changes. |
| 298 | self.x = 1 |
| 299 | |
| 300 | @property |
| 301 | def __bases__(self): |
| 302 | return (A(), ) |
| 303 | |
| 304 | self.assertEqual(True, issubclass(B(), int)) |
| 305 | |
Serhiy Storchaka | 9ece9cd | 2020-10-05 00:55:57 +0300 | [diff] [blame] | 306 | def test_infinite_recursion_in_bases(self): |
| 307 | class X: |
| 308 | @property |
| 309 | def __bases__(self): |
| 310 | return self.__bases__ |
| 311 | |
| 312 | self.assertRaises(RecursionError, issubclass, X(), int) |
| 313 | self.assertRaises(RecursionError, issubclass, int, X()) |
| 314 | self.assertRaises(RecursionError, isinstance, 1, X()) |
| 315 | |
Yonatan Goldschmidt | 1c56f8f | 2020-02-22 15:11:48 +0200 | [diff] [blame] | 316 | |
Brett Cannon | 4f65331 | 2004-03-20 22:52:14 +0000 | [diff] [blame] | 317 | def blowstack(fxn, arg, compare_to): |
| 318 | # Make sure that calling isinstance with a deeply nested tuple for its |
Yury Selivanov | f488fb4 | 2015-07-03 01:04:23 -0400 | [diff] [blame] | 319 | # argument will raise RecursionError eventually. |
Brett Cannon | 4f65331 | 2004-03-20 22:52:14 +0000 | [diff] [blame] | 320 | tuple_arg = (compare_to,) |
Guido van Rossum | 805365e | 2007-05-07 22:24:25 +0000 | [diff] [blame] | 321 | for cnt in range(sys.getrecursionlimit()+5): |
Brett Cannon | 4f65331 | 2004-03-20 22:52:14 +0000 | [diff] [blame] | 322 | tuple_arg = (tuple_arg,) |
| 323 | fxn(arg, tuple_arg) |
Neil Schemenauer | 3b04d63 | 2002-04-24 03:33:02 +0000 | [diff] [blame] | 324 | |
| 325 | |
Barry Warsaw | 906569d | 2002-04-23 22:48:42 +0000 | [diff] [blame] | 326 | if __name__ == '__main__': |
Zachary Ware | 38c707e | 2015-04-13 15:00:43 -0500 | [diff] [blame] | 327 | unittest.main() |