Log message for revision 115074: Avoid a bunch of warning hooks by adjusting tests
Changed: U Zope/trunk/src/Products/ZCatalog/tests/testCatalog.py -=- Modified: Zope/trunk/src/Products/ZCatalog/tests/testCatalog.py =================================================================== --- Zope/trunk/src/Products/ZCatalog/tests/testCatalog.py 2010-07-25 11:01:04 UTC (rev 115073) +++ Zope/trunk/src/Products/ZCatalog/tests/testCatalog.py 2010-07-25 11:11:29 UTC (rev 115074) @@ -74,15 +74,14 @@ class CatalogBase: + def setUp(self): self._catalog = Catalog() - self.warningshook = WarningsHook() - self.warningshook.install() def tearDown(self): self._catalog = None - self.warningshook.uninstall() + class TestAddDelColumn(CatalogBase,unittest.TestCase): def testAdd(self): self._catalog.addColumn('id') @@ -199,9 +198,6 @@ def setUp(self): from Products.ZCatalog.ZCatalog import ZCatalog - self.warningshook = WarningsHook() - self.warningshook.install() - self._catalog = ZCatalog('Catalog') self._catalog.resolve_path = self._resolve_num self._catalog.addIndex('title', 'KeywordIndex') @@ -216,9 +212,6 @@ self.d[str(x)] = ob self._catalog.catalog_object(ob, str(x)) - def tearDown(self): - self.warningshook.uninstall() - def _resolve_num(self, num): return self.d[num] @@ -605,14 +598,13 @@ # Test merging results from multiple catalogs def setUp(self): - self.warningshook = WarningsHook() - self.warningshook.install() self.catalogs = [] for i in range(3): cat = Catalog() cat.lexicon = PLexicon('lexicon') cat.addIndex('num', FieldIndex('num')) cat.addIndex('big', FieldIndex('big')) + cat.addIndex('number', FieldIndex('number')) i = ZCTextIndex('title', caller=cat, index_factory=OkapiIndex, lexicon_id='lexicon') cat.addIndex('title', i) @@ -620,15 +612,14 @@ for i in range(10): obj = zdummy(i) obj.big = i > 5 + obj.number = True cat.catalogObject(obj, str(i)) self.catalogs.append(cat) - def tearDown(self): - self.warningshook.uninstall() - def testNoFilterOrSort(self): from Products.ZCatalog.Catalog import mergeResults - results = [cat.searchResults(_merge=0) for cat in self.catalogs] + results = [cat.searchResults( + dict(number=True), _merge=0) for cat in self.catalogs] merged_rids = [r.getRID() for r in mergeResults( results, has_sort_keys=False, reverse=False)] expected = [r.getRID() for r in chain(*results)] @@ -636,7 +627,8 @@ def testSortedOnly(self): from Products.ZCatalog.Catalog import mergeResults - results = [cat.searchResults(sort_on='num', _merge=0) + results = [cat.searchResults( + dict(number=True, sort_on='num'), _merge=0) for cat in self.catalogs] merged_rids = [r.getRID() for r in mergeResults( results, has_sort_keys=True, reverse=False)] @@ -646,7 +638,8 @@ def testSortReverse(self): from Products.ZCatalog.Catalog import mergeResults - results = [cat.searchResults(sort_on='num', _merge=0) + results = [cat.searchResults( + dict(number=True, sort_on='num'), _merge=0) for cat in self.catalogs] merged_rids = [r.getRID() for r in mergeResults( results, has_sort_keys=True, reverse=True)] @@ -657,7 +650,8 @@ def testLimitSort(self): from Products.ZCatalog.Catalog import mergeResults - results = [cat.searchResults(sort_on='num', sort_limit=2, _merge=0) + results = [cat.searchResults( + dict(number=True, sort_on='num'), sort_limit=2, _merge=0) for cat in self.catalogs] merged_rids = [r.getRID() for r in mergeResults( results, has_sort_keys=True, reverse=False)] @@ -678,7 +672,8 @@ def testSmallIndexSort(self): # Test that small index sort optimization is not used for merging from Products.ZCatalog.Catalog import mergeResults - results = [cat.searchResults(sort_on='big', _merge=0) + results = [cat.searchResults( + dict(number=True, sort_on='big'), _merge=0) for cat in self.catalogs] merged_rids = [r.getRID() for r in mergeResults( results, has_sort_keys=True, reverse=False)] @@ -705,8 +700,6 @@ def setUp(self): from Products.ZCatalog.ZCatalog import ZCatalog - self.warningshook = WarningsHook() - self.warningshook.install() catalog = ZCatalog('catalog') catalog.addIndex('id', 'FieldIndex') root = Folder('') @@ -715,7 +708,6 @@ self.root.catalog = catalog def tearDown(self): - self.warningshook.uninstall() noSecurityManager() if self._old_flag is not None: self._restore_getObject_flag() @@ -735,7 +727,7 @@ catalog = root.catalog root.ob = Folder('ob') catalog.catalog_object(root.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] self.assertEqual(brain.getPath(), '/ob') self.assertEqual(brain.getObject().getId(), 'ob') @@ -747,7 +739,7 @@ catalog = root.catalog root.ob = Folder('ob') catalog.catalog_object(root.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] del root.ob self.assertRaises((NotFound, AttributeError, KeyError), brain.getObject) @@ -760,7 +752,7 @@ root.fold = Folder('fold') root.fold.ob = Folder('ob') catalog.catalog_object(root.fold.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] # allow all accesses pickySecurityManager = PickySecurityManager() setSecurityManager(pickySecurityManager) @@ -783,7 +775,7 @@ catalog = root.catalog root.ob = Folder('ob') catalog.catalog_object(root.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] del root.ob self.assertEqual(brain.getObject(), None) @@ -796,7 +788,7 @@ root.fold = Folder('fold') root.fold.ob = Folder('ob') catalog.catalog_object(root.fold.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] # allow all accesses pickySecurityManager = PickySecurityManager() setSecurityManager(pickySecurityManager) @@ -820,7 +812,7 @@ catalog = root.catalog root.ob = Folder('ob') catalog.catalog_object(root.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] self.assertEqual(brain.getPath(), '/ob') self.assertEqual(brain._unrestrictedGetObject().getId(), 'ob') @@ -832,7 +824,7 @@ root.fold = Folder('fold') root.fold.ob = Folder('ob') catalog.catalog_object(root.fold.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] # allow all accesses pickySecurityManager = PickySecurityManager() setSecurityManager(pickySecurityManager) @@ -854,7 +846,7 @@ catalog = root.catalog root.ob = Folder('ob') catalog.catalog_object(root.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] del root.ob self.assertRaises((NotFound, AttributeError, KeyError), brain._unrestrictedGetObject) @@ -866,7 +858,7 @@ catalog = root.catalog root.ob = Folder('ob') catalog.catalog_object(root.ob) - brain = catalog.searchResults()[0] + brain = catalog.searchResults({'id': 'ob'})[0] del root.ob self.assertEqual(brain._unrestrictedGetObject(), None) _______________________________________________ Zope-Checkins maillist - Zope-Checkins@zope.org https://mail.zope.org/mailman/listinfo/zope-checkins