From f04724a61f6a9c924de9ec4998486651435b8d50 Mon Sep 17 00:00:00 2001
From: jorgep <jorgep@verdnatura.es>
Date: Mon, 22 Jan 2024 07:58:18 +0100
Subject: [PATCH] feat: refs #6276 new test added

---
 loopback/locale/en.json                       |  4 ++-
 loopback/locale/es.json                       |  3 +-
 .../specs/updateTracking.spec.js              | 28 ++++++++++++++++++-
 .../methods/sale-tracking/updateTracking.js   |  3 ++
 4 files changed, 35 insertions(+), 3 deletions(-)

diff --git a/loopback/locale/en.json b/loopback/locale/en.json
index c22aeb311..0f1be5776 100644
--- a/loopback/locale/en.json
+++ b/loopback/locale/en.json
@@ -209,5 +209,7 @@
 	"This pallet does not exist": "This pallet does not exist",
 	"We do not have availability for the selected item": "We do not have availability for the selected item",
 	"You are already using a machine": "You are already using a machine",
-	"This worker does not exist": "This worker does not exist"
+	"This worker does not exist": "This worker does not exist",
+	"this state does not exist": "This state does not exist"
+
 }
\ No newline at end of file
diff --git a/loopback/locale/es.json b/loopback/locale/es.json
index e5293d856..158d351df 100644
--- a/loopback/locale/es.json
+++ b/loopback/locale/es.json
@@ -346,5 +346,6 @@
 	"We do not have availability for the selected item": "No tenemos disponible el item seleccionado",
 	"You are already using a machine": "Ya estás usando una máquina.",
 	"This worker does not exist": "Este trabajador no existe",
-	"Este trabajador no existe": "Este trabajador no existe"
+	"this state does not exist": "Este estado no existe",
+	"Este estado no existe": "Este estado no existe"
 }
\ No newline at end of file
diff --git a/modules/ticket/back/methods/sale-tracking/specs/updateTracking.spec.js b/modules/ticket/back/methods/sale-tracking/specs/updateTracking.spec.js
index 2756bc09e..a58a69fe7 100644
--- a/modules/ticket/back/methods/sale-tracking/specs/updateTracking.spec.js
+++ b/modules/ticket/back/methods/sale-tracking/specs/updateTracking.spec.js
@@ -1,6 +1,6 @@
 const {models} = require('vn-loopback/server/server');
 
-fdescribe('saleTracking updateTracking()', () => {
+describe('saleTracking updateTracking()', () => {
     const saleFk = 1;
     const originalQuantity = 10;
     const code = 'PREPARED';
@@ -13,10 +13,36 @@ fdescribe('saleTracking updateTracking()', () => {
             req: {
                 accessToken: {userId: 104},
                 headers: {origin: 'http://localhost'},
+                __: value => value
             }
         };
     });
 
+    it('should add a new saleTracking and saleBuy', async() => {
+        const tx = await models.SaleTracking.beginTransaction({});
+        const options = {transaction: tx};
+        const code = 'FAKESTATE';
+        try {
+            await models.SaleTracking.updateTracking(
+                ctx,
+                saleFk,
+                originalQuantity,
+                code,
+                isChecked,
+                buyFk,
+                isScanned,
+                options
+            );
+
+            await tx.rollback();
+        } catch (e) {
+            const error = e;
+
+            expect(error.message).toEqual('this state does not exist');
+            await tx.rollback();
+        }
+    });
+
     it('should add a new saleTracking and saleBuy', async() => {
         const tx = await models.SaleTracking.beginTransaction({});
         const options = {transaction: tx};
diff --git a/modules/ticket/back/methods/sale-tracking/updateTracking.js b/modules/ticket/back/methods/sale-tracking/updateTracking.js
index 7cb2b4a58..81eef28b4 100644
--- a/modules/ticket/back/methods/sale-tracking/updateTracking.js
+++ b/modules/ticket/back/methods/sale-tracking/updateTracking.js
@@ -1,3 +1,4 @@
+const UserError = require('vn-loopback/util/user-error');
 module.exports = Self => {
     Self.remoteMethodCtx('updateTracking', {
         description: 'Modify a saleTracking record and, if applicable, add a corresponding record in saleBuy.',
@@ -42,6 +43,7 @@ module.exports = Self => {
     Self.updateTracking = async(ctx, saleFk, originalQuantity, code, isChecked, buyFk, isScanned = null, options) => {
         const userId = ctx.req.accessToken.userId;
         const models = Self.app.models;
+        const $t = ctx.req.__;
         const myOptions = {};
         let tx;
 
@@ -58,6 +60,7 @@ module.exports = Self => {
                 where: {code},
             }, myOptions);
 
+            if (!state) throw new UserError($t('this state does not exist'));
             const uniqueAttributes = {
                 saleFk,
                 workerFk: userId,