[FIX] l10n_es_pos: fixed tests

Author: sjai-odoo

addons/l10n_es_pos/models/__init__.py

@@ -3,3 +3,4 @@
 from . import res_config_settings
 from . import account_move
 from . import res_company
+from . import pos_session

addons/l10n_es_pos/models/pos_order.py

@@ -20,6 +20,3 @@ def _prepare_invoice_vals(self):
         if self.config_id.is_spanish and self.is_l10n_es_simplified_invoice:
             res["journal_id"] = self.config_id.l10n_es_simplified_invoice_journal_id.id
         return res
-
-    def get_invoice_name(self):
-        return self.account_move.name

addons/l10n_es_pos/models/pos_session.py

@@ -0,0 +1,12 @@
+from odoo import models, api
+
+
+class PosSession(models.Model):
+    _inherit = "pos.session"
+
+    @api.model
+    def _load_pos_data_models(self, config_id):
+        data = super()._load_pos_data_models(config_id)
+        if self.env.company.country_id.code == "ES":
+            data += ['account.move']
+        return data

addons/l10n_es_pos/static/src/app/screens/payment_screen/payment_screen.js

@@ -51,17 +51,14 @@ patch(PaymentScreen.prototype, {
         }
         return false;
     },
-    shouldDownloadInvoice() {
-        return this.pos.config.is_spanish
-            ? !this.currentOrder.is_l10n_es_simplified_invoice
-            : super.shouldDownloadInvoice();
-    },
+    // shouldDownloadInvoice() {
+    //     return this.pos.config.is_spanish
+    //         ? !this.currentOrder.is_l10n_es_simplified_invoice
+    //         : super.shouldDownloadInvoice();
+    // },
     async _postPushOrderResolve(order, order_server_ids) {
         if (this.pos.config.is_spanish) {
-            const invoiceName = await this.pos.data.call("pos.order", "get_invoice_name", [
-                order_server_ids,
-            ]);
-            order.invoice_name = invoiceName;
+            order.invoice_name = order.account_move?.name;;
         }
         return super._postPushOrderResolve(...arguments);
     },

addons/l10n_es_pos/tests/test_frontend.py

@@ -34,8 +34,8 @@ def get_number_of_regular_invoices():
         self.main_pos_config.l10n_es_simplified_invoice_journal_id = simp
         # this `limit` value is linked to the `SIMPLIFIED_INVOICE_LIMIT` const in the tour
         self._get_main_company().l10n_es_simplified_invoice_limit = 1000
-        self.main_pos_config.with_user(self.pos_user).open_ui()
-        self.start_pos_tour("spanish_pos_tour")
+        self.main_pos_config.with_user(self.pos_admin).open_ui()
+        self.start_pos_tour("spanish_pos_tour", login="accountman")
         num_of_simp_invoices = self.env['account.move'].search_count([('journal_id', '=', simp.id), ('l10n_es_is_simplified', '=', True)])
         num_of_regular_invoices = get_number_of_regular_invoices() - initial_number_of_regular_invoices
         self.assertEqual(num_of_simp_invoices, 3)
@@ -150,6 +150,6 @@ def test_simplified_invoice_not_override_set_pricelist(self):
         self.assertTrue(len(self.main_pos_config.available_pricelist_ids.ids) > 1)
         self.main_pos_config.l10n_es_simplified_invoice_journal_id = self.main_pos_config.journal_id
         self.main_pos_config.open_ui()
-        self.start_tour("/pos/ui?config_id=%d" % self.main_pos_config.id, 'test_simplified_invoice_not_override_set_pricelist', login="pos_user")
+        self.start_tour("/pos/ui?config_id=%d" % self.main_pos_config.id, 'test_simplified_invoice_not_override_set_pricelist', login="accountman")
         order = self.env['pos.order'].search([('partner_id', '=', self.main_pos_config.simplified_partner_id.id)])
         self.assertNotEqual(order.pricelist_id, self.main_pos_config.simplified_partner_id.property_product_pricelist)