more test cleanup

src/Products/ZCatalog/tests/test_zcatalog.py

@@ -10,7 +10,7 @@
 # FOR A PARTICULAR PURPOSE.
 #
 ##############################################################################
-""" Unittests for Catalog.
+""" Unittests for ZCatalog
 """
 
 import unittest
@@ -61,7 +61,6 @@ class zdummyFalse(zdummy):
     def __nonzero__(self):
         return False
 
-# make objects with failing __len__ and __nonzero__
 
 class dummyLenFail(zdummy):
 
@@ -103,6 +102,20 @@ class fakeparent(Implicit):
         return result
 
 
+class PickySecurityManager:
+
+    def __init__(self, badnames=[]):
+        self.badnames = badnames
+
+    def validateValue(self, value):
+        return 1
+
+    def validate(self, accessed, container, name, value):
+        if name not in self.badnames:
+            return 1
+        raise Unauthorized(name)
+
+
 class TestZCatalog(unittest.TestCase):
 
     def setUp(self):
@@ -228,20 +241,6 @@ class TestZCatalog(unittest.TestCase):
         self.assertEquals(catalog.getobject(rid0), None)
 
 
-class PickySecurityManager:
-
-    def __init__(self, badnames=[]):
-        self.badnames = badnames
-
-    def validateValue(self, value):
-        return 1
-
-    def validate(self, accessed, container, name, value):
-        if name not in self.badnames:
-            return 1
-        raise Unauthorized(name)
-
-
 class TestZCatalogGetObject(unittest.TestCase):
     # Test what objects are returned by brain.getObject()