[Erp5-report] r35625 luke - /erp5/trunk/products/ERP5Security/tests/testERP5Security.py

[nobody at svn.erp5.org]

Author: luke
Date: Tue May 25 18:07:12 2010
New Revision: 35625

URL: http://svn.erp5.org?rev=35625&view=rev
Log:
 - do not clear cache in test, as it is not needed and prevent finding wrong caches

Modified:
    erp5/trunk/products/ERP5Security/tests/testERP5Security.py

Modified: erp5/trunk/products/ERP5Security/tests/testERP5Security.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Security/tests/testERP5Security.py?rev=35625&r1=35624&r2=35625&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Security/tests/testERP5Security.py [utf8] (original)
+++ erp5/trunk/products/ERP5Security/tests/testERP5Security.py [utf8] Tue May 25 18:07:12 2010
@@ -57,8 +57,6 @@
 
   def beforeTearDown(self):
     """Clears person module and invalidate caches when tests are finished."""
-    # XXX Isn't it better to clear the cache when deleting a Person ?
-    clearCache(cache_factory_list=('erp5_content_short', ))
     self.getPersonModule().manage_delObjects([x for x in
                              self.getPersonModule().objectIds()])
     transaction.commit()
@@ -412,8 +410,6 @@
     # commit this
     transaction.commit()
     self.tic()
-    # XXX Isn't it better to clear the cache when deleting a Person ?
-    clearCache(cache_factory_list=('erp5_content_short', ))
 
   def loginAsUser(self, username):
     uf = self.portal.acl_users