Fix a long-standing bug with ZOPE_WATCH_CHECKERS=2

2 files changed, 11 insertions, 1 deletions

diff --git a/src/zope/security/checker.py b/src/zope/security/checker.py
index 89ae214..f92a2ff 100644
--- a/src/zope/security/checker.py
+++ b/src/zope/security/checker.py
@@ -515,7 +515,7 @@ class CheckerLoggingMixin(object):
     _file = sys.stderr
 
     def _log(self, msg, verbosity=1):
-        if verbosity >= self.verbosity:
+        if self.verbosity >= verbosity:
             self._file.write('%s\n' % msg)
 
     def check(self, object, name):
diff --git a/src/zope/security/tests/test_checker.py b/src/zope/security/tests/test_checker.py
index f6ad51b..ffcfc79 100644
--- a/src/zope/security/tests/test_checker.py
+++ b/src/zope/security/tests/test_checker.py
@@ -1162,6 +1162,16 @@ class CheckerLoggingMixinTests(unittest.TestCase):
         self.assertEqual(checker._file[0],
                          '[CHK] - Unauthorized: name on TESTING\n')
 
+    def test_check_unauthorized_raised_verbosity(self):
+        from zope.security.interfaces import Unauthorized
+        checker = self._makeOne(Unauthorized)
+        checker.verbosity = 2
+        self.assertRaises(Unauthorized,
+                          checker.check, self._makeObject(), 'name')
+        self.assertEqual(len(checker._file), 1)
+        self.assertEqual(checker._file[0],
+                         '[CHK] - Unauthorized: name on TESTING\n')
+
     def test_check_forbidden_attribute(self):
         from zope.security.interfaces import ForbiddenAttribute
         checker = self._makeOne(ForbiddenAttribute)