diff --git a/src/persistent/tests/persistenttestbase.py b/src/persistent/tests/persistenttestbase.py index 439d3da78cf00dd2c1e62014d1a1ff39539cf4a7..6d15a4214c4520dc49404183b2ec6c1e92bf494a 100644 --- a/src/persistent/tests/persistenttestbase.py +++ b/src/persistent/tests/persistenttestbase.py @@ -266,26 +266,31 @@ class Test(unittest.TestCase): self.assert_(P.__dictoffset__ < P.__weakrefoffset__) self.assert_(P.__basicsize__ > Persistent.__basicsize__) - def testDeactivateErrors(self): - p = self.klass() - p._p_oid = '\0\0\0\0\0\0hi' - dm = DM() - p._p_jar = dm - - def typeerr(*args, **kwargs): - self.assertRaises(TypeError, p, *args, **kwargs) - - typeerr(1) - typeerr(1, 2) - typeerr(spam=1) - typeerr(spam=1, force=1) - - p._p_changed = True - class Err(object): - def __nonzero__(self): - raise RuntimeError - - typeerr(force=Err()) +# XXX Can anyone defend/explain the test below? The tests classes defined here +# don't define __call__, so this weird test will always pass, but to what +# end? What the heck is the point. If a klass is given that happens +# to define __call__, the test *may* mysteriously fail. Who cares? + +## def testDeactivateErrors(self): +## p = self.klass() +## p._p_oid = '\0\0\0\0\0\0hi' +## dm = DM() +## p._p_jar = dm + +## def typeerr(*args, **kwargs): +## self.assertRaises(TypeError, p, *args, **kwargs) + +## typeerr(1) +## typeerr(1, 2) +## typeerr(spam=1) +## typeerr(spam=1, force=1) + +## p._p_changed = True +## class Err(object): +## def __nonzero__(self): +## raise RuntimeError + +## typeerr(force=Err()) class P(Persistent): def __init__(self):