From 8ab9063b50b4003b0cb1cea62b81a33deae3454f Mon Sep 17 00:00:00 2001
From: Javier Segarra <jsegarra@verdnatura.es>
Date: Mon, 1 Apr 2024 11:00:55 +0200
Subject: [PATCH] refs #6436 feat: use mocks

---
 .../collection/spec/getTickets.spec.js        | 10 +---------
 .../client/specs/createWithUser.spec.js       | 15 ++-------------
 .../invoice-in/specs/corrective.spec.js       | 15 +--------------
 .../item-shelving/specs/upsertItem.spec.js    | 16 ++--------------
 .../back/methods/sale/specs/clone.spec.js     | 19 +++++--------------
 .../ticket/specs/addSaleByCode.spec.js        | 17 +++--------------
 .../back/methods/ticket/specs/clone.spec.js   | 15 +--------------
 .../ticket/specs/transferClient.spec.js       | 11 ++---------
 8 files changed, 17 insertions(+), 101 deletions(-)

diff --git a/back/methods/collection/spec/getTickets.spec.js b/back/methods/collection/spec/getTickets.spec.js
index e6b9e6a17f..c066ead8c6 100644
--- a/back/methods/collection/spec/getTickets.spec.js
+++ b/back/methods/collection/spec/getTickets.spec.js
@@ -1,15 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('collection getTickets()', () => {
-    let ctx;
-    beforeAll(async() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'}
-            }
-        };
-    });
+    const {ctx} = beforeAll;
 
     it('should get tickets, sales and barcodes from collection', async() => {
         const tx = await models.Collection.beginTransaction({});
diff --git a/modules/client/back/methods/client/specs/createWithUser.spec.js b/modules/client/back/methods/client/specs/createWithUser.spec.js
index 074cb289a9..584c49d5ce 100644
--- a/modules/client/back/methods/client/specs/createWithUser.spec.js
+++ b/modules/client/back/methods/client/specs/createWithUser.spec.js
@@ -1,5 +1,5 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
+const {mockLoopBackContext} = require('vn-loopback/../../back/vn-jasmine');
 
 describe('Client Create', () => {
     const newAccount = {
@@ -17,18 +17,7 @@ describe('Client Create', () => {
     delete newAccountWithoutEmail.email;
 
     beforeAll(async() => {
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
+        mockLoopBackContext();
     });
 
     it(`should not find Deadpool as he's not created yet`, async() => {
diff --git a/modules/invoiceIn/back/methods/invoice-in/specs/corrective.spec.js b/modules/invoiceIn/back/methods/invoice-in/specs/corrective.spec.js
index 1047cd0283..ff547f1c86 100644
--- a/modules/invoiceIn/back/methods/invoice-in/specs/corrective.spec.js
+++ b/modules/invoiceIn/back/methods/invoice-in/specs/corrective.spec.js
@@ -1,24 +1,11 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('invoiceIn corrective()', () => {
-    let ctx;
+    const {ctx} = beforeAll;
     let options;
     let tx;
 
     beforeEach(async() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'}
-            },
-            args: {}
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: ctx.req
-        });
-
         options = {transaction: tx};
         tx = await models.Sale.beginTransaction({});
         options.transaction = tx;
diff --git a/modules/item/back/methods/item-shelving/specs/upsertItem.spec.js b/modules/item/back/methods/item-shelving/specs/upsertItem.spec.js
index 9042b743db..e7f2ab0154 100644
--- a/modules/item/back/methods/item-shelving/specs/upsertItem.spec.js
+++ b/modules/item/back/methods/item-shelving/specs/upsertItem.spec.js
@@ -1,26 +1,14 @@
 const {models} = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 // #6276
 describe('ItemShelving upsertItem()', () => {
     const warehouseFk = 1;
-    let ctx;
+
+    const {ctx} = beforeAll;
     let options;
     let tx;
 
     beforeEach(async() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'}
-            },
-            args: {}
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: ctx.req
-        });
-
         options = {transaction: tx};
         tx = await models.ItemShelving.beginTransaction({});
         options.transaction = tx;
diff --git a/modules/ticket/back/methods/sale/specs/clone.spec.js b/modules/ticket/back/methods/sale/specs/clone.spec.js
index e2220c028f..ef2c6a9a3c 100644
--- a/modules/ticket/back/methods/sale/specs/clone.spec.js
+++ b/modules/ticket/back/methods/sale/specs/clone.spec.js
@@ -1,24 +1,15 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
+const {mockLoopBackContext, mockBeforeAll} = require('vn-loopback/../../back/vn-jasmine');
 
 describe('Ticket cloning - clone function', () => {
-    let ctx;
+    let ctx = mockBeforeAll({ejemploe: true});
     let options;
     let tx;
+    beforeAll(async() => {
+        mockLoopBackContext();
+    });
 
     beforeEach(async() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'}
-            },
-            args: {}
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: ctx.req
-        });
-
         options = {transaction: tx};
         tx = await models.Sale.beginTransaction({});
         options.transaction = tx;
diff --git a/modules/ticket/back/methods/ticket/specs/addSaleByCode.spec.js b/modules/ticket/back/methods/ticket/specs/addSaleByCode.spec.js
index b97139178a..29af667525 100644
--- a/modules/ticket/back/methods/ticket/specs/addSaleByCode.spec.js
+++ b/modules/ticket/back/methods/ticket/specs/addSaleByCode.spec.js
@@ -1,22 +1,11 @@
 const {models} = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('Ticket addSaleByCode()', () => {
     const quantity = 3;
     const ticketFk = 13;
     const warehouseFk = 1;
-    beforeAll(async() => {
-        activeCtx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'},
-                __: value => value
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
+
+    const {ctx} = beforeAll;
 
     it('should add a new sale', async() => {
         const tx = await models.Ticket.beginTransaction({});
@@ -26,7 +15,7 @@ describe('Ticket addSaleByCode()', () => {
             const code = '1111111111';
 
             const salesBefore = await models.Sale.find(null, options);
-            await models.Ticket.addSaleByCode(activeCtx, code, quantity, ticketFk, warehouseFk, options);
+            await models.Ticket.addSaleByCode(ctx, code, quantity, ticketFk, warehouseFk, options);
             const salesAfter = await models.Sale.find(null, options);
 
             expect(salesAfter.length).toEqual(salesBefore.length + 1);
diff --git a/modules/ticket/back/methods/ticket/specs/clone.spec.js b/modules/ticket/back/methods/ticket/specs/clone.spec.js
index 26114bd58e..cf222ab8d2 100644
--- a/modules/ticket/back/methods/ticket/specs/clone.spec.js
+++ b/modules/ticket/back/methods/ticket/specs/clone.spec.js
@@ -1,26 +1,13 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('Ticket cloning - clone function', () => {
-    let ctx;
+    const {ctx} = beforeAll;
     let options;
     let tx;
     const ticketId = 1;
     const shipped = Date.vnNew();
 
     beforeEach(async() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'}
-            },
-            args: {}
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: ctx.req
-        });
-
         options = {transaction: tx};
         tx = await models.Ticket.beginTransaction({});
         options.transaction = tx;
diff --git a/modules/ticket/back/methods/ticket/specs/transferClient.spec.js b/modules/ticket/back/methods/ticket/specs/transferClient.spec.js
index 5f1c097760..4aac0833a4 100644
--- a/modules/ticket/back/methods/ticket/specs/transferClient.spec.js
+++ b/modules/ticket/back/methods/ticket/specs/transferClient.spec.js
@@ -4,18 +4,11 @@ describe('Ticket transferClient()', () => {
     const originalTicketId = 8;
     const refundTicketId = 24;
     const clientId = 1;
-    let ctx;
+
+    const {ctx} = beforeAll;
     let options;
     let tx;
     beforeEach(async() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'}
-            },
-            args: {}
-        };
-
         options = {transaction: tx};
         tx = await models.Ticket.beginTransaction({});
         options.transaction = tx;