From d5aabc6605a9ee2cc9ffff4d425442b10366dcce Mon Sep 17 00:00:00 2001 From: Stefan Rijnhart Date: Tue, 2 Feb 2021 09:42:39 +0100 Subject: [PATCH] [FIX] Fix same eror in camt54 module --- .../models/account_statement_import.py | 3 +-- account_statement_import_camt54/models/parser.py | 12 ++++++------ .../tests/test_get_partner_ref.py | 6 +++--- .../tests/test_statement.py | 9 +-------- 4 files changed, 11 insertions(+), 19 deletions(-) diff --git a/account_statement_import_camt54/models/account_statement_import.py b/account_statement_import_camt54/models/account_statement_import.py index 6dc15ec3..40b6dbf5 100644 --- a/account_statement_import_camt54/models/account_statement_import.py +++ b/account_statement_import_camt54/models/account_statement_import.py @@ -25,11 +25,10 @@ class AccountStatementImport(models.TransientModel): amount = -amount statement.line_ids.create( { - "name": statement.name, "amount": amount, "statement_id": statement.id, "date": statement.date, - "payment_ref": "/", + "payment_ref": statement.name, } ) statement.balance_end_real = statement.balance_start diff --git a/account_statement_import_camt54/models/parser.py b/account_statement_import_camt54/models/parser.py index 7c6d4190..b12b1caa 100644 --- a/account_statement_import_camt54/models/parser.py +++ b/account_statement_import_camt54/models/parser.py @@ -58,7 +58,7 @@ class CamtParser(models.AbstractModel): "./ns:RmtInf/ns:Strd/ns:CdtrRefInf/ns:Ref", namespaces={"ns": ns} ) if len(isr_number): - transaction["name"] = isr_number[0].text + transaction["payment_ref"] = isr_number[0].text partner_ref = self._get_partner_ref(isr_number[0].text) if partner_ref: transaction["partner_ref"] = partner_ref @@ -68,11 +68,11 @@ class CamtParser(models.AbstractModel): "./ns:AddtlNtryInf", "/ns:Refs/ns:InstrId", ] - name = transaction["name"] + payment_ref = transaction["payment_ref"] for xpath_expr in xpath_exprs: found_node = node.xpath(xpath_expr, namespaces={"ns": ns}) if found_node: - name = found_node[0].text + payment_ref = found_node[0].text break trans_id_node = ( node.getparent() @@ -80,9 +80,9 @@ class CamtParser(models.AbstractModel): .xpath("./ns:AcctSvcrRef", namespaces={"ns": ns}) ) if trans_id_node: - name = "{} ({})".format(name, trans_id_node[0].text) - if name: - transaction["name"] = name + payment_ref = "{} ({})".format(payment_ref, trans_id_node[0].text) + if payment_ref: + transaction["payment_ref"] = payment_ref # End add esr to the label. # add transaction id to ref diff --git a/account_statement_import_camt54/tests/test_get_partner_ref.py b/account_statement_import_camt54/tests/test_get_partner_ref.py index 01170a18..f42b039e 100644 --- a/account_statement_import_camt54/tests/test_get_partner_ref.py +++ b/account_statement_import_camt54/tests/test_get_partner_ref.py @@ -27,21 +27,21 @@ class TestGetPartnerRef(common.TransactionCase): self.ICP.set_param("isr_partner_ref", "12") ref = "11 11111 11112 34567 11111 11111".replace(" ", "") partner_ref = self.Parser._get_partner_ref(ref) - self.assertEquals(partner_ref, "234567") + self.assertEqual(partner_ref, "234567") def test_ICP_full(self): """Test full format of partner ref definition""" self.ICP.set_param("isr_partner_ref", "12,6") ref = "11 11111 11112 34567 11111 11111".replace(" ", "") partner_ref = self.Parser._get_partner_ref(ref) - self.assertEquals(partner_ref, "234567") + self.assertEqual(partner_ref, "234567") def test_zero_stripped(self): """Test full format of partner ref definition""" self.ICP.set_param("isr_partner_ref", "12,6") ref = "11 11111 11110 00560 11111 11111".replace(" ", "") partner_ref = self.Parser._get_partner_ref(ref) - self.assertEquals(partner_ref, "560") + self.assertEqual(partner_ref, "560") def test_bad_ICP(self): """Test ir config parameter validation""" diff --git a/account_statement_import_camt54/tests/test_statement.py b/account_statement_import_camt54/tests/test_statement.py index b7a16fd3..aa59318f 100644 --- a/account_statement_import_camt54/tests/test_statement.py +++ b/account_statement_import_camt54/tests/test_statement.py @@ -3,8 +3,6 @@ # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). import base64 -import mock - from odoo.modules.module import get_module_resource from odoo.tests.common import SavepointCase @@ -39,12 +37,7 @@ class TestGenerateBankStatement(SavepointCase): } ) - @mock.patch( - "odoo.addons.account.models.sequence_mixin." - "SequenceMixin._constrains_date_sequence", - side_effect=False, - ) - def _load_statement(self, constraint): + def _load_statement(self): testfile = get_module_resource( "account_statement_import_camt", "test_files", "test-camt053" )