initial commit 2

1 files changed, 291 insertions, 0 deletions

diff --git a/addons/account/tests/test_transfer_wizard.py b/addons/account/tests/test_transfer_wizard.py
new file mode 100644
index 00000000..eb78a3da
--- /dev/null
+++ b/addons/account/tests/test_transfer_wizard.py
@@ -0,0 +1,291 @@
+# -*- coding: utf-8 -*-
+from odoo.addons.account.tests.common import AccountTestInvoicingCommon
+from odoo.tests import tagged, Form
+import time
+
+@tagged('post_install', '-at_install')
+class TestTransferWizard(AccountTestInvoicingCommon):
+
+    @classmethod
+    def setUpClass(cls, chart_template_ref=None):
+        super().setUpClass(chart_template_ref=chart_template_ref)
+
+        cls.company = cls.company_data['company']
+        cls.receivable_account = cls.company_data['default_account_receivable']
+        cls.payable_account = cls.company_data['default_account_payable']
+        cls.accounts = cls.env['account.account'].search([('reconcile', '=', False), ('company_id', '=', cls.company.id)], limit=5)
+        cls.journal = cls.company_data['default_journal_misc']
+
+        # Set rate for base currency to 1
+        cls.env['res.currency.rate'].search([('company_id', '=', cls.company.id), ('currency_id', '=', cls.company.currency_id.id)]).write({'rate': 1})
+
+        # Create test currencies
+        cls.test_currency_1 = cls.env['res.currency'].create({
+            'name': "PMK",
+            'symbol':'P',
+        })
+
+        cls.test_currency_2 = cls.env['res.currency'].create({
+            'name': "toto",
+            'symbol':'To',
+        })
+
+        cls.test_currency_3 = cls.env['res.currency'].create({
+            'name': "titi",
+            'symbol':'Ti',
+        })
+
+        # Create test rates
+        cls.env['res.currency.rate'].create({
+            'name': time.strftime('%Y') + '-' + '01' + '-01',
+            'rate': 0.5,
+            'currency_id': cls.test_currency_1.id,
+            'company_id': cls.company.id
+        })
+
+        cls.env['res.currency.rate'].create({
+            'name': time.strftime('%Y') + '-' + '01' + '-01',
+            'rate': 2,
+            'currency_id': cls.test_currency_2.id,
+            'company_id': cls.company.id
+        })
+
+        cls.env['res.currency.rate'].create({
+            'name': time.strftime('%Y') + '-' + '01' + '-01',
+            'rate': 10,
+            'currency_id': cls.test_currency_3.id,
+            'company_id': cls.company.id
+        })
+
+        # Create an account using a foreign currency
+        cls.test_currency_account = cls.env['account.account'].create({
+            'name': 'test destination account',
+            'code': 'test_dest_acc',
+            'user_type_id': cls.env['ir.model.data'].xmlid_to_res_id('account.data_account_type_current_assets'),
+            'currency_id': cls.test_currency_3.id,
+        })
+
+        # Create test account.move
+        cls.move_1 = cls.env['account.move'].create({
+            'journal_id': cls.journal.id,
+            'line_ids': [
+                (0, 0, {
+                    'name': "test1_1",
+                    'account_id': cls.receivable_account.id,
+                    'debit': 500,
+                }),
+                (0, 0, {
+                    'name': "test1_2",
+                    'account_id': cls.accounts[0].id,
+                    'credit': 500,
+                }),
+                (0, 0, {
+                    'name': "test1_3",
+                    'account_id': cls.accounts[0].id,
+                    'debit': 800,
+                    'partner_id': cls.partner_a.id,
+                }),
+                (0, 0, {
+                    'name': "test1_4",
+                    'account_id': cls.accounts[1].id,
+                    'credit': 500,
+                }),
+                (0, 0, {
+                    'name': "test1_5",
+                    'account_id': cls.accounts[2].id,
+                    'credit': 300,
+                    'partner_id': cls.partner_a.id,
+                }),
+                (0, 0, {
+                    'name': "test1_6",
+                    'account_id': cls.accounts[0].id,
+                    'debit': 270,
+                    'currency_id': cls.test_currency_1.id,
+                    'amount_currency': 540,
+                }),
+                (0, 0, {
+                    'name': "test1_7",
+                    'account_id': cls.accounts[1].id,
+                    'credit': 140,
+                }),
+                (0, 0, {
+                    'name': "test1_8",
+                    'account_id': cls.accounts[2].id,
+                    'credit': 160,
+                }),
+                (0, 0, {
+                    'name': "test1_9",
+                    'account_id': cls.accounts[2].id,
+                    'debit': 30,
+                    'currency_id': cls.test_currency_2.id,
+                    'amount_currency': 15,
+                }),
+            ]
+        })
+        cls.move_1.action_post()
+
+        cls.move_2 = cls.env['account.move'].create({
+            'journal_id': cls.journal.id,
+            'line_ids': [
+                (0, 0, {
+                    'name': "test2_1",
+                    'account_id': cls.accounts[1].id,
+                    'debit': 400,
+                }),
+                (0, 0, {
+                    'name': "test2_2",
+                    'account_id': cls.payable_account.id,
+                    'credit': 400,
+                }),
+                (0, 0, {
+                    'name': "test2_3",
+                    'account_id': cls.accounts[3].id,
+                    'debit': 250,
+                    'partner_id': cls.partner_a.id,
+                }),
+                (0, 0, {
+                    'name': "test2_4",
+                    'account_id': cls.accounts[1].id,
+                    'debit': 480,
+                    'partner_id': cls.partner_b.id,
+                }),
+                (0, 0, {
+                    'name': "test2_5",
+                    'account_id': cls.accounts[2].id,
+                    'credit': 730,
+                    'partner_id': cls.partner_a.id,
+                }),
+                (0, 0, {
+                    'name': "test2_6",
+                    'account_id': cls.accounts[2].id,
+                    'credit': 412,
+                    'partner_id': cls.partner_a.id,
+                    'currency_id': cls.test_currency_2.id,
+                    'amount_currency': -633,
+                }),
+                (0, 0, {
+                    'name': "test2_7",
+                    'account_id': cls.accounts[1].id,
+                    'debit': 572,
+                }),
+                (0, 0, {
+                    'name': "test2_8",
+                    'account_id': cls.accounts[2].id,
+                    'credit': 100,
+                    'partner_id': cls.partner_a.id,
+                    'currency_id': cls.test_currency_2.id,
+                    'amount_currency': -123,
+                }),
+                (0, 0, {
+                    'name': "test2_9",
+                    'account_id': cls.accounts[2].id,
+                    'credit': 60,
+                    'partner_id': cls.partner_a.id,
+                    'currency_id': cls.test_currency_1.id,
+                    'amount_currency': -10,
+                }),
+            ]
+        })
+        cls.move_2.action_post()
+
+
+    def test_transfer_wizard_reconcile(self):
+        """ Tests reconciliation when doing a transfer with the wizard
+        """
+        active_move_lines = (self.move_1 + self.move_2).mapped('line_ids').filtered(lambda x: x.account_id.user_type_id.type in ('receivable', 'payable'))
+
+        # We use a form to pass the context properly to the depends_context move_line_ids field
+        context = {'active_model': 'account.move.line', 'active_ids': active_move_lines.ids}
+        with Form(self.env['account.automatic.entry.wizard'].with_context(context)) as wizard_form:
+            wizard_form.action = 'change_account'
+            wizard_form.destination_account_id = self.receivable_account
+            wizard_form.journal_id = self.journal
+        wizard = wizard_form.save()
+
+        transfer_move_id = wizard.do_action()['res_id']
+        transfer_move = self.env['account.move'].browse(transfer_move_id)
+
+        payable_transfer = transfer_move.line_ids.filtered(lambda x: x.account_id == self.payable_account)
+        receivable_transfer = transfer_move.line_ids.filtered(lambda x: x.account_id == self.receivable_account)
+
+        self.assertTrue(payable_transfer.reconciled, "Payable line of the transfer move should be fully reconciled")
+        self.assertAlmostEqual(self.move_1.line_ids.filtered(lambda x: x.account_id == self.receivable_account).amount_residual, 100, self.company.currency_id.decimal_places, "Receivable line of the original move should be partially reconciled, and still have a residual amount of 100 (500 - 400 from payable account)")
+        self.assertTrue(self.move_2.line_ids.filtered(lambda x: x.account_id == self.payable_account).reconciled, "Payable line of the original move should be fully reconciled")
+        self.assertAlmostEqual(receivable_transfer.amount_residual, 0, self.company.currency_id.decimal_places, "Receivable line from the transfer move should have nothing left to reconcile")
+        self.assertAlmostEqual(payable_transfer.debit, 400, self.company.currency_id.decimal_places, "400 should have been debited from payable account to apply the transfer")
+        self.assertAlmostEqual(receivable_transfer.credit, 400, self.company.currency_id.decimal_places, "400 should have been credited to receivable account to apply the transfer")
+
+    def test_transfer_wizard_grouping(self):
+        """ Tests grouping (by account and partner) when doing a transfer with the wizard
+        """
+        active_move_lines = (self.move_1 + self.move_2).mapped('line_ids').filtered(lambda x: x.name in ('test1_3', 'test1_4', 'test1_5', 'test2_3', 'test2_4', 'test2_5', 'test2_6', 'test2_8'))
+
+        # We use a form to pass the context properly to the depends_context move_line_ids field
+        context = {'active_model': 'account.move.line', 'active_ids': active_move_lines.ids}
+        with Form(self.env['account.automatic.entry.wizard'].with_context(context)) as wizard_form:
+            wizard_form.action = 'change_account'
+            wizard_form.destination_account_id = self.accounts[4]
+            wizard_form.journal_id = self.journal
+        wizard = wizard_form.save()
+
+        transfer_move_id = wizard.do_action()['res_id']
+        transfer_move = self.env['account.move'].browse(transfer_move_id)
+
+        groups = {}
+        for line in transfer_move.line_ids:
+            key = (line.account_id, line.partner_id or None, line.currency_id)
+            self.assertFalse(groups.get(key), "There should be only one line per (account, partner, currency) group in the transfer move.")
+            groups[key] = line
+
+        self.assertAlmostEqual(groups[(self.accounts[0], self.partner_a, self.company_data['currency'])].balance, -800, self.company.currency_id.decimal_places)
+        self.assertAlmostEqual(groups[(self.accounts[1], None, self.company_data['currency'])].balance, 500, self.company.currency_id.decimal_places)
+        self.assertAlmostEqual(groups[(self.accounts[1], self.partner_b, self.company_data['currency'])].balance, -480, self.company.currency_id.decimal_places)
+        self.assertAlmostEqual(groups[(self.accounts[2], self.partner_a, self.company_data['currency'])].balance, 1030, self.company.currency_id.decimal_places)
+        self.assertAlmostEqual(groups[(self.accounts[2], self.partner_a, self.test_currency_2)].balance, 512, self.company.currency_id.decimal_places)
+        self.assertAlmostEqual(groups[(self.accounts[3], self.partner_a, self.company_data['currency'])].balance, -250, self.company.currency_id.decimal_places)
+
+
+    def test_transfer_wizard_currency_conversion(self):
+        """ Tests multi currency use of the transfer wizard, checking the conversion
+        is propperly done when using a destination account with a currency_id set.
+        """
+        active_move_lines = self.move_1.mapped('line_ids').filtered(lambda x: x.name in ('test1_6', 'test1_9'))
+
+        # We use a form to pass the context properly to the depends_context move_line_ids field
+        context = {'active_model': 'account.move.line', 'active_ids': active_move_lines.ids}
+        with Form(self.env['account.automatic.entry.wizard'].with_context(context)) as wizard_form:
+            wizard_form.action = 'change_account'
+            wizard_form.destination_account_id = self.test_currency_account
+            wizard_form.journal_id = self.journal
+        wizard = wizard_form.save()
+
+        transfer_move_id = wizard.do_action()['res_id']
+        transfer_move = self.env['account.move'].browse(transfer_move_id)
+
+        destination_line = transfer_move.line_ids.filtered(lambda x: x.account_id == self.test_currency_account)
+        self.assertEqual(destination_line.currency_id, self.test_currency_3, "Transferring to an account with a currency set should keep this currency on the transfer line.")
+        self.assertAlmostEqual(destination_line.amount_currency, 3000, self.company.currency_id.decimal_places, "Transferring two lines with different currencies (and the same partner) on an account with a currency set should convert the balance of these lines into this account's currency (here (270 + 30) * 10 = 3000)")
+
+
+    def test_transfer_wizard_no_currency_conversion(self):
+        """ Tests multi currency use of the transfer wizard, verifying that
+        currency amounts are kept on distinct lines when transferring to an
+        account without any currency specified.
+        """
+        active_move_lines = self.move_2.mapped('line_ids').filtered(lambda x: x.name in ('test2_9', 'test2_6', 'test2_8'))
+
+        # We use a form to pass the context properly to the depends_context move_line_ids field
+        context = {'active_model': 'account.move.line', 'active_ids': active_move_lines.ids}
+        with Form(self.env['account.automatic.entry.wizard'].with_context(context)) as wizard_form:
+            wizard_form.action = 'change_account'
+            wizard_form.destination_account_id = self.receivable_account
+            wizard_form.journal_id = self.journal
+        wizard = wizard_form.save()
+
+        transfer_move_id = wizard.do_action()['res_id']
+        transfer_move = self.env['account.move'].browse(transfer_move_id)
+
+        destination_lines = transfer_move.line_ids.filtered(lambda x: x.account_id == self.receivable_account)
+        self.assertEqual(len(destination_lines), 2, "Two lines should have been created on destination account: one for each currency (the lines with same partner and currency should have been aggregated)")
+        self.assertAlmostEqual(destination_lines.filtered(lambda x: x.currency_id == self.test_currency_1).amount_currency, -10, self.test_currency_1.decimal_places)
+        self.assertAlmostEqual(destination_lines.filtered(lambda x: x.currency_id == self.test_currency_2).amount_currency, -756, self.test_currency_2.decimal_places)