From e69ec2c8561ae508e037a25585da5430ae2a067d Mon Sep 17 00:00:00 2001
From: jorgep <jorgep@verdnatura.es>
Date: Wed, 17 Jan 2024 12:07:22 +0100
Subject: [PATCH] feat: refs #6276 test expeditionPallet_get

---
 loopback/locale/en.json                       |  3 +-
 loopback/locale/es.json                       |  4 +--
 .../methods/expedition-pallet/getPallet.js    | 14 ++++++---
 .../expedition-pallet/specs/getPallet.spec.js | 31 +++++++++++++++++++
 4 files changed, 45 insertions(+), 7 deletions(-)
 create mode 100644 modules/ticket/back/methods/expedition-pallet/specs/getPallet.spec.js

diff --git a/loopback/locale/en.json b/loopback/locale/en.json
index ba9acecaeb..3910341264 100644
--- a/loopback/locale/en.json
+++ b/loopback/locale/en.json
@@ -204,5 +204,6 @@
 	"The notification subscription of this worker cant be modified": "The notification subscription of this worker cant be modified",
 	"It was not able to remove the next expeditions:": "It was not able to remove the next expeditions: {{expeditions}}",
 	"Incorrect pin": "Incorrect pin.",
-	"The notification subscription of this worker cant be modified": "The notification subscription of this worker cant be modified"
+	"This machine is already in use.": "This machine is already in use.",
+	"This pallet does not exist": "This pallet does not exist"
 }
\ No newline at end of file
diff --git a/loopback/locale/es.json b/loopback/locale/es.json
index babb4bd2a8..b273f7506c 100644
--- a/loopback/locale/es.json
+++ b/loopback/locale/es.json
@@ -340,6 +340,6 @@
 	"No tickets to invoice": "No hay tickets para facturar",
 	"No hay tickets para sacar": "No hay tickets para sacar",
 	"There is no zone for these parameters 999999": "There is no zone for these parameters 999999",
-	"Esta máquina ya está en uso.": "Esta máquina ya está en uso.",
-	"This machine is already in use.": "This machine is already in use."
+	"This machine is already in use.": "Esta máquina ya está en uso.",
+	"This pallet does not exist": "Este palet no existe"
 }
\ No newline at end of file
diff --git a/modules/ticket/back/methods/expedition-pallet/getPallet.js b/modules/ticket/back/methods/expedition-pallet/getPallet.js
index 22cbbc210b..340dc02b0f 100644
--- a/modules/ticket/back/methods/expedition-pallet/getPallet.js
+++ b/modules/ticket/back/methods/expedition-pallet/getPallet.js
@@ -1,7 +1,7 @@
 
 const UserError = require('vn-loopback/util/user-error');
 module.exports = Self => {
-    Self.remoteMethod('getPallet', {
+    Self.remoteMethodCtx('getPallet', {
         description: 'Get pallet',
         accessType: 'READ',
         accepts: [
@@ -20,7 +20,13 @@ module.exports = Self => {
         },
     });
 
-    Self.getPallet = async expeditionFk => {
+    Self.getPallet = async(ctx, expeditionFk, options) => {
+        const myOptions = {};
+        const $t = ctx.req.__;
+
+        if (typeof options == 'object')
+            Object.assign(myOptions, options);
+
         try {
             const pallet = await Self.findOne({
                 fields: ['truckFk'],
@@ -35,7 +41,7 @@ module.exports = Self => {
                         }
                     }
                 ],
-            });
+            }, myOptions);
 
             if (pallet) {
                 const truck = pallet.expeditionTruck();
@@ -46,7 +52,7 @@ module.exports = Self => {
                 };
             }
 
-            throw new UserError('palletDoesNotExist');
+            throw new UserError($t('This pallet does not exist'));
         } catch (e) {
             return {message: e.message};
         }
diff --git a/modules/ticket/back/methods/expedition-pallet/specs/getPallet.spec.js b/modules/ticket/back/methods/expedition-pallet/specs/getPallet.spec.js
new file mode 100644
index 0000000000..8ffc7d8c23
--- /dev/null
+++ b/modules/ticket/back/methods/expedition-pallet/specs/getPallet.spec.js
@@ -0,0 +1,31 @@
+const {models} = require('vn-loopback/server/server');
+
+fdescribe('expeditonPallet getPallet()', () => {
+    beforeAll(async() => {
+        ctx = {
+            accessToken: {userId: 9},
+            req: {
+                headers: {origin: 'http://localhost'},
+                __: value => value
+            }
+        };
+    });
+
+    it('should obtain the pallet data', async() => {
+        const pallet = await models.ExpeditionPallet.getPallet(ctx, 1);
+
+        expect(pallet).toBeDefined();
+        expect(pallet.truckFk).toEqual(1);
+        expect(pallet.description).toEqual('BEST TRUCK IN FLEET');
+    });
+
+    it('should throw an error when the pallet does not exist', async() => {
+        try {
+            await models.ExpeditionPallet.getPallet(ctx, 1);
+        } catch (e) {
+            const error = e;
+
+            expect(error.message).toEqual('This pallet does not exist');
+        }
+    });
+});