slapos_accounting: Rework creation of the Reversal Sale Invoice Transactions

  Unify and drop duplicated code present on slapos_payzen and slapos_wechat.
  Update and move tests related to the change

master/bt5/slapos_accounting/ActionTemplateItem/portal_types/Sale%20Invoice%20Transaction/create_slapos_reversal.xml

@@ -88,7 +88,7 @@
       <dictionary>
         <item>
             <key> <string>text</string> </key>
-            <value> <string>python: object.getSimulationState() in (\'stopped\', )</string> </value>
+            <value> <string>python: object.getSimulationState() in (\'stopped\', ) and not object.SaleInvoiceTransaction_isLettered()</string> </value>
         </item>
       </dictionary>
     </pickle>

master/bt5/slapos_wechat/SkinTemplateItem/portal_skins/slapos_wechat/SaleInvoiceTransaction_createReversalWechatTransaction.py → master/bt5/slapos_accounting/SkinTemplateItem/portal_skins/slapos_accounting/SaleInvoiceTransaction_createReversalSaleInvoiceTransaction.py

-""" Create a reversal transaction from current wechat transaction. """
-from zExceptions import Unauthorized
-if REQUEST is not None:
-  raise Unauthorized
-
+""" Create a reversal transaction from current transaction. """
 portal = context.getPortalObject()
 
+if not batch_mode and context.getPaymentMode() not in ["payzen", "wechat"]:
+  message = context.Base_translateString("The payment mode is unsupported.")
+  return context.Base_redirect(keep_items={'portal_status_message': message})
+
 # Check that we are in state that we are waiting for user manual payment
 assert context.getPortalType() == 'Sale Invoice Transaction'
-assert context.getPaymentMode() == 'wechat'
+assert context.getPaymentMode() in ('payzen', 'wechat')
 assert context.getSimulationState() == 'stopped'
 assert context.getTotalPrice() != 0
 assert context.getSpecialise() in ("sale_trade_condition_module/slapos_aggregated_trade_condition",
                                    "sale_trade_condition_module/slapos_aggregated_subscription_trade_condition")
 
-
 # Dont create if the invoice is already paied
 assert not context.SaleInvoiceTransaction_isLettered()
 
@@ -21,9 +20,16 @@ payment = portal.portal_catalog.getResultValue(
   portal_type="Payment Transaction",
   simulation_state="started",
   default_causality_uid=context.getUid(),
-  default_payment_mode_uid=portal.portal_categories.payment_mode.wechat.getUid(),
+  default_payment_mode_uid=[
+    portal.portal_categories.payment_mode.payzen.getUid(),
+    portal.portal_categories.payment_mode.wechat.getUid()],
 )
-if payment is not None and payment.PaymentTransaction_getWechatId()[1] is None:
+if payment is not None:
+  payment_mode = payment.getPaymentMode()
+  if payment_mode == 'payzen' and payment.PaymentTransaction_getPayzenId()[1] is not None:
+    # The payment transaction will be cancelled by a proper alarm.
+    raise ValueError("Payment Transaction is waiting for External Payzen confirmation!")
+  elif payment_mode == 'wechat' and payment.PaymentTransaction_getWechatId()[1] is not None:
     # The payment transaction will be cancelled by a proper alarm.
     raise ValueError("Payment Transaction is waiting for External Wechat confirmation!")
 
@@ -44,4 +50,9 @@ for line in reversal_transaction.getMovementList():
 reversal_transaction.confirm(comment="Automatic because of reversal creation")
 reversal_transaction.stop(comment="Automatic because of reversal creation")
 
-return reversal_transaction
+if batch_mode:
+  return reversal_transaction
+
+message = context.Base_translateString("Reversal Transaction created.")
+return reversal_transaction.Base_redirect(
+  keep_items={'portal_status_message': message})

master/bt5/slapos_wechat/SkinTemplateItem/portal_skins/slapos_wechat/SaleInvoiceTransaction_createReversalWechatTransaction.xml → master/bt5/slapos_accounting/SkinTemplateItem/portal_skins/slapos_accounting/SaleInvoiceTransaction_createReversalSaleInvoiceTransaction.xml

@@ -50,11 +50,11 @@
         </item>
         <item>
             <key> <string>_params</string> </key>
-            <value> <string>REQUEST=None</string> </value>
+            <value> <string>batch_mode=False</string> </value>
         </item>
         <item>
             <key> <string>id</string> </key>
-            <value> <string>SaleInvoiceTransaction_createReversalWechatTransaction</string> </value>
+            <value> <string>SaleInvoiceTransaction_createReversalSaleInvoiceTransaction</string> </value>
         </item>
       </dictionary>
     </pickle>

master/bt5/slapos_accounting/SkinTemplateItem/portal_skins/slapos_accounting/SaleInvoiceTransaction_createSlapOSReversalTransaction.py

-if context.getPaymentMode() == "payzen":
-  reversal_payment = context.SaleInvoiceTransaction_createReversalPayzenTransaction()
-elif context.getPaymentMode() == "wechat":
-  reversal_payment = context.SaleInvoiceTransaction_createReversalWechatTransaction()
-else:
-  message = context.Base_translateString("The payment mode is unsupported.")
-  return context.Base_redirect(keep_items={'portal_status_message': message})
-
-message = context.Base_translateString("Reversal Transaction created.")
-return reversal_payment.Base_redirect(keep_items={'portal_status_message': message})

master/bt5/slapos_accounting/SkinTemplateItem/portal_skins/slapos_accounting/SaleInvoiceTransaction_createSlapOSReversalTransaction.xml

-<?xml version="1.0"?>
-<ZopeData>
-  <record id="1" aka="AAAAAAAAAAE=">
-    <pickle>
-      <global name="PythonScript" module="Products.PythonScripts.PythonScript"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>Script_magic</string> </key>
-            <value> <int>3</int> </value>
-        </item>
-        <item>
-            <key> <string>_bind_names</string> </key>
-            <value>
-              <object>
-                <klass>
-                  <global name="NameAssignments" module="Shared.DC.Scripts.Bindings"/>
-                </klass>
-                <tuple/>
-                <state>
-                  <dictionary>
-                    <item>
-                        <key> <string>_asgns</string> </key>
-                        <value>
-                          <dictionary>
-                            <item>
-                                <key> <string>name_container</string> </key>
-                                <value> <string>container</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_context</string> </key>
-                                <value> <string>context</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_m_self</string> </key>
-                                <value> <string>script</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_subpath</string> </key>
-                                <value> <string>traverse_subpath</string> </value>
-                            </item>
-                          </dictionary>
-                        </value>
-                    </item>
-                  </dictionary>
-                </state>
-              </object>
-            </value>
-        </item>
-        <item>
-            <key> <string>_params</string> </key>
-            <value> <string>**kw</string> </value>
-        </item>
-        <item>
-            <key> <string>id</string> </key>
-            <value> <string>SaleInvoiceTransaction_createSlapOSReversalTransaction</string> </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-</ZopeData>

master/bt5/slapos_accounting/SkinTemplateItem/portal_skins/slapos_accounting/SaleInvoiceTransaction_viewCancelPaymentAndCreateReversalTransactionDialog.xml

@@ -35,7 +35,11 @@
         </item>
         <item>
             <key> <string>action</string> </key>
-            <value> <string>SaleInvoiceTransaction_createSlapOSReversalTransaction</string> </value>
+            <value> <string>SaleInvoiceTransaction_createReversalSaleInvoiceTransaction</string> </value>
+        </item>
+        <item>
+            <key> <string>action_title</string> </key>
+            <value> <string></string> </value>
         </item>
         <item>
             <key> <string>description</string> </key>

master/bt5/slapos_erp5/TestTemplateItem/portal_components/test.erp5.testSlapOSERP5DefaultScenario.py

@@ -505,13 +505,10 @@ class TestSlapOSDefaultCRMEscalation(DefaultScenarioMixin):
 
 
     # Manually cancel the users invoice
-    payment = self.portal.portal_catalog.getResultValue(
-      portal_type="Payment Transaction",
-      destination_section_uid=person.getUid(),
-      simulation_state="started")
-
-    invoice = payment.getCausalityValue(portal_type="Sale Invoice Transaction")
-    invoice.SaleInvoiceTransaction_createReversalPayzenTransaction()
+    invoice_list = person.Entity_getOutstandingAmountList()
+    self.assertEqual(len(invoice_list), 1)
+    sale_transaction_invoice = invoice_list[0].getObject()
+    sale_transaction_invoice.SaleInvoiceTransaction_createReversalSaleInvoiceTransaction(batch_mode=1)
 
     self.tic()
 

master/bt5/slapos_payzen/SkinTemplateItem/portal_skins/slapos_payzen/SaleInvoiceTransaction_createReversalPayzenTransaction.py

-""" Create a reversal transaction from current payzen transaction. """
-from zExceptions import Unauthorized
-if REQUEST is not None:
-  raise Unauthorized
-
-portal = context.getPortalObject()
-
-# Check that we are in state that we are waiting for user manual payment
-assert context.getPortalType() == 'Sale Invoice Transaction'
-assert context.getPaymentMode() == 'payzen'
-assert context.getSimulationState() == 'stopped'
-assert context.getTotalPrice() != 0
-assert context.getSpecialise() in ("sale_trade_condition_module/slapos_aggregated_trade_condition",
-                                   "sale_trade_condition_module/slapos_aggregated_subscription_trade_condition")
-
-
-# Dont create if the invoice is already paied
-assert not context.SaleInvoiceTransaction_isLettered()
-
-payment = portal.portal_catalog.getResultValue(
-  portal_type="Payment Transaction",
-  simulation_state="started",
-  default_causality_uid=context.getUid(),
-  default_payment_mode_uid=portal.portal_categories.payment_mode.payzen.getUid(),
-)
-if payment is not None and payment.PaymentTransaction_getPayzenId()[1] is None:
-  # The payment transaction will be cancelled by a proper alarm.
-  raise ValueError("Payment Transaction is waiting for External Payzen confirmation!")
-
-# Should be safe now to fix everything
-context.SaleInvoiceTransaction_resetPaymentMode()
-reversal_transaction = context.Base_createCloneDocument(batch_mode=1)
-
-reversal_transaction.edit(
-  title="Reversal Transaction for %s" % context.getTitle(),
-  causality_value=context,
-  description="Reversal Transaction for %s" % context.getTitle(),
-  specialise_value=portal.sale_trade_condition_module.slapos_manual_accounting_trade_condition,
-)
-
-for line in reversal_transaction.getMovementList():
-  line.edit(quantity=(-line.getQuantity()))
-
-reversal_transaction.confirm(comment="Automatic because of reversal creation")
-reversal_transaction.stop(comment="Automatic because of reversal creation")
-
-return reversal_transaction

master/bt5/slapos_payzen/SkinTemplateItem/portal_skins/slapos_payzen/SaleInvoiceTransaction_createReversalPayzenTransaction.xml

-<?xml version="1.0"?>
-<ZopeData>
-  <record id="1" aka="AAAAAAAAAAE=">
-    <pickle>
-      <global name="PythonScript" module="Products.PythonScripts.PythonScript"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>Script_magic</string> </key>
-            <value> <int>3</int> </value>
-        </item>
-        <item>
-            <key> <string>_bind_names</string> </key>
-            <value>
-              <object>
-                <klass>
-                  <global name="NameAssignments" module="Shared.DC.Scripts.Bindings"/>
-                </klass>
-                <tuple/>
-                <state>
-                  <dictionary>
-                    <item>
-                        <key> <string>_asgns</string> </key>
-                        <value>
-                          <dictionary>
-                            <item>
-                                <key> <string>name_container</string> </key>
-                                <value> <string>container</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_context</string> </key>
-                                <value> <string>context</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_m_self</string> </key>
-                                <value> <string>script</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_subpath</string> </key>
-                                <value> <string>traverse_subpath</string> </value>
-                            </item>
-                          </dictionary>
-                        </value>
-                    </item>
-                  </dictionary>
-                </state>
-              </object>
-            </value>
-        </item>
-        <item>
-            <key> <string>_params</string> </key>
-            <value> <string>REQUEST=None</string> </value>
-        </item>
-        <item>
-            <key> <string>id</string> </key>
-            <value> <string>SaleInvoiceTransaction_createReversalPayzenTransaction</string> </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-</ZopeData>

master/bt5/slapos_payzen/TestTemplateItem/portal_components/test.erp5.testSlapOSPayzenSkins.py

@@ -713,7 +713,7 @@ class TestSlapOSPayzenAccountingTransaction_getPaymentState(
   def test_AccountingTransaction_getPaymentState_reversed_payment(self):
     invoice =  self.createPayzenSaleInvoiceTransaction()
     self.tic()
-    reversal = invoice.SaleInvoiceTransaction_createReversalPayzenTransaction()
+    reversal = invoice.SaleInvoiceTransaction_createReversalSaleInvoiceTransaction()
     self.tic()
     self.assertEqual("Cancelled", invoice.AccountingTransaction_getPaymentState())
     self.assertEqual(0, invoice.getTotalPrice() + reversal.getTotalPrice())
@@ -850,146 +850,3 @@ return dict(vads_url_already_registered="%s/already_registered" % (payment_trans
 
     self.assertEqual("%s/already_registered" % payment.getRelativeUrl(),
                       redirect)
-
-class TestSlapOSPayzenSaleInvoiceTransaction_createReversalPayzenTransaction(
-                                                    SlapOSTestCaseMixinWithAbort):
-
-  def test_createReversalPayzenTransaction_REQUEST_disallowed(self):
-    self.assertRaises(
-      Unauthorized,
-      self.portal.SaleInvoiceTransaction_createReversalPayzenTransaction,
-      REQUEST={})
-
-  def test_createReversalPayzenTransaction_bad_portal_type(self):
-    self.assertRaises(
-      AssertionError,
-      self.portal.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_bad_payment_mode(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    invoice.edit(payment_mode="cash")
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_bad_state(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    self.portal.portal_workflow._jumpToStateFor(invoice, 'delivered')
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_zero_price(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    invoice.manage_delObjects(invoice.contentIds())
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_wrong_trade_condition(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    invoice.edit(specialise=None)
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_paid(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    line = invoice.contentValues(portal_type="Sale Invoice Transaction Line")[0]
-    line.edit(grouping_reference="azerty")
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_no_payment(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    # Do not reindex payment. portal_catalog will not find it.
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_no_payzen_payment(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.getCausalityRelatedValue()
-    payment.edit(payment_mode="cash")
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_no_payment_state(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.getCausalityRelatedValue()
-    self.portal.portal_workflow._jumpToStateFor(payment, 'cancelled')
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_registered_payment(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.getCausalityRelatedValue()
-    payment.PaymentTransaction_generatePayzenId()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalPayzenTransaction)
-
-  def test_createReversalPayzenTransaction_ok(self):
-    invoice = self.createPayzenSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.getCausalityRelatedValue()
-    reversale_invoice = invoice.\
-      SaleInvoiceTransaction_createReversalPayzenTransaction()
-
-    self.assertEqual(invoice.getPaymentMode(""), "")
-    self.assertEqual(payment.getPaymentMode(""), "")
-    self.assertEqual(payment.getSimulationState(), "cancelled")
-    self.assertEqual(reversale_invoice.getTitle(),
-                     "Reversal Transaction for %s" % invoice.getTitle())
-    self.assertEqual(reversale_invoice.getDescription(),
-                     "Reversal Transaction for %s" % invoice.getTitle())
-    self.assertEqual(reversale_invoice.getCausality(),
-                     invoice.getRelativeUrl())
-    self.assertEqual(reversale_invoice.getSimulationState(), "stopped")
-    self.assertEqual(invoice.getSimulationState(), "stopped")
-
-    invoice_line_id = invoice.contentValues(portal_type="Invoice Line")[0].getId()
-    transaction_line_id = invoice.contentValues(
-      portal_type="Sale Invoice Transaction Line")[0].getId()
-
-    self.assertEqual(invoice[invoice_line_id].getQuantity(),
-                     -reversale_invoice[invoice_line_id].getQuantity())
-    self.assertEqual(reversale_invoice[invoice_line_id].getQuantity(), 2)
-
-    self.assertEqual(invoice[transaction_line_id].getQuantity(),
-                     -reversale_invoice[transaction_line_id].getQuantity())
-    self.assertEqual(reversale_invoice[transaction_line_id].getQuantity(), 3)
-    self.assertEqual(len(invoice.getMovementList()), 2)
-
-    # Both invoice should have a grouping reference
-    self.assertNotEqual(invoice[transaction_line_id].getGroupingReference(""),
-                        "")
-    self.assertEqual(
-      invoice[transaction_line_id].getGroupingReference("1"),
-      reversale_invoice[transaction_line_id].getGroupingReference("2"))
-
-    # All references should be regenerated
-    self.assertNotEqual(invoice.getReference(""),
-                        reversale_invoice.getReference(""))
-    self.assertNotEqual(invoice.getSourceReference(""),
-                        reversale_invoice.getSourceReference(""))
-    self.assertNotEqual(invoice.getDestinationReference(""),
-                        reversale_invoice.getDestinationReference(""))
-
-    # Another trade condition
-    self.assertEqual(
-      reversale_invoice.getSpecialise(),
-      "sale_trade_condition_module/slapos_manual_accounting_trade_condition")
-    self.tic()
-

master/bt5/slapos_subscription_request/TestTemplateItem/portal_components/test.erp5.testSlapOSSubscriptionScenario.py

@@ -381,7 +381,7 @@ return dict(vads_url_already_registered="%s/already_registered" % (payment_trans
 
     self.logout()
     self.login()
-    sale_transaction_invoice.SaleInvoiceTransaction_createReversalPayzenTransaction()
+    sale_transaction_invoice.SaleInvoiceTransaction_createReversalSaleInvoiceTransaction(batch_mode=1)
 
   def checkSubscriptionRequestPayment(self, subscription_request, authAmount):
 

master/bt5/slapos_wechat/SkinTemplateItem/portal_skins/slapos_wechat/SaleInvoiceTransaction_getWechatPaymentRelatedValue.py

-portal = context.getPortalObject()
-return portal.portal_catalog.getResultValue(
-    portal_type="Payment Transaction",
-    simulation_state="started",
-    default_causality_uid=context.getUid(),
-    default_payment_mode_uid=portal.portal_categories.payment_mode.wechat.getUid(),
-  )

master/bt5/slapos_wechat/SkinTemplateItem/portal_skins/slapos_wechat/SaleInvoiceTransaction_getWechatPaymentRelatedValue.xml

-<?xml version="1.0"?>
-<ZopeData>
-  <record id="1" aka="AAAAAAAAAAE=">
-    <pickle>
-      <global name="PythonScript" module="Products.PythonScripts.PythonScript"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>Script_magic</string> </key>
-            <value> <int>3</int> </value>
-        </item>
-        <item>
-            <key> <string>_bind_names</string> </key>
-            <value>
-              <object>
-                <klass>
-                  <global name="NameAssignments" module="Shared.DC.Scripts.Bindings"/>
-                </klass>
-                <tuple/>
-                <state>
-                  <dictionary>
-                    <item>
-                        <key> <string>_asgns</string> </key>
-                        <value>
-                          <dictionary>
-                            <item>
-                                <key> <string>name_container</string> </key>
-                                <value> <string>container</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_context</string> </key>
-                                <value> <string>context</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_m_self</string> </key>
-                                <value> <string>script</string> </value>
-                            </item>
-                            <item>
-                                <key> <string>name_subpath</string> </key>
-                                <value> <string>traverse_subpath</string> </value>
-                            </item>
-                          </dictionary>
-                        </value>
-                    </item>
-                  </dictionary>
-                </state>
-              </object>
-            </value>
-        </item>
-        <item>
-            <key> <string>_params</string> </key>
-            <value> <string>**kw</string> </value>
-        </item>
-        <item>
-            <key> <string>id</string> </key>
-            <value> <string>SaleInvoiceTransaction_getWechatPaymentRelatedValue</string> </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
-</ZopeData>

master/bt5/slapos_wechat/TestTemplateItem/portal_components/test.erp5.testSlapOSWechatSkins.py

@@ -741,165 +741,3 @@ return dict(vads_url_already_registered="%s/already_registered" % (payment_trans
 
     self.assertEqual("%s/already_registered" % payment.getRelativeUrl(),
                       redirect)
-
-class TestSlapOSWechatSaleInvoiceTransaction_getWechatPaymentRelatedValue(
-                                                    SlapOSTestCaseMixinWithAbort):
-
-  def test_SaleInvoiceTransaction_getWechatPaymentRelatedValue(self):
-    invoice =  self.createWechatSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.SaleInvoiceTransaction_getWechatPaymentRelatedValue()
-    self.assertNotEqual(None, payment)
-    self.assertEqual(payment.getSimulationState(), "started")
-    self.assertEqual(payment.getCausalityValue(), invoice)
-    self.assertEqual(payment.getPaymentModeUid(),
-      self.portal.portal_categories.payment_mode.wechat.getUid())
-
-    payment.setStartDate(DateTime())
-    payment.stop()
-    payment.immediateReindexObject()
-    payment = invoice.SaleInvoiceTransaction_getWechatPaymentRelatedValue()
-    self.assertEqual(None, payment)
-
-class TestSlapOSWechatSaleInvoiceTransaction_createReversalWechatTransaction(
-                                                    SlapOSTestCaseMixinWithAbort):
-
-  def test_createReversalWechatTransaction_REQUEST_disallowed(self):
-    self.assertRaises(
-      Unauthorized,
-      self.portal.SaleInvoiceTransaction_createReversalWechatTransaction,
-      REQUEST={})
-
-  def test_createReversalWechatTransaction_bad_portal_type(self):
-    self.assertRaises(
-      AssertionError,
-      self.portal.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_bad_payment_mode(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    invoice.edit(payment_mode="cash")
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_bad_state(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    self.portal.portal_workflow._jumpToStateFor(invoice, 'delivered')
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_zero_price(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    invoice.manage_delObjects(invoice.contentIds())
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_wrong_trade_condition(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    invoice.edit(specialise=None)
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_paid(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    line = invoice.contentValues(portal_type="Sale Invoice Transaction Line")[0]
-    line.edit(grouping_reference="azerty")
-    self.tic()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_no_payment(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    # Do not reindex payment. portal_catalog will not find it.
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_no_wechat_payment(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.getCausalityRelatedValue()
-    payment.edit(payment_mode="cash")
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_no_payment_state(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.getCausalityRelatedValue()
-    self.portal.portal_workflow._jumpToStateFor(payment, 'cancelled')
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_registered_payment(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.getCausalityRelatedValue()
-    payment.PaymentTransaction_generateWechatId()
-    self.assertRaises(
-      AssertionError,
-      invoice.SaleInvoiceTransaction_createReversalWechatTransaction)
-
-  def test_createReversalWechatTransaction_ok(self):
-    invoice = self.createWechatSaleInvoiceTransaction()
-    self.tic()
-    payment = invoice.getCausalityRelatedValue()
-    reversale_invoice = invoice.\
-      SaleInvoiceTransaction_createReversalWechatTransaction()
-
-    self.assertEqual(invoice.getPaymentMode(""), "")
-    self.assertEqual(payment.getPaymentMode(""), "")
-    self.assertEqual(payment.getSimulationState(), "cancelled")
-    self.assertEqual(reversale_invoice.getTitle(),
-                     "Reversal Transaction for %s" % invoice.getTitle())
-    self.assertEqual(reversale_invoice.getDescription(),
-                     "Reversal Transaction for %s" % invoice.getTitle())
-    self.assertEqual(reversale_invoice.getCausality(),
-                     invoice.getRelativeUrl())
-    self.assertEqual(reversale_invoice.getSimulationState(), "stopped")
-    self.assertEqual(invoice.getSimulationState(), "stopped")
-
-    invoice_line_id = invoice.contentValues(portal_type="Invoice Line")[0].getId()
-    transaction_line_id = invoice.contentValues(
-      portal_type="Sale Invoice Transaction Line")[0].getId()
-
-    self.assertEqual(invoice[invoice_line_id].getQuantity(),
-                     -reversale_invoice[invoice_line_id].getQuantity())
-    self.assertEqual(reversale_invoice[invoice_line_id].getQuantity(), 2)
-
-    self.assertEqual(invoice[transaction_line_id].getQuantity(),
-                     -reversale_invoice[transaction_line_id].getQuantity())
-    self.assertEqual(reversale_invoice[transaction_line_id].getQuantity(), 3)
-    self.assertEqual(len(invoice.getMovementList()), 2)
-
-    # Both invoice should have a grouping reference
-    self.assertNotEqual(invoice[transaction_line_id].getGroupingReference(""),
-                        "")
-    self.assertEqual(
-      invoice[transaction_line_id].getGroupingReference("1"),
-      reversale_invoice[transaction_line_id].getGroupingReference("2"))
-
-    # All references should be regenerated
-    self.assertNotEqual(invoice.getReference(""),
-                        reversale_invoice.getReference(""))
-    self.assertNotEqual(invoice.getSourceReference(""),
-                        reversale_invoice.getSourceReference(""))
-    self.assertNotEqual(invoice.getDestinationReference(""),
-                        reversale_invoice.getDestinationReference(""))
-
-    # Another trade condition
-    self.assertEqual(
-      reversale_invoice.getSpecialise(),
-      "sale_trade_condition_module/slapos_manual_accounting_trade_condition")
-    self.tic()
-