[Erp5-report] r42865 nicolas.dumazet - /erp5/trunk/products/ERP5Type/tests/

[nobody at svn.erp5.org]

Author: nicolas.dumazet
Date: Tue Feb  1 05:53:45 2011
New Revision: 42865

URL: http://svn.erp5.org?rev=42865&view=rev
Log:
arg, several tests were using those utility methods.
Clarify the situation: raise useful errors.

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

Modified: erp5/trunk/products/ERP5Type/tests/ERP5TypeTestCase.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/tests/ERP5TypeTestCase.py?rev=42865&r1=42864&r2=42865&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/tests/ERP5TypeTestCase.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/tests/ERP5TypeTestCase.py [utf8] Tue Feb  1 05:53:45 2011
@@ -399,12 +399,16 @@ class ERP5TypeTestCaseMixin(ProcessingNo
           getattr(self.getPortal(), 'currency', None))
 
     def _addPropertySheet(self, portal_type_name,
-                          property_sheet_name='TestPropertySheet'):
+                          property_sheet_name='TestPropertySheet',
+                          deprecated=None):
       """Utility method to add a property sheet to a type information.
       You might be interested in the higer level method _addProperty
       This method registers all added property sheets, to be able to remove
       them in tearDown.
       """
+      if deprecated is not None:
+        raise ValueError("Please update this test to use ZODB property sheets")
+
       portal_property_sheets = self.portal.portal_property_sheets
       property_sheet = getattr(portal_property_sheets, property_sheet_name, None)
       if property_sheet is None:

Modified: erp5/trunk/products/ERP5Type/tests/testERP5Type.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/tests/testERP5Type.py?rev=42865&r1=42864&r2=42865&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/tests/testERP5Type.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/tests/testERP5Type.py [utf8] Tue Feb  1 05:53:45 2011
@@ -83,18 +83,31 @@ class PropertySheetTestCase(ERP5TypeTest
     transaction.commit()
     super(PropertySheetTestCase, self).tearDown()
 
-  def _addProperty(self, portal_type_name, property_sheet_id, property_id,
+  def _addProperty(self, portal_type_name, property_sheet_id,
+                   property_id=None,
                    commit=True, **kw):
-    """quickly add a property to a type"""
-    self.assertTrue('portal_type' in kw)
+    """quickly add a property to a type
+
+    It always associate / create the property sheet with id
+    property_sheet_id to the portal type portal_type_name.
+
+    When property_sheet_id is passed, we create a property of this
+    id, using kw** as parameters to the constructor.
+    """
 
     ps = self._addPropertySheet(portal_type_name,
                                 property_sheet_name=property_sheet_id)
-    property = getattr(ps, property_id, None)
-    if property is not None:
-      ps._delObject(property_id)
 
-    property = ps.newContent(reference=property_id, **kw)
+    if property_id is not None:
+      if "\n" in property_id:
+        raise ValueError("Please update this test to use ZODB property sheets")
+
+      self.assertTrue('portal_type' in kw)
+      property = getattr(ps, property_id, None)
+      if property is not None:
+        ps._delObject(property_id)
+
+      property = ps.newContent(reference=property_id, **kw)
     if commit:
       transaction.commit()