Commit 20e42434 authored by Jérome Perrin's avatar Jérome Perrin

change the way to test simulation movements generated by invoice transaction

rule, because their IDs are no longer predictables


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@25228 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 1b5b2386
...@@ -290,22 +290,29 @@ class TestInvoiceMixin(TestPackingListMixin, ...@@ -290,22 +290,29 @@ class TestInvoiceMixin(TestPackingListMixin,
def stepCheckInvoiceTransactionRule(self, sequence=None, sequence_list=None, def stepCheckInvoiceTransactionRule(self, sequence=None, sequence_list=None,
**kw): **kw):
""" """
Checks that the invoice_transaction_rule is expanded and its movements are Checks that the applied invoice_transaction_rule is expanded and its movements are
consistent with its parent movement consistent with its parent movement
""" """
invoice_transaction_rule_list = \ invoice_transaction_rule_list = \
sequence.get('invoice_transaction_rule_list') sequence.get('invoice_transaction_rule_list')
for invoice_transaction_rule in invoice_transaction_rule_list: for applied_invoice_transaction_rule in invoice_transaction_rule_list:
parent_movement = aq_parent(invoice_transaction_rule) parent_movement = aq_parent(applied_invoice_transaction_rule)
self.assertEquals(3, len(invoice_transaction_rule.objectValues())) invoice_transaction_rule = \
applied_invoice_transaction_rule.getSpecialiseValue()
self.assertEquals(3, len(applied_invoice_transaction_rule.objectValues()))
for line_id, line_source_id, line_destination_id, line_ratio in \ for line_id, line_source_id, line_destination_id, line_ratio in \
self.transaction_line_definition_list: self.transaction_line_definition_list:
movement = getattr(invoice_transaction_rule, line_id, None) movement = None
for simulation_movement in \
applied_invoice_transaction_rule.objectValues():
if simulation_movement.getSourceId() == line_source_id and\
simulation_movement.getDestinationId() == line_destination_id:
movement = simulation_movement
break
self.assertTrue(movement is not None) self.assertTrue(movement is not None)
self.assertEquals(movement.getCorrectedQuantity(), parent_movement.getPrice() * self.assertEquals(movement.getCorrectedQuantity(), parent_movement.getPrice() *
parent_movement.getCorrectedQuantity() * line_ratio) parent_movement.getCorrectedQuantity() * line_ratio)
self.assertEquals(movement.getSourceId(), line_source_id)
self.assertEquals(movement.getDestinationId(), line_destination_id)
self.assertEquals(movement.getStartDate(), self.assertEquals(movement.getStartDate(),
parent_movement.getStartDate()) parent_movement.getStartDate())
self.assertEquals(movement.getStopDate(), self.assertEquals(movement.getStopDate(),
...@@ -927,7 +934,6 @@ class TestInvoice(TestInvoiceMixin): ...@@ -927,7 +934,6 @@ class TestInvoice(TestInvoiceMixin):
price=5, price=5,
variation_category_list=['size/Child/32'], variation_category_list=['size/Child/32'],
mapped_value_property_list=['quantity', 'price'],) mapped_value_property_list=['quantity', 'price'],)
order.confirm() order.confirm()
get_transaction().commit() get_transaction().commit()
self.tic() self.tic()
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment