[Erp5-report] r42801 nicolas.dumazet - /erp5/trunk/products/ERP5Type/tests/testERP5Type.py

[nobody at svn.erp5.org]

Author: nicolas.dumazet
Date: Mon Jan 31 11:17:40 2011
New Revision: 42801

URL: http://svn.erp5.org?rev=42801&view=rev
Log:
this test was only passing due to luck, as destination category is manipulated
elsewhere.

Modified:
    erp5/trunk/products/ERP5Type/tests/testERP5Type.py

Modified: erp5/trunk/products/ERP5Type/tests/testERP5Type.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/tests/testERP5Type.py?rev=42801&r1=42800&r2=42801&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/tests/testERP5Type.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/tests/testERP5Type.py [utf8] Mon Jan 31 11:17:40 2011
@@ -2399,10 +2399,24 @@ class TestPropertySheet:
     def test_aq_reset_on_portal_types_properties_change(self):
       doc = self.portal.person_module.newContent(portal_type='Person')
       ti = self.getTypesTool()['Person']
-      self.assertFalse(hasattr(doc, 'getDestination'))
-      ti.edit(type_base_category_list=
-        ti.getTypeBaseCategoryList() + ['destination'])
-      self.assertTrue(hasattr(doc, 'getDestination'))
+      base_category_list = ti.getTypeBaseCategoryList()
+      # this test is poorly isolated, and the _19*_ add destination
+      # to the base categories
+      if 'destination' not in base_category_list:
+
+        self.assertFalse(hasattr(doc, 'getDestination'))
+        ti.edit(type_base_category_list=
+          base_category_list + ['destination'])
+
+        transaction.commit()
+        self.assertTrue(hasattr(doc, 'getDestination'))
+      else:
+        self.assertTrue(hasattr(doc, 'getDestination'))
+        base_category_list.remove('destination')
+        ti.edit(type_base_category_list=base_category_list)
+
+        transaction.commit()
+        self.assertFalse(hasattr(doc, 'getDestination'))
 
     def test_aq_reset_on_workflow_chain_change(self):
       doc = self.portal.person_module.newContent(portal_type='Person')