diff options
author | Christian Heimes <christian@cheimes.de> | 2008-03-03 18:28:04 (GMT) |
---|---|---|
committer | Christian Heimes <christian@cheimes.de> | 2008-03-03 18:28:04 (GMT) |
commit | 608c1d8e87b20116011fe8fce634e980e115d514 (patch) | |
tree | e7204b4c31d30b37d308dda03cc03c032f5df08e /Lib | |
parent | 1f178a6fac790f380b6830642b8c9afc0a3a9931 (diff) | |
download | cpython-608c1d8e87b20116011fe8fce634e980e115d514.zip cpython-608c1d8e87b20116011fe8fce634e980e115d514.tar.gz cpython-608c1d8e87b20116011fe8fce634e980e115d514.tar.bz2 |
Since abc._Abstract was replaces by a new type flags the regression test suite fails. I've added a new function inspect.isabstract(). Is the mmethod fine or should I check if object is a instance of type or subclass of object, too?
Diffstat (limited to 'Lib')
-rw-r--r-- | Lib/inspect.py | 8 | ||||
-rwxr-xr-x | Lib/test/regrtest.py | 7 | ||||
-rw-r--r-- | Lib/test/test_abc.py | 5 |
3 files changed, 16 insertions, 4 deletions
diff --git a/Lib/inspect.py b/Lib/inspect.py index 42f2c31..3f9199f 100644 --- a/Lib/inspect.py +++ b/Lib/inspect.py @@ -38,11 +38,15 @@ import dis import imp import tokenize import linecache +from abc import ABCMeta from operator import attrgetter from collections import namedtuple from compiler.consts import (CO_OPTIMIZED, CO_NEWLOCALS, CO_VARARGS, CO_VARKEYWORDS, CO_GENERATOR) +# See Include/object.h +TPFLAGS_IS_ABSTRACT = 1 << 20 + # ----------------------------------------------------------- type-checking def ismodule(object): """Return true if the object is a module. @@ -241,6 +245,10 @@ def isgenerator(object): """Return true if the object is a generator object.""" return isinstance(object, types.GeneratorType) +def isabstract(object): + """Return true if the object is an abstract base class (ABC).""" + return object.__flags__ & TPFLAGS_IS_ABSTRACT + def getmembers(object, predicate=None): """Return all members of an object as (name, value) pairs sorted by name. Optionally, only return members that satisfy a given predicate.""" diff --git a/Lib/test/regrtest.py b/Lib/test/regrtest.py index b6aa96b..dfb01ba 100755 --- a/Lib/test/regrtest.py +++ b/Lib/test/regrtest.py @@ -129,6 +129,7 @@ import warnings import re import cStringIO import traceback +from inspect import isabstract # I see no other way to suppress these warnings; # putting them in test_grammar.py has no effect: @@ -649,7 +650,6 @@ def cleanup_test_droppings(testname, verbose): def dash_R(the_module, test, indirect_test, huntrleaks): # This code is hackish and inelegant, but it seems to do the job. import copy_reg, _abcoll - from abc import _Abstract if not hasattr(sys, 'gettotalrefcount'): raise Exception("Tracking reference leaks requires a debug build " @@ -661,7 +661,7 @@ def dash_R(the_module, test, indirect_test, huntrleaks): pic = sys.path_importer_cache.copy() abcs = {} for abc in [getattr(_abcoll, a) for a in _abcoll.__all__]: - if not issubclass(abc, _Abstract): + if not isabstract(abc): continue for obj in abc.__subclasses__() + [abc]: abcs[obj] = obj._abc_registry.copy() @@ -699,7 +699,6 @@ def dash_R_cleanup(fs, ps, pic, abcs): import _strptime, linecache, dircache import urlparse, urllib, urllib2, mimetypes, doctest import struct, filecmp, _abcoll - from abc import _Abstract from distutils.dir_util import _path_created # Restore some original values. @@ -714,7 +713,7 @@ def dash_R_cleanup(fs, ps, pic, abcs): # Clear ABC registries, restoring previously saved ABC registries. for abc in [getattr(_abcoll, a) for a in _abcoll.__all__]: - if not issubclass(abc, _Abstract): + if not isabstract(abc): continue for obj in abc.__subclasses__() + [abc]: obj._abc_registry = abcs.get(obj, {}).copy() diff --git a/Lib/test/test_abc.py b/Lib/test/test_abc.py index c53dcea..f1c0c3b 100644 --- a/Lib/test/test_abc.py +++ b/Lib/test/test_abc.py @@ -7,6 +7,7 @@ import unittest from test import test_support import abc +from inspect import isabstract class TestABC(unittest.TestCase): @@ -43,19 +44,23 @@ class TestABC(unittest.TestCase): def bar(self): pass # concrete self.assertEqual(C.__abstractmethods__, set(["foo"])) self.assertRaises(TypeError, C) # because foo is abstract + self.assert_(isabstract(C)) class D(C): def bar(self): pass # concrete override of concrete self.assertEqual(D.__abstractmethods__, set(["foo"])) self.assertRaises(TypeError, D) # because foo is still abstract + self.assert_(isabstract(D)) class E(D): def foo(self): pass self.assertEqual(E.__abstractmethods__, set()) E() # now foo is concrete, too + self.failIf(isabstract(E)) class F(E): @abstractthing def bar(self): pass # abstract override of concrete self.assertEqual(F.__abstractmethods__, set(["bar"])) self.assertRaises(TypeError, F) # because bar is abstract now + self.assert_(isabstract(F)) def test_subclass_oldstyle_class(self): class A: |