# -*- coding: utf-8 -*- from odoo import Command from odoo.addons.account.tests.common import AccountTestInvoicingCommon from odoo.tests import tagged, new_test_user from odoo.tests.common import Form @tagged('post_install', '-at_install') class TestAccountPayment(AccountTestInvoicingCommon): @classmethod def setUpClass(cls, chart_template_ref=None): super().setUpClass(chart_template_ref=chart_template_ref) company = cls.company_data['default_journal_bank'].company_id cls.payment_debit_account_id = cls.copy_account(company.account_journal_payment_debit_account_id) cls.payment_credit_account_id = cls.copy_account(company.account_journal_payment_credit_account_id) cls.bank_journal_1 = cls.company_data['default_journal_bank'] cls.bank_journal_2 = cls.company_data['default_journal_bank'].copy() cls.partner_bank_account1 = cls.env['res.partner.bank'].create({ 'acc_number': "0123456789", 'partner_id': cls.partner_a.id, 'acc_type': 'bank', }) cls.partner_bank_account2 = cls.env['res.partner.bank'].create({ 'acc_number': "9876543210", 'partner_id': cls.partner_a.id, 'acc_type': 'bank', }) cls.comp_bank_account1 = cls.env['res.partner.bank'].create({ 'acc_number': "985632147", 'partner_id': cls.env.company.partner_id.id, 'acc_type': 'bank', }) cls.comp_bank_account2 = cls.env['res.partner.bank'].create({ 'acc_number': "741258963", 'partner_id': cls.env.company.partner_id.id, 'acc_type': 'bank', }) company.write({ 'account_journal_payment_debit_account_id': cls.payment_debit_account_id.id, 'account_journal_payment_credit_account_id': cls.payment_credit_account_id.id }) def test_payment_move_sync_create_write(self): copy_receivable = self.copy_account(self.company_data['default_account_receivable']) payment = self.env['account.payment'].create({ 'amount': 50.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'destination_account_id': copy_receivable.id, }) expected_payment_values = { 'amount': 50.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'payment_reference': False, 'is_reconciled': False, 'currency_id': self.company_data['currency'].id, 'partner_id': False, 'destination_account_id': copy_receivable.id, 'payment_method_line_id': self.inbound_payment_method_line.id, } expected_move_values = { 'currency_id': self.company_data['currency'].id, 'partner_id': False, } expected_liquidity_line = { 'debit': 50.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.payment_debit_account_id.id, } expected_counterpart_line = { 'debit': 0.0, 'credit': 50.0, 'amount_currency': -50.0, 'currency_id': self.company_data['currency'].id, 'account_id': copy_receivable.id, } self.assertRecordValues(payment, [expected_payment_values]) self.assertRecordValues(payment.move_id, [expected_move_values]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ expected_counterpart_line, expected_liquidity_line, ]) # ==== Check editing the account.payment ==== payment.write({ 'partner_type': 'supplier', 'currency_id': self.currency_data['currency'].id, 'partner_id': self.partner_a.id, }) self.assertRecordValues(payment, [{ **expected_payment_values, 'partner_type': 'supplier', 'destination_account_id': self.partner_a.property_account_payable_id.id, 'currency_id': self.currency_data['currency'].id, 'partner_id': self.partner_a.id, }]) self.assertRecordValues(payment.move_id, [{ **expected_move_values, 'currency_id': self.currency_data['currency'].id, 'partner_id': self.partner_a.id, }]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { **expected_counterpart_line, 'debit': 0.0, 'credit': 25.0, 'amount_currency': -50.0, 'currency_id': self.currency_data['currency'].id, 'account_id': self.partner_a.property_account_payable_id.id, }, { **expected_liquidity_line, 'debit': 25.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.currency_data['currency'].id, }, ]) # ==== Check editing the account.move.line ==== liquidity_lines, counterpart_lines, writeoff_lines = payment._seek_for_lines() payment.move_id.write({ 'line_ids': [ (1, counterpart_lines.id, { 'debit': 0.0, 'credit': 75.0, 'amount_currency': -75.0, 'currency_id': self.company_data['currency'].id, 'account_id': copy_receivable.id, 'partner_id': self.partner_b.id, }), (1, liquidity_lines.id, { 'debit': 100.0, 'credit': 0.0, 'amount_currency': 100.0, 'currency_id': self.company_data['currency'].id, 'partner_id': self.partner_b.id, }), # Additional write-off: (0, 0, { 'debit': 0.0, 'credit': 25.0, 'amount_currency': -25.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['default_account_revenue'].id, 'partner_id': self.partner_b.id, }), ] }) self.assertRecordValues(payment, [{ **expected_payment_values, 'amount': 100.0, 'partner_id': self.partner_b.id, }]) self.assertRecordValues(payment.move_id, [{ **expected_move_values, 'partner_id': self.partner_b.id, }]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { **expected_counterpart_line, 'debit': 0.0, 'credit': 75.0, 'amount_currency': -75.0, 'partner_id': self.partner_b.id, }, { 'debit': 0.0, 'credit': 25.0, 'amount_currency': -25.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['default_account_revenue'].id, 'partner_id': self.partner_b.id, }, { **expected_liquidity_line, 'debit': 100.0, 'credit': 0.0, 'amount_currency': 100.0, 'account_id': self.payment_debit_account_id.id, 'partner_id': self.partner_b.id, }, ]) def test_payment_move_sync_update_journal_custom_accounts(self): """The objective is to edit the journal of a payment in order to check if the accounts are updated.""" company = self.company_data['company'] # Create two different inbound accounts outstanding_payment_A = company.account_journal_payment_debit_account_id outstanding_payment_B = company.account_journal_payment_debit_account_id.copy() # Create two different journals with a different account journal_A = self.company_data['default_journal_bank'] journal_A.inbound_payment_method_line_ids.payment_account_id = outstanding_payment_A journal_B = self.company_data['default_journal_bank'].copy() journal_B.inbound_payment_method_line_ids.payment_account_id = outstanding_payment_B # Fill the form payment pay_form = Form(self.env['account.payment'].with_context(default_journal_id=self.company_data['default_journal_bank'].id)) pay_form.amount = 50.0 pay_form.payment_type = 'inbound' pay_form.partner_id = self.partner_a pay_form.journal_id = journal_A # Save the form (to create move and move line) payment = pay_form.save() # Check the payment self.assertRecordValues(payment, [{ 'amount': 50.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'payment_reference': False, 'is_reconciled': False, 'currency_id': self.company_data['currency'].id, 'partner_id': self.partner_a.id, 'journal_id': journal_A.id }]) self.assertRecordValues(payment.move_id, [{ 'currency_id': self.company_data['currency'].id, 'partner_id': self.partner_a.id, 'journal_id': journal_A.id, }]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { 'debit': 0.0, 'credit': 50.0, 'amount_currency': -50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['default_account_receivable'].id, }, { 'debit': 50.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.company_data['currency'].id, 'account_id': outstanding_payment_A.id, }, ]) # Change the journal on the form pay_form.journal_id = journal_B # Save the form (to write move and move line) payment = pay_form.save() # Check the payment self.assertRecordValues(payment, [{ 'amount': 50.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'payment_reference': False, 'is_reconciled': False, 'currency_id': self.company_data['currency'].id, 'partner_id': self.partner_a.id, 'journal_id': journal_B.id }]) self.assertRecordValues(payment.move_id, [{ 'currency_id': self.company_data['currency'].id, 'partner_id': self.partner_a.id, 'journal_id': journal_B.id, }]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { 'debit': 0.0, 'credit': 50.0, 'amount_currency': -50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['default_account_receivable'].id, }, { 'debit': 50.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.company_data['currency'].id, 'account_id': outstanding_payment_B.id, }, ]) def test_payment_move_sync_onchange(self): pay_form = Form(self.env['account.payment'].with_context( default_journal_id=self.company_data['default_journal_bank'].id, # The `partner_type` is set through the window action context in the web client # the field is otherwise invisible in the form. default_partner_type='customer', )) pay_form.amount = 50.0 pay_form.payment_type = 'inbound' pay_form.partner_id = self.partner_a payment = pay_form.save() expected_payment_values = { 'amount': 50.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'payment_reference': False, 'is_reconciled': False, 'currency_id': self.company_data['currency'].id, 'partner_id': self.partner_a.id, 'destination_account_id': self.partner_a.property_account_receivable_id.id, 'payment_method_line_id': self.inbound_payment_method_line.id, } expected_move_values = { 'currency_id': self.company_data['currency'].id, 'partner_id': self.partner_a.id, } expected_liquidity_line = { 'debit': 50.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.payment_debit_account_id.id, } expected_counterpart_line = { 'debit': 0.0, 'credit': 50.0, 'amount_currency': -50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['default_account_receivable'].id, } self.assertRecordValues(payment, [expected_payment_values]) self.assertRecordValues(payment.move_id, [expected_move_values]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ expected_counterpart_line, expected_liquidity_line, ]) # ==== Check editing the account.payment ==== # `partner_type` on payment is always invisible. It's supposed to be set through a context `default_` key # In this case the goal of the test is to take an existing customer payment and change it to a supplier payment, # which is not supposed to be possible through the web interface. # So, change the payment partner_type beforehand rather than in the form view. payment.partner_type = 'supplier' pay_form = Form(payment) pay_form.currency_id = self.currency_data['currency'] pay_form.partner_id = self.partner_a payment = pay_form.save() self.assertRecordValues(payment, [{ **expected_payment_values, 'partner_type': 'supplier', 'destination_account_id': self.partner_a.property_account_payable_id.id, 'currency_id': self.currency_data['currency'].id, 'partner_id': self.partner_a.id, }]) self.assertRecordValues(payment.move_id, [{ **expected_move_values, 'currency_id': self.currency_data['currency'].id, 'partner_id': self.partner_a.id, }]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { **expected_counterpart_line, 'debit': 0.0, 'credit': 25.0, 'amount_currency': -50.0, 'currency_id': self.currency_data['currency'].id, 'account_id': self.partner_a.property_account_payable_id.id, }, { **expected_liquidity_line, 'debit': 25.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.currency_data['currency'].id, }, ]) # ==== Check editing the account.move.line ==== move_form = Form(payment.move_id) with move_form.line_ids.edit(0) as line_form: line_form.currency_id = self.company_data['currency'] line_form.amount_currency = 100.0 line_form.partner_id = self.partner_b with move_form.line_ids.edit(1) as line_form: line_form.currency_id = self.company_data['currency'] line_form.amount_currency = -75.0 line_form.partner_id = self.partner_b line_form.account_id = self.company_data['default_account_receivable'] with move_form.line_ids.new() as line_form: line_form.currency_id = self.company_data['currency'] line_form.amount_currency = -25.0 line_form.partner_id = self.partner_b line_form.account_id = self.company_data['default_account_revenue'] move_form.save() self.assertRecordValues(payment, [{ **expected_payment_values, 'amount': 100.0, 'partner_id': self.partner_b.id, }]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { **expected_counterpart_line, 'debit': 0.0, 'credit': 75.0, 'amount_currency': -75.0, 'partner_id': self.partner_b.id, }, { 'debit': 0.0, 'credit': 25.0, 'amount_currency': -25.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['default_account_revenue'].id, 'partner_id': self.partner_b.id, }, { **expected_liquidity_line, 'debit': 100.0, 'credit': 0.0, 'amount_currency': 100.0, 'account_id': self.payment_debit_account_id.id, 'partner_id': self.partner_b.id, }, ]) def test_inbound_payment_sync_writeoff_debit_sign(self): payment = self.env['account.payment'].create({ 'amount': 100.0, 'payment_type': 'inbound', 'partner_type': 'customer', }) # ==== Edit the account.move.line ==== liquidity_lines, counterpart_lines, dummy = payment._seek_for_lines() payment.move_id.write({ 'line_ids': [ (1, liquidity_lines.id, {'debit': 100.0}), (1, counterpart_lines.id, {'credit': 125.0}), (0, 0, {'debit': 25.0, 'account_id': self.company_data['default_account_revenue'].id}), ], }) self.assertRecordValues(payment, [{ 'payment_type': 'inbound', 'partner_type': 'customer', 'amount': 100.0, }]) # ==== Edit the account.payment amount ==== payment.write({ 'partner_type': 'supplier', 'amount': 100.1, 'destination_account_id': self.company_data['default_account_payable'].id, }) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { 'debit': 0.0, 'credit': 125.1, 'account_id': self.company_data['default_account_payable'].id, }, { 'debit': 25.0, 'credit': 0.0, 'account_id': self.company_data['default_account_revenue'].id, }, { 'debit': 100.1, 'credit': 0.0, 'account_id': self.payment_debit_account_id.id, }, ]) def test_inbound_payment_sync_writeoff_credit_sign(self): payment = self.env['account.payment'].create({ 'amount': 100.0, 'payment_type': 'inbound', 'partner_type': 'customer', }) # ==== Edit the account.move.line ==== liquidity_lines, counterpart_lines, dummy = payment._seek_for_lines() payment.move_id.write({ 'line_ids': [ (1, liquidity_lines.id, {'debit': 100.0}), (1, counterpart_lines.id, {'credit': 75.0}), (0, 0, {'credit': 25.0, 'account_id': self.company_data['default_account_revenue'].id}), ], }) self.assertRecordValues(payment, [{ 'payment_type': 'inbound', 'partner_type': 'customer', 'amount': 100.0, }]) # ==== Edit the account.payment amount ==== payment.write({ 'partner_type': 'supplier', 'amount': 100.1, 'destination_account_id': self.company_data['default_account_payable'].id, }) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { 'debit': 0.0, 'credit': 75.1, 'account_id': self.company_data['default_account_payable'].id, }, { 'debit': 0.0, 'credit': 25.0, 'account_id': self.company_data['default_account_revenue'].id, }, { 'debit': 100.1, 'credit': 0.0, 'account_id': self.payment_debit_account_id.id, }, ]) def test_outbound_payment_sync_writeoff_debit_sign(self): payment = self.env['account.payment'].create({ 'amount': 100.0, 'payment_type': 'outbound', 'partner_type': 'supplier', }) # ==== Edit the account.move.line ==== liquidity_lines, counterpart_lines, dummy = payment._seek_for_lines() payment.move_id.write({ 'line_ids': [ (1, liquidity_lines.id, {'credit': 100.0}), (1, counterpart_lines.id, {'debit': 75.0}), (0, 0, {'debit': 25.0, 'account_id': self.company_data['default_account_revenue'].id}), ], }) self.assertRecordValues(payment, [{ 'payment_type': 'outbound', 'partner_type': 'supplier', 'amount': 100.0, }]) # ==== Edit the account.payment amount ==== payment.write({ 'partner_type': 'customer', 'amount': 100.1, 'destination_account_id': self.company_data['default_account_receivable'].id, }) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { 'debit': 0.0, 'credit': 100.1, 'account_id': self.payment_credit_account_id.id, }, { 'debit': 25.0, 'credit': 0.0, 'account_id': self.company_data['default_account_revenue'].id, }, { 'debit': 75.1, 'credit': 0.0, 'account_id': self.company_data['default_account_receivable'].id, }, ]) def test_outbound_payment_sync_writeoff_credit_sign(self): payment = self.env['account.payment'].create({ 'amount': 100.0, 'payment_type': 'outbound', 'partner_type': 'supplier', }) # ==== Edit the account.move.line ==== liquidity_lines, counterpart_lines, dummy = payment._seek_for_lines() payment.move_id.write({ 'line_ids': [ (1, liquidity_lines.id, {'credit': 100.0}), (1, counterpart_lines.id, {'debit': 125.0}), (0, 0, {'credit': 25.0, 'account_id': self.company_data['default_account_revenue'].id}), ], }) self.assertRecordValues(payment, [{ 'payment_type': 'outbound', 'partner_type': 'supplier', 'amount': 100.0, }]) # ==== Edit the account.payment amount ==== payment.write({ 'partner_type': 'customer', 'amount': 100.1, 'destination_account_id': self.company_data['default_account_receivable'].id, }) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { 'debit': 0.0, 'credit': 100.1, 'account_id': self.payment_credit_account_id.id, }, { 'debit': 0.0, 'credit': 25.0, 'account_id': self.company_data['default_account_revenue'].id, }, { 'debit': 125.1, 'credit': 0.0, 'account_id': self.company_data['default_account_receivable'].id, }, ]) def test_internal_transfer(self): copy_receivable = self.copy_account(self.company_data['default_account_receivable']) payment = self.env['account.payment'].create({ 'amount': 50.0, 'is_internal_transfer': True, 'destination_journal_id': self.company_data['default_journal_cash'].id, }) expected_payment_values = { 'amount': 50.0, 'payment_type': 'inbound', 'currency_id': self.company_data['currency'].id, 'partner_id': self.company_data['company'].partner_id.id, 'destination_account_id': self.company_data['company'].transfer_account_id.id, } expected_move_values = { 'currency_id': self.company_data['currency'].id, 'partner_id': self.company_data['company'].partner_id.id, } expected_liquidity_line = { 'debit': 50.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.payment_debit_account_id.id, } expected_counterpart_line = { 'debit': 0.0, 'credit': 50.0, 'amount_currency': -50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['company'].transfer_account_id.id, } self.assertRecordValues(payment, [expected_payment_values]) self.assertRecordValues(payment.move_id, [expected_move_values]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ expected_counterpart_line, expected_liquidity_line, ]) # ==== Check editing the account.payment ==== payment.write({ 'partner_type': 'customer', 'partner_id': self.partner_a.id, 'destination_account_id': copy_receivable.id, }) self.assertRecordValues(payment, [{ **expected_payment_values, 'partner_type': 'customer', 'destination_account_id': copy_receivable.id, 'partner_id': self.partner_a.id, 'is_internal_transfer': False, }]) self.assertRecordValues(payment.move_id, [{ **expected_move_values, 'partner_id': self.partner_a.id, }]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { **expected_counterpart_line, 'account_id': copy_receivable.id, }, expected_liquidity_line, ]) # ==== Check editing the account.move.line ==== liquidity_lines, counterpart_lines, writeoff_lines = payment._seek_for_lines() payment.move_id.write({ 'line_ids': [ (1, counterpart_lines.id, { 'account_id': self.company_data['company'].transfer_account_id.id, 'partner_id': self.company_data['company'].partner_id.id, }), (1, liquidity_lines.id, { 'partner_id': self.company_data['company'].partner_id.id, }), ] }) self.assertRecordValues(payment, [expected_payment_values]) self.assertRecordValues(payment.move_id, [expected_move_values]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ expected_counterpart_line, expected_liquidity_line, ]) # ==== Check creation of paired internal transfer payment ==== payment = self.env['account.payment'].create({ 'amount': 50.0, 'is_internal_transfer': True, 'payment_type': 'inbound', 'destination_journal_id': self.company_data['default_journal_cash'].id, }) payment.action_post() paired_payment = self.env['account.payment'].search([('payment_type', '=', 'outbound')]) expected_payment_values = { 'amount': 50.0, 'payment_type': 'outbound', 'currency_id': self.company_data['currency'].id, 'partner_id': self.company_data['company'].partner_id.id, 'destination_account_id': self.company_data['company'].transfer_account_id.id, } expected_move_values = { 'currency_id': self.company_data['currency'].id, 'partner_id': self.company_data['company'].partner_id.id, } expected_liquidity_line = { 'debit': 0.0, 'credit': 50.0, 'amount_currency': -50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['default_journal_cash'].company_id.account_journal_payment_credit_account_id.id, } expected_counterpart_line = { 'debit': 50.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.company_data['currency'].id, 'account_id': self.company_data['company'].transfer_account_id.id, } self.assertRecordValues(paired_payment, [expected_payment_values]) self.assertRecordValues(paired_payment.move_id, [expected_move_values]) self.assertRecordValues(paired_payment.line_ids.sorted('balance'), [ expected_liquidity_line, expected_counterpart_line, ]) def test_compute_currency_id(self): ''' When creating a new account.payment without specifying a currency, the default currency should be the one set on the journal. ''' self.company_data['default_journal_bank'].write({ 'currency_id': self.currency_data['currency'].id, }) payment = self.env['account.payment'].create({ 'amount': 50.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'partner_id': self.partner_a.id, 'journal_id': self.company_data['default_journal_bank'].id, }) self.assertRecordValues(payment, [{ 'currency_id': self.currency_data['currency'].id, }]) self.assertRecordValues(payment.move_id, [{ 'currency_id': self.currency_data['currency'].id, }]) self.assertRecordValues(payment.line_ids.sorted('balance'), [ { 'debit': 0.0, 'credit': 25.0, 'amount_currency': -50.0, 'currency_id': self.currency_data['currency'].id, }, { 'debit': 25.0, 'credit': 0.0, 'amount_currency': 50.0, 'currency_id': self.currency_data['currency'].id, }, ]) def test_reconciliation_payment_states(self): payment = self.env['account.payment'].create({ 'amount': 50.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'destination_account_id': self.company_data['default_account_receivable'].id, }) liquidity_lines, counterpart_lines, writeoff_lines = payment._seek_for_lines() self.assertRecordValues(payment, [{ 'is_reconciled': False, 'is_matched': False, }]) invoice = self.env['account.move'].create({ 'move_type': 'out_invoice', 'partner_id': self.partner_a.id, 'invoice_line_ids': [(0, 0, { 'name': '50 to pay', 'price_unit': 50.0, 'quantity': 1, 'account_id': self.company_data['default_account_revenue'].id, })], }) payment.action_post() invoice.action_post() (counterpart_lines + invoice.line_ids.filtered(lambda line: line.account_type == 'asset_receivable'))\ .reconcile() self.assertRecordValues(payment, [{ 'is_reconciled': True, 'is_matched': False, }]) statement_line = self.env['account.bank.statement.line'].create({ 'payment_ref': '50 to pay', 'journal_id': self.company_data['default_journal_bank'].id, 'partner_id': self.partner_a.id, 'amount': 50.0, }) # Reconcile without the bank reconciliation widget since the widget is in enterprise. _st_liquidity_lines, st_suspense_lines, _st_other_lines = statement_line\ .with_context(skip_account_move_synchronization=True)\ ._seek_for_lines() st_suspense_lines.account_id = liquidity_lines.account_id (st_suspense_lines + liquidity_lines).reconcile() self.assertRecordValues(payment, [{ 'is_reconciled': True, 'is_matched': True, }]) def test_reconciliation_payment_states_reverse_payment_move(self): invoice = self.env['account.move'].create({ 'move_type': 'out_invoice', 'partner_id': self.partner_a.id, 'invoice_line_ids': [Command.create({'product_id': self.product_a.id})], }) invoice.action_post() payment = self.env['account.payment.register']\ .with_context(active_model='account.move', active_ids=invoice.ids)\ .create({})\ ._create_payments() self.assertTrue(invoice.payment_state in ('paid', 'in_payment')) self.assertRecordValues(payment, [{'reconciled_invoice_ids': invoice.ids}]) # Reverse the payment move reversal_wizard = self.env['account.move.reversal']\ .with_context(active_model='account.move', active_ids=payment.move_id.ids)\ .create({'reason': "oopsie", 'journal_id': payment.journal_id.id}) reversal_wizard.refund_moves() self.assertRecordValues(invoice, [{'payment_state': 'not_paid'}]) self.assertRecordValues(payment.line_ids, [{'reconciled': True}] * 2) def test_payment_name(self): AccountPayment = self.env['account.payment'] AccountPayment.search([]).unlink() payment = AccountPayment.create({ 'journal_id': self.company_data['default_journal_bank'].id, }) self.assertRegex(payment.name, r'BNK1/\d{4}/00001') with Form(AccountPayment.with_context(default_move_journal_types=('bank', 'cash'))) as payment_form: self.assertEqual(payment_form.name, '/') payment_form.journal_id = self.company_data['default_journal_cash'] self.assertRegex(payment_form.name, r'CSH1/\d{4}/00001') payment_form.journal_id = self.company_data['default_journal_bank'] payment = payment_form.save() self.assertEqual(payment.name, '/') payment.action_post() self.assertRegex(payment.name, r'BNK1/\d{4}/00002') def test_payment_without_default_company_account(self): """ The purpose of this test is to check the specific behavior when duplicating an inbound payment, then change the copy to an outbound payment when we set the outstanding accounts (payments and receipts) on a journal but not on the company level. """ company = self.company_data['company'] bank_journal = self.company_data['default_journal_bank'] bank_journal.outbound_payment_method_line_ids.payment_account_id = company.account_journal_payment_credit_account_id bank_journal.inbound_payment_method_line_ids.payment_account_id = company.account_journal_payment_debit_account_id company.account_journal_payment_debit_account_id = False company.account_journal_payment_credit_account_id = False payment = self.env['account.payment'].create({ 'amount': 5.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'journal_id': bank_journal.id, }) self.assertRecordValues(payment, [{ 'amount': 5.0, 'payment_type': 'inbound', 'partner_type': 'customer', 'payment_reference': False, 'is_reconciled': False, 'currency_id': self.company_data['currency'].id, 'partner_id': False, 'payment_method_line_id': self.inbound_payment_method_line.id, }]) payment.payment_type = 'outbound' self.assertRecordValues(payment, [{ 'amount': 5.0, 'payment_type': 'outbound', 'partner_type': 'customer', 'payment_reference': False, 'is_reconciled': False, 'currency_id': self.company_data['currency'].id, 'partner_id': False, 'payment_method_line_id': self.outbound_payment_method_line.id, }]) def test_suggested_default_partner_bank(self): """ Ensure the 'partner_bank_id' is well computed on payments. When the payment is inbound, the money must be received by a bank account linked to the company. In case of outbound payment, the bank account must be found on the partner. """ payment = self.env['account.payment'].create({ 'journal_id': self.bank_journal_1.id, 'amount': 50.0, 'payment_type': 'outbound', 'partner_type': 'supplier', 'partner_id': self.partner_a.id, }) self.assertRecordValues(payment, [{ 'available_partner_bank_ids': self.partner_a.bank_ids.ids, 'partner_bank_id': self.partner_bank_account1.id, }]) payment.payment_type = 'inbound' self.assertRecordValues(payment, [{ 'available_partner_bank_ids': [], 'partner_bank_id': False, }]) self.bank_journal_2.bank_account_id = self.comp_bank_account2 # A sequence is automatically added on the first move. We need to clean it before changing the journal. payment.name = False payment.journal_id = self.bank_journal_2 self.assertRecordValues(payment, [{ 'available_partner_bank_ids': self.comp_bank_account2.ids, 'partner_bank_id': self.comp_bank_account2.id, }]) def test_internal_transfer_right_accounts(self): """The purpose of this test is to check that the right accounts are computed when making an internal bank transfer""" company = self.env.company transfer_account = company.transfer_account_id bank = self.bank_journal_1 bank_2 = self.bank_journal_2 payment = self.env['account.payment'].create({ 'is_internal_transfer': True, 'payment_type': 'outbound', 'amount': 100.0, 'journal_id': bank.id, 'destination_journal_id': bank_2.id, }) payment.action_post() self.assertRecordValues(payment.line_ids, [ {'account_id': company.account_journal_payment_credit_account_id.id}, {'account_id': transfer_account.id}, ]) self.assertRecordValues(payment.paired_internal_transfer_payment_id.line_ids, [ {'account_id': company.account_journal_payment_debit_account_id.id}, {'account_id': transfer_account.id}, ]) # We check the behavior when setting specific receipts/payments account on bank journals bank.inbound_payment_method_line_ids.payment_account_id = company.account_journal_payment_debit_account_id.copy({'code': 1001}) bank.outbound_payment_method_line_ids.payment_account_id = company.account_journal_payment_credit_account_id.copy({'code': 1002}) bank_2.inbound_payment_method_line_ids.payment_account_id = company.account_journal_payment_debit_account_id.copy({'code': 2001}) bank_2.outbound_payment_method_line_ids.payment_account_id = company.account_journal_payment_credit_account_id.copy({'code': 2002}) payment = self.env['account.payment'].create({ 'is_internal_transfer': True, 'payment_type': 'outbound', 'amount': 100.0, 'journal_id': bank.id, 'destination_journal_id': bank_2.id, }) payment.action_post() self.assertRecordValues(payment.line_ids, [ {'account_id': bank.outbound_payment_method_line_ids.payment_account_id.id,}, {'account_id': transfer_account.id}, ]) self.assertRecordValues(payment.paired_internal_transfer_payment_id.line_ids, [ {'account_id': bank_2.inbound_payment_method_line_ids.payment_account_id.id}, {'account_id': transfer_account.id}, ]) def test_reconciliation_with_old_oustanding_account(self): """ Test the reconciliation of an invoice with a payment after changing the outstanding account of the journal. """ outstanding_account_1 = self.company_data['company'].account_journal_payment_debit_account_id.copy() outstanding_account_2 = outstanding_account_1.copy() self.company_data['default_journal_bank'].inbound_payment_method_line_ids.payment_account_id = outstanding_account_1 payment = self.env['account.payment'].create({ 'payment_type': 'inbound', 'partner_type': 'customer', 'partner_id': self.partner_a.id, 'journal_id': self.company_data['default_journal_bank'].id, 'amount': 1150, }) payment.action_post() self.company_data['default_journal_bank'].inbound_payment_method_line_ids.payment_account_id = outstanding_account_2 invoice = self.init_invoice('out_invoice', post=True, amounts=[1000.0], taxes=self.env['account.tax']) credit_line = payment.line_ids.filtered(lambda l: l.credit and l.account_id == self.company_data['default_account_receivable']) invoice.js_assign_outstanding_line(credit_line.id) self.assertTrue(invoice.payment_state in ('in_payment', 'paid'), "Invoice should be paid") invoice.button_draft() self.assertTrue(invoice.payment_state == 'not_paid', "Invoice should'nt be paid anymore") self.assertTrue(invoice.state == 'draft', "Invoice should be draft") def test_journal_onchange(self): """Ensure that the payment method line is recomputed when switching journal in form view.""" context = { 'payment_type': 'inbound', 'partner_type': 'customer', } with Form(self.env['account.payment'].with_context(context)) as payment: default_journal = payment.journal_id self.assertTrue(default_journal) self.assertEqual(payment.payment_method_line_id.journal_id.id, default_journal.id) other_journal = self.bank_journal_2 if default_journal != self.bank_journal_2 else self.bank_journal_1 payment.journal_id = other_journal self.assertEqual(payment.payment_method_line_id.journal_id.id, other_journal.id) payment.journal_id = default_journal self.assertEqual(payment.payment_method_line_id.journal_id.id, default_journal.id)