From 018393fa93fec4d0642521852eb31686b569ac70 Mon Sep 17 00:00:00 2001
From: guillermo <guillermo@verdnatura.es>
Date: Wed, 13 Nov 2024 14:04:02 +0100
Subject: [PATCH] feat: refs #8151 Added test

---
 .../methods/expedition/moveExpeditions.js     |  4 +-
 .../expedition/specs/moveExpeditions.spec.js  | 44 ++++++++++++++++++-
 2 files changed, 43 insertions(+), 5 deletions(-)

diff --git a/modules/ticket/back/methods/expedition/moveExpeditions.js b/modules/ticket/back/methods/expedition/moveExpeditions.js
index 3e9c95ec6..3ad41897f 100644
--- a/modules/ticket/back/methods/expedition/moveExpeditions.js
+++ b/modules/ticket/back/methods/expedition/moveExpeditions.js
@@ -82,9 +82,7 @@ module.exports = Self => {
             }
             await Promise.all(promises);
             await models.Ticket.updateAll({id: ticket.id}, {packages: promises.length}, myOptions);
-            const state = await models.State.findOne({
-                where: {code: 'PACKED'}
-            }, myOptions);
+            const state = await models.State.findOne({where: {code: 'PACKED'}});
 
             await models.Ticket.state(ctx, {
                 ticketFk: ticket.id,
diff --git a/modules/ticket/back/methods/expedition/specs/moveExpeditions.spec.js b/modules/ticket/back/methods/expedition/specs/moveExpeditions.spec.js
index c1c7c2c12..4c4971a0d 100644
--- a/modules/ticket/back/methods/expedition/specs/moveExpeditions.spec.js
+++ b/modules/ticket/back/methods/expedition/specs/moveExpeditions.spec.js
@@ -1,6 +1,6 @@
 const models = require('vn-loopback/server/server').models;
 
-describe('ticket moveExpeditions()', () => {
+fdescribe('ticket moveExpeditions()', () => {
     const ctx = beforeAll.getCtx();
 
     it('should move the selected expeditions to new ticket', async() => {
@@ -32,5 +32,45 @@ describe('ticket moveExpeditions()', () => {
             throw e;
         }
     });
-});
 
+    it('should move expeditions to a new ticket, set state to PACKED, and update packages count correctly', async() => {
+        const tx = await models.Expedition.beginTransaction({});
+        const myCtx = ctx;
+
+        try {
+            const options = {transaction: tx};
+            myCtx.args = {
+                clientId: 1101,
+                landed: Date.vnNew(),
+                warehouseId: 1,
+                addressId: 121,
+                agencyModeId: 1,
+                routeId: null,
+                expeditionIds: [1, 2]
+            };
+
+            const ticket = await models.Expedition.moveExpeditions(myCtx, options);
+
+            const newestTicketIdInFixtures = 27;
+
+            expect(ticket.id).toBeGreaterThan(newestTicketIdInFixtures);
+
+            const updatedTicket = await models.Ticket.findById(ticket.id, null, options);
+            const packedState = await models.State.findOne({where: {code: 'PACKED'}}, options);
+
+            const state = await models.TicketState.findOne({where: {ticketFk: ticket.id}}, options);
+
+            expect(state.stateFk).toBe(packedState.id);
+            expect(state.userFk).toBe(myCtx.req.accessToken.userId);
+
+            const expectedPackagesCount = myCtx.args.expeditionIds.length;
+
+            expect(updatedTicket.$packages).toBe(expectedPackagesCount);
+
+            await tx.rollback();
+        } catch (e) {
+            await tx.rollback();
+            throw e;
+        }
+    });
+});