[FIX] account_reconcile_payment_order: Fix tests

Create journal instead of searching
Change hardcoded value for amount of payment statements, after we cleared the taxes of the invoice lines in https://github.com/OCA/bank-payment/pull/798

TT28962
This commit is contained in:
João Marques
2021-03-30 11:25:33 +01:00
committed by CarlosRoca13
parent 5077757a48
commit ed53232d21
2 changed files with 14 additions and 12 deletions

View File

@@ -4,7 +4,7 @@
{ {
"name": "Reconcile payment orders", "name": "Reconcile payment orders",
"version": "13.0.1.0.0", "version": "13.0.1.0.1",
"author": "Therp BV," "Tecnativa," "Odoo Community Association (OCA)", "author": "Therp BV," "Tecnativa," "Odoo Community Association (OCA)",
"license": "AGPL-3", "license": "AGPL-3",
"website": "https://github.com/OCA/account-reconcile", "website": "https://github.com/OCA/account-reconcile",

View File

@@ -12,14 +12,8 @@ class TestAccountReconcilePaymentOrder(TestPaymentOrderInboundBase):
def setUpClass(cls): def setUpClass(cls):
super().setUpClass() super().setUpClass()
cls.widget_obj = cls.env["account.reconciliation.widget"] cls.widget_obj = cls.env["account.reconciliation.widget"]
cls.bank_journal = cls.env["account.journal"].search( cls.bank_journal = cls.env["account.journal"].create(
[ {"name": "Test bank journal", "type": "bank"}
("type", "=", "bank"),
"|",
("company_id", "=", cls.env.company.id),
("company_id", "=", False),
],
limit=1,
) )
# Create second invoice for being sure it handles the payment order # Create second invoice for being sure it handles the payment order
cls.invoice2 = cls._create_customer_invoice(cls) cls.invoice2 = cls._create_customer_invoice(cls)
@@ -32,7 +26,7 @@ class TestAccountReconcilePaymentOrder(TestPaymentOrderInboundBase):
cls.invoice2.action_post() cls.invoice2.action_post()
# Add to payment order using the wizard # Add to payment order using the wizard
cls.env["account.invoice.payment.line.multi"].with_context( cls.env["account.invoice.payment.line.multi"].with_context(
active_model="account.move", active_ids=cls.invoice2.ids, active_model="account.move", active_ids=cls.invoice2.ids
).create({}).run() ).create({}).run()
# Prepare statement # Prepare statement
cls.statement = cls.env["account.bank.statement"].create( cls.statement = cls.env["account.bank.statement"].create(
@@ -41,7 +35,15 @@ class TestAccountReconcilePaymentOrder(TestPaymentOrderInboundBase):
"date": "2019-01-01", "date": "2019-01-01",
"journal_id": cls.bank_journal.id, "journal_id": cls.bank_journal.id,
"line_ids": [ "line_ids": [
(0, 0, {"date": "2019-01-01", "name": "Test line", "amount": 230}), (
0,
0,
{
"date": "2019-01-01",
"name": "Test line",
"amount": 200, # 100 * 2
},
),
], ],
} }
) )
@@ -56,7 +58,7 @@ class TestAccountReconcilePaymentOrder(TestPaymentOrderInboundBase):
self.inbound_order.open2generated() self.inbound_order.open2generated()
self.inbound_order.generated2uploaded() self.inbound_order.generated2uploaded()
# Check widget result # Check widget result
res = self.widget_obj.get_bank_statement_line_data(self.statement.line_ids.ids,) res = self.widget_obj.get_bank_statement_line_data(self.statement.line_ids.ids)
self.assertEqual(len(res["lines"][0]["reconciliation_proposition"]), 2) self.assertEqual(len(res["lines"][0]["reconciliation_proposition"]), 2)
def test_reconcile_payment_order_transfer_account(self): def test_reconcile_payment_order_transfer_account(self):