[Zope3-checkins] CVS: Zope3/src/zope/security/tests - test_checker.py:1.7

Steve Alexander steve@cat-box.net
Wed, 23 Apr 2003 14:18:04 -0400


Update of /cvs-repository/Zope3/src/zope/security/tests
In directory cvs.zope.org:/tmp/cvs-serv6978/src/zope/security/tests

Modified Files:
	test_checker.py 
Log Message:
Reverted my change as Guido pointed out why exception classes need to
be unwrapped as well as instances. This needs more thinking about
though, as no tests failed...

If all extension instances are unwrapped, and extension classes are
included inline in the code, then there is no problem with comparison.

I'll talk to people and experiment tomorrow, and perhaps re-check this in.


=== Zope3/src/zope/security/tests/test_checker.py 1.6 => 1.7 ===
--- Zope3/src/zope/security/tests/test_checker.py:1.6	Wed Apr 23 13:38:49 2003
+++ Zope3/src/zope/security/tests/test_checker.py	Wed Apr 23 14:18:02 2003
@@ -180,8 +180,7 @@
         for rock in (1, 1.0, 1l, 1j,
                      '1', u'1', None,
                      AttributeError(),
-                     # Not AttributeError though. No reason to consider
-                     # exception classes as rocks.
+                     AttributeError
                      ):
             proxy = checker.proxy(rock)