[Checkins] SVN: Products.CMFDefault/trunk/Products/CMFDefault/browser/search/tests/test_search.py Testing of query handling split into two tests.

Charlie Clark charlie at begeistert.org
Tue Oct 12 06:05:10 EDT 2010


Log message for revision 117471:
  Testing of query handling split into two tests.

Changed:
  U   Products.CMFDefault/trunk/Products/CMFDefault/browser/search/tests/test_search.py

-=-
Modified: Products.CMFDefault/trunk/Products/CMFDefault/browser/search/tests/test_search.py
===================================================================
--- Products.CMFDefault/trunk/Products/CMFDefault/browser/search/tests/test_search.py	2010-10-12 10:03:55 UTC (rev 117470)
+++ Products.CMFDefault/trunk/Products/CMFDefault/browser/search/tests/test_search.py	2010-10-12 10:05:09 UTC (rev 117471)
@@ -39,12 +39,19 @@
         self.assertFalse(view.is_anonymous)
         self.assertNotEqual(view.search_fields.get('review_state'), None)
 
+    def test_strip_unused_paramaters(self):
+        view = self._getTargetClass()
+        data = {'portal_type': ['Document'], 'review_state':u"None",
+                'Subject':u"None"}
+        view.handle_search('search', data)
+        self.assertEqual(view._query, {'portal_type':['Document']})
+
     def test_add_search_vars_to_hidden(self):
         view = self._getTargetClass()
-        self.assertEqual(view._query, {})
+        self.assertFalse(hasattr(view, '_query'))
         data = {'portal_type': ['Document']}
         view.handle_search('search', data)
-        self.assertEqual(view._query, data)
+        self.assertEqual(view._getHiddenVars(), data)
 
     def test_search_returns_results(self):
         view = self._getTargetClass()
@@ -52,9 +59,6 @@
         view.handle_search('search', {})
         self.assertEqual(view.template.filename, view.results.filename)
 
-    def results(self):
-        pass
-
 def test_suite():
     suite = unittest.TestSuite()
     suite.addTest(unittest.makeSuite(SearchFormTests))



More information about the checkins mailing list