1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
|
from odoo.addons.account.tests.common import AccountTestInvoicingCommon
from odoo.tests import tagged, Form
@tagged('post_install', '-at_install')
class TestAccountPaymentMethodLine(AccountTestInvoicingCommon):
@classmethod
def setUpClass(cls):
super().setUpClass()
cls.bank_journal_1 = cls.company_data['default_journal_bank']
cls.bank_journal_2 = cls.company_data['default_journal_bank'].copy()
cls.inbound_payment_method_line_1 = cls.env['account.payment.method.line'].create({
'name': 'new inbound payment method line 1',
'payment_method_id': cls.bank_journal_1.available_payment_method_ids[0].id,
'payment_type': 'inbound',
'journal_id': cls.bank_journal_1.id,
})
cls.inbound_payment_method_line_2 = cls.env['account.payment.method.line'].create({
'name': 'new inbound payment method line 2',
'payment_method_id': cls.bank_journal_1.available_payment_method_ids[0].id,
'payment_type': 'inbound',
'journal_id': cls.bank_journal_1.id,
})
cls.inbound_payment_method_line_other_journal = cls.env['account.payment.method.line'].create({
'name': 'new inbound payment method line other journal',
'payment_method_id': cls.bank_journal_2.available_payment_method_ids[0].id,
'payment_type': 'inbound',
'journal_id': cls.bank_journal_2.id,
})
cls.partner_c = cls.partner_a.copy()
cls.partner_a.property_inbound_payment_method_line_id = cls.inbound_payment_method_line_1
cls.partner_b.property_inbound_payment_method_line_id = cls.inbound_payment_method_line_2
cls.partner_c.property_inbound_payment_method_line_id = cls.inbound_payment_method_line_other_journal
cls.move_partner_a = cls.init_invoice(move_type='out_invoice', partner=cls.partner_a, products=cls.product_a, post=True)
cls.move_partner_b = cls.init_invoice(move_type='out_invoice', partner=cls.partner_b, products=cls.product_a, post=True)
cls.move_partner_c = cls.init_invoice(move_type='out_invoice', partner=cls.partner_c, products=cls.product_a, post=True)
def assertRegisterPayment(self, expected_journal, expected_payment_method, move_partner, payment_method_line=False):
if payment_method_line and expected_payment_method:
move_partner.preferred_payment_method_line_id = expected_payment_method
payment = self.env['account.payment.register'].with_context(
active_model='account.move',
active_ids=move_partner.ids,
).create({})
if not expected_payment_method:
expected_payment_method = payment.journal_id._get_available_payment_method_lines(payment.payment_type)[0]._origin
self.assertRecordValues(payment, [{
'journal_id': expected_journal.id,
'payment_method_line_id': expected_payment_method.id,
}])
def test_move_register_payment_wizard(self):
"""
This test will do a basic flow where we do a register payment from an invoice by using the register payment
wizard. If we have a payment method set on the partner, the preferred payment method will be the one from
the partner and so the wizard will have the payment method line from the partner. However, we can modify the
preferred payment line on the move and so the payment method line and journal of the wizard will be changed.
"""
# The preferred payment method will be the one set on the partner
self.assertRegisterPayment(
self.bank_journal_1,
self.inbound_payment_method_line_1,
self.move_partner_a,
)
# We then modify it from the move and check if that still works
self.assertRegisterPayment(
self.bank_journal_1,
self.inbound_payment_method_line_2,
self.move_partner_a,
True,
)
self.assertRegisterPayment(
self.bank_journal_2,
self.inbound_payment_method_line_other_journal,
self.move_partner_a,
True,
)
def test_multiple_moves_register_payment(self):
"""
This will test the register payment wizard when selecting multiple move with different partner to see if the
payment method lines are set correctly.
"""
# Test with two moves with same payment method lines and same partners
move_partner_a_copy = self.move_partner_a.copy()
move_partner_a_copy.action_post()
self.assertRegisterPayment(
self.bank_journal_1,
self.inbound_payment_method_line_1,
self.move_partner_a + move_partner_a_copy,
)
# Test with two moves with same payment method lines but different partners
self.partner_d = self.partner_a.copy()
move_partner_d = self.init_invoice(move_type='out_invoice', partner=self.partner_d, products=self.product_a, post=True)
self.assertRegisterPayment(
self.bank_journal_1,
self.inbound_payment_method_line_1,
self.move_partner_a + move_partner_d,
)
# Test with two moves with different partners and different payment method lines
self.assertRegisterPayment(
self.bank_journal_1,
None, # We will get in the assertRegisterPayment the first payment method line of the journal
self.move_partner_a + self.move_partner_b,
)
def test_move_register_payment_view(self):
"""
This test will check the payment method line on a payment from the account payment view.
When setting a partner the payment method must change and the journal if the payment method line is from
another journal that the one that has been set.
"""
with Form(self.env['account.payment'].with_context(default_partner_id=self.partner_a)) as pay_form:
self.assertEqual(pay_form.journal_id.id, self.bank_journal_1.id)
self.assertEqual(pay_form.payment_method_line_id.id, self.inbound_payment_method_line_1.id)
pay_form.partner_id = self.partner_b
self.assertEqual(pay_form.payment_method_line_id.id, self.inbound_payment_method_line_2.id)
pay_form.partner_id = self.partner_c
self.assertEqual(pay_form.journal_id.id, self.bank_journal_2.id)
self.assertEqual(pay_form.payment_method_line_id.id, self.inbound_payment_method_line_other_journal.id)
|
