[REF] account: create 'account.reconciliation.widget' model

Contains widget reconciliation methods. This refactoring does not modify
the feature, some errors have been notified in the code but not modified.

5 files changed, 19 insertions, 31 deletions

diff --git a/addons/account/tests/__init__.py b/addons/account/tests/__init__.py
index 02b00337259..813896bdec0 100644
--- a/addons/account/tests/__init__.py
+++ b/addons/account/tests/__init__.py
@@ -8,10 +8,8 @@ from . import test_account_validate_account_move
 from . import test_account_invoice_rounding
 from . import test_account_supplier_invoice_recurrent
 from . import test_bank_statement_reconciliation
-#TODO re-enableand fix this test
-#from . import test_bank_stmt_reconciliation_widget_ui
 from . import test_fiscal_position
-from . import test_manual_reconciliation
+from . import test_reconciliation_widget
 from . import test_payment
 from . import test_product_id_change
 from . import test_reconciliation
diff --git a/addons/account/tests/test_bank_statement_reconciliation.py b/addons/account/tests/test_bank_statement_reconciliation.py
index 4bf89f7a108..6da03b3c69e 100644
--- a/addons/account/tests/test_bank_statement_reconciliation.py
+++ b/addons/account/tests/test_bank_statement_reconciliation.py
@@ -11,6 +11,7 @@ class TestBankStatementReconciliation(AccountingTestCase):
         self.il_model = self.env['account.invoice.line']
         self.bs_model = self.env['account.bank.statement']
         self.bsl_model = self.env['account.bank.statement.line']
+        self.reconciliation_widget = self.env['account.reconciliation.widget']
         self.partner_agrolait = self.env.ref("base.res_partner_2")
 
     def test_reconciliation_proposition(self):
@@ -18,9 +19,11 @@ class TestBankStatementReconciliation(AccountingTestCase):
         st_line = self.create_statement_line(100)
 
         # exact amount match
-        rec_prop = st_line.get_reconciliation_proposition()
-        self.assertEqual(len(rec_prop), 1)
-        self.assertEqual(rec_prop[0].id, rcv_mv_line.id)
+        rec_prop = self.reconciliation_widget.get_bank_statement_line_data(st_line.ids)
+        prop = rec_prop[0]['reconciliation_proposition']
+
+        self.assertEqual(len(prop), 1)
+        self.assertEqual(prop[0]['id'], rcv_mv_line.id)
 
     def test_full_reconcile(self):
         rcv_mv_line = self.create_invoice(100)
diff --git a/addons/account/tests/test_bank_stmt_reconciliation_widget_ui.py b/addons/account/tests/test_bank_stmt_reconciliation_widget_ui.py
deleted file mode 100644
index 785ce234022..00000000000
--- a/addons/account/tests/test_bank_stmt_reconciliation_widget_ui.py
+++ /dev/null
@@ -1,7 +0,0 @@
-from odoo.tests import HttpCase, tagged
-
-@odoo.tests.tagged('post_install','-at_install')
-class TestUi(HttpCase):
-
-    def test_01_admin_bank_statement_reconciliation(self):
-        self.phantom_js("/", "odoo.__DEBUG__.services['web.Tour'].run('bank_statement_reconciliation', 'test')", "odoo.__DEBUG__.services['web.Tour'].tours.bank_statement_reconciliation", login="admin")
diff --git a/addons/account/tests/test_manual_reconciliation.py b/addons/account/tests/test_manual_reconciliation.py
deleted file mode 100644
index 5f01f0165d7..00000000000
--- a/addons/account/tests/test_manual_reconciliation.py
+++ /dev/null
@@ -1,18 +0,0 @@
-from odoo.addons.account.tests.account_test_classes import AccountingTestCase
-from odoo.tests import tagged
-
-
-@tagged('post_install','-at_install')
-class TestManualReconciliation(AccountingTestCase):
-
-    def test_reconciliation_proposition(self):
-        pass
-
-    def test_full_reconcile(self):
-        pass
-
-    def test_partial_reconcile(self):
-        pass
-
-    def test_reconcile_with_write_off(self):
-        pass
diff --git a/addons/account/tests/test_reconciliation_widget.py b/addons/account/tests/test_reconciliation_widget.py
new file mode 100644
index 00000000000..9ca8e36c4df
--- /dev/null
+++ b/addons/account/tests/test_reconciliation_widget.py
@@ -0,0 +1,12 @@
+import odoo.tests
+
+
+@odoo.tests.tagged('post_install', '-at_install')
+@odoo.tests.common.at_install(False)
+@odoo.tests.common.post_install(True)
+class TestUi(odoo.tests.HttpCase):
+
+    def test_01_admin_bank_statement_reconciliation(self):
+        self.phantom_js("/web",
+            "odoo.__DEBUG__.services['web_tour.tour'].run('bank_statement_reconciliation')",
+            "odoo.__DEBUG__.services['web_tour.tour'].tours.bank_statement_reconciliation.ready", login="admin")