[Erp5-report] r36050 jerome - /erp5/trunk/products/ERP5/tests/testInventoryAPI.py

[nobody at svn.erp5.org]

Author: jerome
Date: Mon Jun  7 18:59:23 2010
New Revision: 36050

URL: http://svn.erp5.org?rev=36050&view=rev
Log:
Mark expected failures using expectedFailure decorator.
There has been a minor regression in r36023, passing the same group twice to
section_category (or other *_category such as node_category) was working in
getMovementHistoryList, eventhough it was not working in getInventoryList, for
now this is an expected failure, but it could be turned into a TypeError raised
by inventory API


Modified:
    erp5/trunk/products/ERP5/tests/testInventoryAPI.py

Modified: erp5/trunk/products/ERP5/tests/testInventoryAPI.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/tests/testInventoryAPI.py?rev=36050&r1=36049&r2=36050&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/tests/testInventoryAPI.py [utf8] (original)
+++ erp5/trunk/products/ERP5/tests/testInventoryAPI.py [utf8] Mon Jun  7 18:59:23 2010
@@ -43,6 +43,7 @@
 
 from Products.ERP5Type.tests.ERP5TypeTestCase import ERP5TypeTestCase
 from Products.ERP5Type.tests.utils import reindex
+from Products.ERP5Type.tests.backportUnittest import expectedFailure
 from Products.DCWorkflow.DCWorkflow import ValidationFailed
 from Products.ERP5Type.Base import _aq_reset
 
@@ -565,8 +566,8 @@
       self.getInventoryEquals(total_quantity,
                               node_category=category.getRelativeUrl())
   
-  # FIXME: this test is currently broken
-  def TODO_test_DoubleSectionCategory(self):
+  @expectedFailure
+  def test_DoubleCategoryMembershipSectionCategory(self):
     """Tests inventory on section category, when the section is twice member\
     of the same category like it happens for group and mapping"""
     self.section.setGroup('level1/level2')
@@ -1115,7 +1116,7 @@
                               'group/level1/level2',
                              ['group/level1', 'group/anotherlevel'],
                              ['group/level1', 'group/level1'],
-                             ['group/level1', 'group/level1/level2'], ]:
+                             ]:
       movement_history_list = getMovementHistoryList(
                                 section_category=section_category)
       self.assertEquals(len(movement_history_list), 1)
@@ -1130,6 +1131,19 @@
                         section_category='group/level1',
                         ignored='argument')), 1)
     
+  @expectedFailure
+  def testDoubleSectionCategory(self):
+    # it is currently invalid to pass the same category twice to inventory API
+    getMovementHistoryList = self.getSimulationTool().getMovementHistoryList
+    self.section.setGroup('level1/level2')
+    mvt = self._makeMovement(quantity=100)
+    movement_history_list = getMovementHistoryList(
+                              section_category=['group/level1',
+                                                'group/level1/level2'])
+    self.assertEquals(len(movement_history_list), 1)
+    self.assertEquals(movement_history_list[0].total_quantity, 100)
+
+
   def testNodeCategoryAndSectionCategory(self):
     getMovementHistoryList = self.getSimulationTool().getMovementHistoryList
     self.section.setGroup('level1/level2')
@@ -1138,8 +1152,7 @@
 
     valid_category_list = [ 'group/level1',
                            ['group/level1', 'group/anotherlevel'],
-                           ['group/level1', 'group/level1'],
-                           ['group/level1', 'group/level1/level2'], ]
+                           ['group/level1', 'group/level1'], ]
     invalid_category_list = ['group/anotherlevel', 'product_line/level1']
 
     # both valid