CHANGELOG.md

@@ -22,6 +22,7 @@
 - feat: refs #7438 Added volume to item_valuateInventory  by:guillermo
 - feat: refs #7438 Requested changes and little changes  by:guillermo
 - refs #6281 feat:buyFk in itemShelving  by:sergiodt
+- feat: refs #6449 item ID is displayed in the sale line  by:jorgep
 
 ### Changed 📦
 

back/methods/collection/spec/getSales.spec.js

@@ -4,15 +4,7 @@ describe('collection getSales()', () => {
     const collectionOrTicketFk = 999999;
     const print = true;
     const source = 'CHECKER';
-
+    const ctx = beforeAll.getCtx();
-    beforeAll(() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'},
-            }
-        };
-    });
 
     it('should return a collection with tickets, placements and barcodes settled correctly', async() => {
         const tx = await models.Collection.beginTransaction({});

back/methods/collection/spec/getTickets.spec.js

@@ -1,15 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('collection getTickets()', () => {
-    let ctx;
+    const ctx = beforeAll.getCtx();
-    beforeAll(async() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'}
-            }
-        };
-    });
 
     it('should get tickets, sales and barcodes from collection', async() => {
         const tx = await models.Collection.beginTransaction({});

back/methods/collection/spec/setSaleQuantity.spec.js

@@ -1,20 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('setSaleQuantity()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should change quantity sale', async() => {
         const tx = await models.Ticket.beginTransaction({});

back/methods/starred-module/specs/getStarredModules.spec.js

@@ -1,22 +1,7 @@
 const {models} = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('getStarredModules()', () => {
-    const activeCtx = {
+    const ctx = beforeAll.getCtx();
-        accessToken: {userId: 9},
-        http: {
-            req: {
-                headers: {origin: 'http://localhost'}
-            }
-        }
-    };
-    const ctx = {req: activeCtx};
-
-    beforeEach(() => {
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it(`should return the starred modules for a given user`, async() => {
         const newStarred = await models.StarredModule.create({workerFk: 9, moduleFk: 'customer', position: 1});

back/methods/starred-module/specs/setPosition.spec.js

@@ -1,24 +1,8 @@
 const {models} = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('setPosition()', () => {
-    const activeCtx = {
+    const ctx = beforeAll.getCtx();
-        accessToken: {userId: 9},
+    beforeAll.mockLoopBackContext();
-        http: {
-            req: {
-                headers: {origin: 'http://localhost'}
-            }
-        }
-    };
-    const ctx = {
-        req: activeCtx
-    };
-
-    beforeEach(() => {
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should increase the orders module position by replacing it with clients and vice versa', async() => {
         const tx = await models.StarredModule.beginTransaction({});

back/methods/starred-module/specs/toggleStarredModule.spec.js

@@ -1,24 +1,7 @@
 const {models} = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('toggleStarredModule()', () => {
-    const activeCtx = {
+    const ctx = beforeAll.getCtx();
-        accessToken: {userId: 9},
-        http: {
-            req: {
-                headers: {origin: 'http://localhost'}
-            }
-        }
-    };
-    const ctx = {
-        req: activeCtx
-    };
-
-    beforeEach(() => {
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should create a new starred module and then remove it by calling the method again with same args', async() => {
         const starredModule = await models.StarredModule.toggleStarredModule(ctx, 'order');
@@ -26,7 +9,7 @@ describe('toggleStarredModule()', () => {
 
         expect(starredModules.length).toEqual(1);
         expect(starredModule.moduleFk).toEqual('order');
-        expect(starredModule.workerFk).toEqual(activeCtx.accessToken.userId);
+        expect(starredModule.workerFk).toEqual(ctx.req.accessToken.userId);
         expect(starredModule.position).toEqual(starredModules.length);
 
         await models.StarredModule.toggleStarredModule(ctx, 'order');

back/methods/user-config/specs/getUserConfig.spec.js

@@ -1,12 +1,12 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('userConfig getUserConfig()', () => {
+    const ctx = beforeAll.getCtx();
     it(`should return the configuration data of a given user`, async() => {
         const tx = await models.Item.beginTransaction({});
         const options = {transaction: tx};
 
         try {
-            const ctx = {req: {accessToken: {userId: 9}}};
             const result = await models.UserConfig.getUserConfig(ctx, options);
 
             expect(result.warehouseFk).toEqual(1);

back/models/specs/mailAliasAccount.spec.js

@@ -1,12 +1,13 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('loopback model MailAliasAccount', () => {
+    const ctx = beforeAll.getCtx();
     it('should add a mail Alias', async() => {
         const tx = await models.MailAliasAccount.beginTransaction({});
         let error;
 
         try {
-            const options = {transaction: tx, accessToken: {userId: 9}};
+            const options = {transaction: tx, ctx};
             await models.MailAliasAccount.create({mailAlias: 2, account: 5}, options);
 
             await tx.rollback();
@@ -23,7 +24,7 @@ describe('loopback model MailAliasAccount', () => {
         let error;
 
         try {
-            const options = {transaction: tx, accessToken: {userId: 9}};
+            const options = {transaction: tx, ctx};
             await models.MailAliasAccount.create({mailAlias: 3, account: 5}, options);
 
             await tx.rollback();

back/models/warehouse.json

@@ -9,24 +9,24 @@
   },
   "properties": {
     "id": {
-        "id": true,
+      "id": true,
-        "type": "number",
+      "type": "number",
-        "forceId": false
+      "forceId": false
     },
     "name": {
-        "type": "string"
+      "type": "string"
     },
     "code": {
-        "type": "string"
+      "type": "string"
     },
     "isInventory": {
-        "type": "number"
+      "type": "number"
     },
-    "isManaged":{
+    "isManaged": {
-        "type": "boolean"
+      "type": "boolean"
     },
     "countryFk": {
-        "type": "number"
+      "type": "number"
     }
   },
   "relations": {
@@ -34,6 +34,11 @@
       "type": "belongsTo",
       "model": "Country",
       "foreignKey": "countryFk"
+    },
+    "address": {
+      "type": "belongsTo",
+      "model": "Address",
+      "foreignKey": "addressFk"
     }
   },
   "acls": [
@@ -44,5 +49,11 @@
       "permission": "ALLOW"
     }
   ],
-  "scope" : {"where": {"isForTicket": {"neq": 0}}}
+  "scope": {
+    "where": {
+      "isForTicket": {
+        "neq": 0
+      }
+    }
+  }
 }

back/tests-helper.js

@@ -10,7 +10,6 @@ async function init() {
         host: process.env.DB_HOST,
         port: process.env.DB_PORT
     });
-
     const bootOptions = {dataSources};
     await new Promise((resolve, reject) => {
         app.boot(bootOptions,
@@ -33,3 +32,4 @@ module.exports = {
 
 if (require.main === module)
     init();
+

back/tests.js

@@ -84,7 +84,7 @@ async function test() {
             'loopback/**/*[sS]pec.js',
             'modules/*/back/**/*.[sS]pec.js'
         ],
-        helpers: []
+        helpers: [`back/vn-jasmine.js`],
     };
 
     if (PARALLEL) {
@@ -114,7 +114,6 @@ async function test() {
     if (opts.ci)
         runner.jasmine.DEFAULT_TIMEOUT_INTERVAL = SPEC_TIMEOUT;
 
-    // runner.loadConfigFile('back/jasmine.json');
     runner.loadConfig(config);
     process.env.SPEC_IS_RUNNING = true;
     await runner.execute();

back/vn-jasmine.js

@@ -0,0 +1,48 @@
+
+const LoopBackContext = require('loopback-context');
+const getAccessToken = (userId = 9) => {
+    return {accessToken: {userId}};
+};
+const DEFAULT_HEADERS = {headers: {origin: 'http://localhost'}};
+const default_before_all = userId => {
+    return {
+        req: {
+            ...getAccessToken(userId),
+            ...DEFAULT_HEADERS,
+            ...{__: value => value}
+
+        },
+        args: {}
+    };
+};
+const default_loopback_ctx = userId => {
+    return {
+        ...getAccessToken(userId),
+        ...default_before_all(userId),
+        http: {
+            ...default_before_all(userId)
+        },
+        args: {}
+    };
+};
+
+function vnBeforeAll() {
+    Object.assign(beforeAll, {getCtx: default_before_all, mockLoopBackContext});
+}
+
+const mockLoopBackContext = userId => {
+    const activeCtx = default_loopback_ctx(userId);
+    beforeAll(() => {
+        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
+            active: activeCtx
+        });
+    });
+    return activeCtx;
+};
+module.exports = {
+    mockLoopBackContext
+};
+
+(function init() {
+    vnBeforeAll();
+})();

db/dump/fixtures.before.sql

@@ -3911,4 +3911,3 @@ VALUES(1, '');
 INSERT INTO dipole.expedition_PrintOut (expeditionFk, ticketFk, addressFk, street, postalCode, city, shopName, isPrinted, created, printerFk, routeFk, parkingCode,
 truckName, clientFk, phone, province, agency, m3, workerCode, itemFk, quantity, longName, shelvingFk, comments)
 VALUES(1, 1, 0, ' ', ' ', ' ', ' ', 0, '2001-01-01 00:00:00', 1, 0, ' ', ' ', 0, NULL, '', NULL, 0.000, NULL, 10, NULL, NULL, 'NCC', NULL);
-

db/versions/10986-pinkChrysanthemum/00-addSummaryId.sql

@@ -0,0 +1,22 @@
+ALTER TABLE vn.claimLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.clientLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.deviceProductionLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.entryLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.invoiceInLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.itemLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.packingSiteDeviceLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.parkingLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.rateLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.routeLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.shelvingLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.supplierLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.ticketLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.travelLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.userLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.workerLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE vn.zoneLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+
+ALTER TABLE account.userLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE account.signInLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE account.roleLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);
+ALTER TABLE account.accountLog ADD COLUMN IF NOT EXISTS summaryId varchar(30);

db/versions/11081-wheatRaphis/00-firstScript.sql

@@ -0,0 +1,8 @@
+-- Place your SQL code here
+
+USE vn;
+
+ALTER TABLE vn.roadmap ADD kmStart mediumint(9) DEFAULT NULL NULL;
+ALTER TABLE vn.roadmap ADD kmEnd mediumint(9) DEFAULT NULL NULL;
+ALTER TABLE vn.roadmap ADD started DATETIME NULL;
+ALTER TABLE vn.roadmap ADD finished DATETIME NULL;

db/versions/11118-limeCymbidium/00-firstScript.sql

@@ -0,0 +1,21 @@
+
+INSERT IGNORE INTO salix.ACL (`model`,`property`,`accessType`,`permission`,`principalType`,`principalId`)
+	VALUES ('Entry','filter','READ','ALLOW','ROLE','supplier');
+
+INSERT IGNORE INTO salix.ACL (`model`,`property`,`accessType`,`permission`,`principalType`,`principalId`)
+	VALUES ('Entry','getBuys','READ','ALLOW','ROLE','supplier');
+
+INSERT IGNORE INTO salix.ACL (`model`,`property`,`accessType`,`permission`,`principalType`,`principalId`)
+	VALUES ('Entry','buyLabel','READ','ALLOW','ROLE','supplier');
+
+INSERT IGNORE INTO `account`.`role` (`name`,`description`,`hasLogin`,`created`,`modified`)
+	VALUES ('supplier','Proveedores',1,'2017-10-10 14:58:58.000','2017-10-10 14:59:20.000');
+SET @supplierFk =LAST_INSERT_ID();
+INSERT IGNORE INTO account.roleInherit (`role`,`inheritsFrom`)
+	VALUES (@supplierFk,2);
+
+UPDATE salix.ACL
+	SET principalId='$authenticated'
+	WHERE id=264;
+
+

db/versions/11128-turquoiseCymbidium/00-firstScript.sql

@@ -0,0 +1 @@
+ALTER TABLE vn.calendar ADD created timestamp DEFAULT current_timestamp() NOT NULL AFTER dated;

loopback/common/methods/application/spec/execute.spec.js

@@ -2,18 +2,9 @@ const models = require('vn-loopback/server/server').models;
 
 describe('Application execute()/executeProc()/executeFunc()', () => {
     const userWithoutPrivileges = 1;
-    const userWithPrivileges = 9;
     const userWithInheritedPrivileges = 120;
     let tx;
-
+    const ctx = beforeAll.getCtx();
-    function getCtx(userId) {
-        return {
-            req: {
-                accessToken: {userId},
-                headers: {origin: 'http://localhost'}
-            }
-        };
-    }
 
     beforeEach(async() => {
         tx = await models.Application.beginTransaction({});
@@ -42,7 +33,7 @@ describe('Application execute()/executeProc()/executeFunc()', () => {
     });
 
     it('should throw error when execute procedure and not have privileges', async() => {
-        const ctx = getCtx(userWithoutPrivileges);
+        const ctx = {req: {accessToken: {userId: userWithoutPrivileges}}};
 
         let error;
         try {
@@ -66,7 +57,6 @@ describe('Application execute()/executeProc()/executeFunc()', () => {
     });
 
     it('should execute procedure and get data', async() => {
-        const ctx = getCtx(userWithPrivileges);
         try {
             const options = {transaction: tx};
 
@@ -90,7 +80,6 @@ describe('Application execute()/executeProc()/executeFunc()', () => {
 
     describe('Application executeProc()', () => {
         it('should execute procedure and get data (executeProc)', async() => {
-            const ctx = getCtx(userWithPrivileges);
             try {
                 const options = {transaction: tx};
 
@@ -115,7 +104,6 @@ describe('Application execute()/executeProc()/executeFunc()', () => {
 
     describe('Application executeFunc()', () => {
         it('should execute function and get data', async() => {
-            const ctx = getCtx(userWithPrivileges);
             try {
                 const options = {transaction: tx};
 
@@ -137,7 +125,7 @@ describe('Application execute()/executeProc()/executeFunc()', () => {
         });
 
         it('should execute function and get data with user with inherited privileges', async() => {
-            const ctx = getCtx(userWithInheritedPrivileges);
+            const ctx = {req: {accessToken: {userId: userWithInheritedPrivileges}}};
             try {
                 const options = {transaction: tx};
 

loopback/common/methods/schema/model-info.js

@@ -92,6 +92,7 @@ module.exports = Self => {
             const locale = modelLocale && modelLocale.get(lang);
 
             json[modelName] = {
+                http: model.sharedClass.http.path,
                 properties: model.definition.rawProperties,
                 validations: jsonValidations,
                 locale

loopback/common/methods/vn-model/specs/crud.spec.js

@@ -1,20 +1,7 @@
 const app = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('Model crud()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
     let insertId;
     const barcodeModel = app.models.ItemBarcode;
 

loopback/common/methods/vn-model/specs/rewriteDbError.spec.js

@@ -1,20 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('Model rewriteDbError()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should extend rewriteDbError properties to any model passed', () => {
         const exampleModel = models.ItemTag;

modules/claim/back/methods/claim/specs/filter.spec.js

@@ -2,15 +2,7 @@ const app = require('vn-loopback/server/server');
 const models = require('vn-loopback/server/server').models;
 
 describe('claim filter()', () => {
-    let ctx;
+    const ctx = beforeAll.getCtx();
-    beforeEach(() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 9},
-                headers: {origin: 'http://localhost'}
-            }
-        };
-    });
 
     it('should return 1 result filtering by id', async() => {
         const tx = await app.models.Claim.beginTransaction({});

modules/claim/back/methods/claim/specs/regularizeClaim.spec.js

@@ -1,18 +1,11 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('claim regularizeClaim()', () => {
     const userId = 18;
-    const ctx = {
+    const ctx = beforeAll.mockLoopBackContext(userId);
-        req: {
-            accessToken: {userId: userId},
-            headers: {origin: 'http://localhost'}
-        }
-    };
     ctx.req.__ = (value, params) => {
         return params.nickname;
     };
-
     const chatModel = models.Chat;
     const claimId = 1;
     const ticketId = 1;
@@ -40,20 +33,6 @@ describe('claim regularizeClaim()', () => {
         return await models.ClaimEnd.create(claimEnds, options);
     }
 
-    beforeAll(async() => {
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
-
     it('should send a chat message with value "Trash" and then change claim state to resolved', async() => {
         const tx = await models.Claim.beginTransaction({});
 

modules/claim/back/methods/claim/specs/updateClaim.spec.js

@@ -1,22 +1,10 @@
 const app = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 const i18n = require('i18n');
 describe('Update Claim', () => {
     let url;
     let claimStatesMap = {};
+    beforeAll.mockLoopBackContext();
     beforeAll(async() => {
-        url = await app.models.Url.getUrl();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
         const claimStates = await app.models.ClaimState.find();
         claimStatesMap = claimStates.reduce((acc, state) => ({...acc, [state.code]: state.id}), {});
     });

modules/claim/back/methods/claim/specs/updateClaimAction.spec.js

@@ -1,20 +1,7 @@
 const app = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('Update Claim', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
     const newDate = Date.vnNew();
     const original = {
         ticketFk: 3,

modules/client/back/methods/client/specs/addressesPropagateRe.spec.js

@@ -1,23 +1,10 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('Client addressesPropagateRe', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
 
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
+    it('should propagate the isEqualizated on both addresses of Mr Wayne' +
-            active: activeCtx
+        ' and set hasToInvoiceByAddress to false', async() => {
-        });
-    });
-
-    it('should propagate the isEqualizated on both addresses of Mr Wayne and set hasToInvoiceByAddress to false', async() => {
         const tx = await models.Client.beginTransaction({});
 
         try {

modules/client/back/methods/client/specs/createAddress.spec.js

@@ -1,5 +1,4 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('Address createAddress', () => {
     const clientFk = 1101;
@@ -7,20 +6,7 @@ describe('Address createAddress', () => {
     const incotermsFk = 'FAS';
     const customAgentOneId = 1;
 
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should throw a non uee member error if no incoterms is defined', async() => {
         const tx = await models.Client.beginTransaction({});

modules/client/back/methods/client/specs/createWithUser.spec.js

@@ -1,6 +1,4 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
-
 describe('Client Create', () => {
     const newAccount = {
         userName: 'deadpool',
@@ -16,20 +14,7 @@ describe('Client Create', () => {
     const newAccountWithoutEmail = JSON.parse(JSON.stringify(newAccount));
     delete newAccountWithoutEmail.email;
 
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it(`should not find deadpool as he's not created yet`, async() => {
         const tx = await models.Client.beginTransaction({});

modules/client/back/methods/client/specs/getCard.spec.js

@@ -1,11 +1,11 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('Client getCard()', () => {
+    const ctx = beforeAll.getCtx();
     it('should receive a formated card of Bruce Wayne', async() => {
         const tx = await models.Client.beginTransaction({});
 
         try {
-            const ctx = {req: {accessToken: {userId: 9}}};
             const options = {transaction: tx};
 
             const id = 1101;

modules/client/back/methods/client/specs/getDebt.spec.js

@@ -1,9 +1,9 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('client getDebt()', () => {
+    const ctx = beforeAll.getCtx();
     it('should return the client debt', async() => {
         const tx = await models.Client.beginTransaction({});
-        const ctx = {req: {accessToken: {userId: 9}}};
 
         try {
             const options = {transaction: tx};

modules/client/back/methods/client/specs/sendSms.spec.js

@@ -1,12 +1,12 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('client sendSms()', () => {
+    const ctx = beforeAll.getCtx();
     it('should now send a message and log it', async() => {
         const tx = await models.Client.beginTransaction({});
 
         try {
             const options = {transaction: tx};
-            const ctx = {req: {accessToken: {userId: 9}}};
             const id = 1101;
             const destination = 222222222;
             const message = 'this is the message created in a test';

modules/client/back/methods/client/specs/summary.spec.js

@@ -1,7 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('client summary()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
     it('should return a summary object containing data', async() => {
         const clientId = 1101;
         const tx = await models.Client.beginTransaction({});

modules/client/back/methods/client/specs/updateAddress.spec.js

@@ -1,6 +1,4 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
-
 describe('Address updateAddress', () => {
     const clientId = 1101;
     const addressId = 1;
@@ -14,20 +12,7 @@ describe('Address updateAddress', () => {
         }
     };
 
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should throw the non uee member error if no incoterms is defined', async() => {
         const tx = await models.Client.beginTransaction({});

modules/client/back/models/specs/address.spec.js

@@ -1,24 +1,10 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('loopback model address', () => {
     let createdAddressId;
     const clientId = 1101;
 
-    const activeCtx = {
+    beforeAll.mockLoopBackContext();
-        accessToken: {userId: 9},
-        http: {
-            req: {
-                headers: {origin: 'http://localhost'}
-            }
-        }
-    };
-
-    beforeAll(() => {
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     afterAll(async() => {
         const client = await models.Client.findById(clientId);

modules/entry/back/methods/entry/buyLabel.js

@@ -29,7 +29,8 @@ module.exports = Self => {
         http: {
             path: '/:id/buy-label',
             verb: 'GET'
-        }
+        },
+        accessScopes: ['DEFAULT', 'read:multimedia']
     });
 
     Self.buyLabel = (ctx, id) => Self.printReport(ctx, id, 'buy-label');

modules/entry/back/methods/entry/filter.js

@@ -112,7 +112,6 @@ module.exports = Self => {
 
         if (typeof options == 'object')
             Object.assign(myOptions, options);
-
         const conn = Self.dataSource.connector;
         const where = buildFilter(ctx.args, (param, value) => {
             switch (param) {
@@ -146,7 +145,12 @@ module.exports = Self => {
             }
         });
         filter = mergeFilters(ctx.args.filter, {where});
-
+        const userId = ctx.req.accessToken.userId;
+        const isSupplier = await Self.app.models.Supplier.findById(userId, options);
+        if (isSupplier) {
+            if (!filter.where) filter.where = {};
+            filter.where[`e.supplierFk`] = ctx.req.accessToken.userId;
+        }
         const stmts = [];
         let stmt;
         stmt = new ParameterizedSQL(
@@ -158,7 +162,7 @@ module.exports = Self => {
                     e.invoiceNumber,
                     e.isBooked,
                     e.isExcludedFromAvailable,
-                    e.evaNotes AS observation,
+                    e.evaNotes observation,
                     e.isConfirmed,
                     e.isOrdered,
                     e.isRaid,
@@ -170,14 +174,19 @@ module.exports = Self => {
                     e.gestDocFk,
                     e.invoiceInFk,
                     t.landed,
-                    s.name AS supplierName,
+                    s.name supplierName,
-                    s.nickname AS supplierAlias,
+                    s.nickname supplierAlias,
-                    co.code AS companyCode,
+                    co.code companyCode,
-                    cu.code AS currencyCode,
+                    cu.code currencyCode,
+                    t.shipped,
+                    t.landed,
                     t.ref AS travelRef
+                    t.warehouseInFk,
+                    w.name warehouseInName
                 FROM vn.entry e
                     JOIN vn.supplier s ON s.id = e.supplierFk
                     JOIN vn.travel t ON t.id = e.travelFk
+                    JOIN vn.warehouse w ON w.id = t.warehouseInFk
                     JOIN vn.company co ON co.id = e.companyFk
                     JOIN vn.currency cu ON cu.id = e.currencyFk`
         );

modules/entry/back/methods/entry/getBuys.js

@@ -1,7 +1,9 @@
+const UserError = require('vn-loopback/util/user-error');
+
 const mergeFilters = require('vn-loopback/util/filter').mergeFilters;
 
 module.exports = Self => {
-    Self.remoteMethod('getBuys', {
+    Self.remoteMethodCtx('getBuys', {
         description: 'Returns buys for one entry',
         accessType: 'READ',
         accepts: [{
@@ -27,13 +29,19 @@ module.exports = Self => {
         }
     });
 
-    Self.getBuys = async(id, filter, options) => {
+    Self.getBuys = async(ctx, id, filter, options) => {
+        const userId = ctx.req.accessToken.userId;
         const models = Self.app.models;
         const myOptions = {};
 
         if (typeof options == 'object')
             Object.assign(myOptions, options);
+        const isSupplier = await Self.app.models.Supplier.findById(userId, myOptions);
+        if (isSupplier) {
+            const isEntryOwner = (await Self.findById(id)).supplierFk === userId;
 
+            if (!isEntryOwner) throw new UserError('Access Denied');
+        }
         let defaultFilter = {
             where: {entryFk: id},
             fields: [
@@ -49,9 +57,23 @@ module.exports = Self => {
                 'buyingValue',
                 'price2',
                 'price3',
-                'printedStickers'
+                'printedStickers',
+                'entryFk'
             ],
-            include: {
+            include: [{
+                relation: 'entry',
+                scope: {
+                    fields: [
+                        'id', 'supplierFk'
+                    ],
+                    include: {
+                        relation: 'supplier', scope: {
+                            fields: ['id']
+                        }
+                    }
+                }
+            },
+            {
                 relation: 'item',
                 scope: {
                     fields: [
@@ -82,9 +104,8 @@ module.exports = Self => {
                         }
                     }
                 }
-            }
+            }]
         };
-
         defaultFilter = mergeFilters(defaultFilter, filter);
 
         return models.Buy.find(defaultFilter, myOptions);

modules/entry/back/methods/entry/specs/addFromPackaging.spec.js

@@ -1,5 +1,4 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('entry addFromPackaging()', () => {
     const supplier = 442;
@@ -7,22 +6,10 @@ describe('entry addFromPackaging()', () => {
     const yesterday = new Date(today);
     yesterday.setDate(today.getDate() - 1);
 
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 49},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'},
-                },
-            },
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx,
-        });
-    });
 
     it('should create an incoming travel', async() => {
-        const ctx = {args: {isTravelReception: true, supplier}};
+        const ctx = {accessToken: {userId: 49}, args: {isTravelReception: true, supplier}};
         const tx = await models.Entry.beginTransaction({});
         const options = {transaction: tx};
 

modules/entry/back/methods/entry/specs/editLatestBuys.spec.js

@@ -1,21 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('Buy editLatestsBuys()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should change the value of a given column for the selected buys', async() => {
         const tx = await models.Buy.beginTransaction({});

modules/entry/back/methods/entry/specs/filter.spec.js

@@ -9,7 +9,8 @@ describe('Entry filter()', () => {
             const ctx = {
                 args: {
                     search: 1
-                }
+                },
+                req: {accessToken: {userId: 9}}
             };
 
             const result = await models.Entry.filter(ctx, options);
@@ -32,7 +33,8 @@ describe('Entry filter()', () => {
             const ctx = {
                 args: {
                     currencyFk: 1
-                }
+                },
+                req: {accessToken: {userId: 9}}
             };
 
             const result = await models.Entry.filter(ctx, options);
@@ -46,26 +48,73 @@ describe('Entry filter()', () => {
         }
     });
 
-    it('should return the entry matching the supplier', async() => {
+    describe('should return the entry matching the supplier', () => {
-        const tx = await models.Entry.beginTransaction({});
+        it('when userId is supplier ', async() => {
-        const options = {transaction: tx};
+            const tx = await models.Entry.beginTransaction({});
+            const options = {transaction: tx};
 
-        try {
+            try {
-            const ctx = {
+                const ctx = {
-                args: {
+                    args: {},
-                    supplierFk: 2
+                    req: {accessToken: {userId: 2}}
-                }
+                };
-            };
 
-            const result = await models.Entry.filter(ctx, options);
+                const result = await models.Entry.filter(ctx, options);
 
-            expect(result.length).toEqual(6);
+                expect(result.length).toEqual(6);
 
-            await tx.rollback();
+                await tx.rollback();
-        } catch (e) {
+            } catch (e) {
-            await tx.rollback();
+                await tx.rollback();
-            throw e;
+                throw e;
-        }
+            }
+        });
+
+        it('when userId is supplier fetching other supplier', async() => {
+            const tx = await models.Entry.beginTransaction({});
+            const options = {transaction: tx};
+
+            try {
+                const ctx = {
+                    args: {
+                        supplierFk: 1
+                    },
+                    req: {accessToken: {userId: 2}}
+                };
+
+                const result = await models.Entry.filter(ctx, options);
+
+                expect(result.length).toEqual(6);
+
+                await tx.rollback();
+            } catch (e) {
+                await tx.rollback();
+                throw e;
+            }
+        });
+
+        it('when userId is not supplier', async() => {
+            const tx = await models.Entry.beginTransaction({});
+            const options = {transaction: tx};
+
+            try {
+                const ctx = {
+                    args: {
+                        supplierFk: 2
+                    },
+                    req: {accessToken: {userId: 9}}
+                };
+
+                const result = await models.Entry.filter(ctx, options);
+
+                expect(result.length).toEqual(6);
+
+                await tx.rollback();
+            } catch (e) {
+                await tx.rollback();
+                throw e;
+            }
+        });
     });
 
     it('should return the entry matching the company', async() => {
@@ -76,7 +125,8 @@ describe('Entry filter()', () => {
             const ctx = {
                 args: {
                     companyFk: 442
-                }
+                },
+                req: {accessToken: {userId: 9}}
             };
 
             const result = await models.Entry.filter(ctx, options);
@@ -98,7 +148,8 @@ describe('Entry filter()', () => {
             const ctx = {
                 args: {
                     isBooked: true,
-                }
+                },
+                req: {accessToken: {userId: 9}}
             };
 
             const result = await models.Entry.filter(ctx, options);
@@ -121,7 +172,8 @@ describe('Entry filter()', () => {
                 args: {
                     reference: 'movement',
                     travelFk: '2'
-                }
+                },
+                req: {accessToken: {userId: 9}}
             };
 
             const result = await models.Entry.filter(ctx, options);

modules/entry/back/methods/entry/specs/getBuys.spec.js

@@ -1,24 +1,82 @@
+const UserError = require('vn-loopback/util/user-error');
 const models = require('vn-loopback/server/server').models;
 
 describe('entry getBuys()', () => {
     const entryId = 4;
-    it('should get the buys and items of an entry', async() => {
+    describe('should get the buys and items of an entry ', () => {
-        const tx = await models.Entry.beginTransaction({});
+        it('when is supplier and entry owner', async() => {
-        const options = {transaction: tx};
+            const tx = await models.Entry.beginTransaction({});
+            const options = {transaction: tx};
 
-        try {
+            try {
-            const result = await models.Entry.getBuys(entryId, options);
+                const ctx = {
+                    args: {
+                        search: 1
+                    },
+                    req: {accessToken: {userId: 2}}
+                };
 
-            const length = result.length;
+                const result = await models.Entry.getBuys(ctx, entryId, options);
-            const anyResult = result[Math.floor(Math.random() * Math.floor(length))];
 
-            expect(result.length).toEqual(4);
+                const length = result.length;
-            expect(anyResult.item).toBeDefined();
+                const anyResult = result[Math.floor(Math.random() * Math.floor(length))];
 
-            await tx.rollback();
+                expect(result.length).toEqual(4);
-        } catch (e) {
+                expect(anyResult.item).toBeDefined();
-            await tx.rollback();
+
-            throw e;
+                await tx.rollback();
-        }
+            } catch (e) {
+                await tx.rollback();
+                throw e;
+            }
+        });
+
+        it('when is supplier but not entry owner', async() => {
+            const tx = await models.Entry.beginTransaction({});
+            const options = {transaction: tx};
+            const entryId = 1;
+            try {
+                const ctx = {
+                    args: {
+                        search: 1
+                    },
+                    req: {accessToken: {userId: 2}}
+                };
+
+                const result = await models.Entry.getBuys(ctx, entryId, options);
+
+                expect(result).toBeUndefined();
+            } catch (error) {
+                expect(error).toBeInstanceOf(UserError);
+                expect(error.message).toBe('Access Denied');
+            }
+        });
+
+        it('when is not supplier', async() => {
+            const tx = await models.Entry.beginTransaction({});
+            const options = {transaction: tx};
+
+            try {
+                const ctx = {
+                    args: {
+                        search: 1
+                    },
+                    req: {accessToken: {userId: 9}}
+                };
+
+                const result = await models.Entry.getBuys(ctx, entryId, options);
+
+                const length = result.length;
+                const anyResult = result[Math.floor(Math.random() * Math.floor(length))];
+
+                expect(result.length).toEqual(4);
+                expect(anyResult.item).toBeDefined();
+
+                await tx.rollback();
+            } catch (e) {
+                await tx.rollback();
+                throw e;
+            }
+        });
     });
 });

modules/entry/back/methods/entry/specs/importBuysPreview.spec.js

@@ -1,14 +1,8 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
-const activeCtx = {accessToken: {userId: 9}};
 
 describe('entry importBuysPreview()', () => {
     const entryId = 1;
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should return the buys with the calculated packagingFk', async() => {
         const tx = await models.Entry.beginTransaction({});

modules/invoiceIn/back/methods/invoice-in-due-day/specs/new.spec.js

@@ -1,15 +1,7 @@
-const LoopBackContext = require('loopback-context');
 const models = require('vn-loopback/server/server').models;
 
 describe('invoiceInDueDay new()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should correctly create a new due day', async() => {
         const userId = 9;

modules/invoiceIn/back/methods/invoice-in/specs/clone.spec.js

@@ -1,24 +1,11 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('invoiceIn clone()', () => {
-    let ctx;
+    const ctx = beforeAll.getCtx();
     let options;
     let tx;
 
     beforeEach(async() => {
-        ctx = {
-            req: {
-                accessToken: {userId: 1},
-                headers: {origin: 'http://localhost'}
-            },
-            args: {}
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: ctx.req
-        });
-
         options = {transaction: tx};
         tx = await models.Sale.beginTransaction({});
         options.transaction = tx;
@@ -28,7 +15,8 @@ describe('invoiceIn clone()', () => {
         await tx.rollback();
     });
 
-    it('should return the cloned invoiceIn and also clone invoiceInDueDays and invoiceInTaxes if there are any referencing the invoiceIn', async() => {
+    it('should return the cloned invoiceIn and also clone invoiceInDueDays ' +
+        'and invoiceInTaxes if there are any referencing the invoiceIn', async() => {
         const clone = await models.InvoiceIn.clone(ctx, 1, false, options);
 
         expect(clone.supplierRef).toEqual('1234(2)');
@@ -51,7 +39,8 @@ describe('invoiceIn clone()', () => {
         expect(invoiceInDueDay.length).toEqual(2);
     });
 
-    it('should return the cloned invoiceIn and also clone invoiceInIntrastat and invoiceInTaxes if it is rectificative', async() => {
+    it('should return the cloned invoiceIn and also clone invoiceInIntrastat ' +
+        'and invoiceInTaxes if it is rectificative', async() => {
         const clone = await models.InvoiceIn.clone(ctx, 1, true, options);
 
         expect(clone.supplierRef).toEqual('1234(2)');

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.getCtx();
     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;

modules/invoiceOut/back/methods/invoiceOut/specs/book.spec.js

@@ -1,7 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('invoiceOut book()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
     const invoiceOutId = 5;
 
     it('should update the booked property', async() => {

modules/item/back/methods/fixed-price/specs/upsertFixedPrice.spec.js

@@ -1,24 +1,13 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('upsertFixedPrice()', () => {
     const now = Date.vnNew();
     const fixedPriceId = 1;
     let originalFixedPrice;
 
+    beforeAll.mockLoopBackContext();
     beforeAll(async() => {
         originalFixedPrice = await models.FixedPrice.findById(fixedPriceId);
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
     });
 
     it(`should toggle the hasMinPrice boolean if there's a minPrice and update the rest of the data`, async() => {

modules/item/back/methods/item-shelving/specs/getAlternative.spec.js

@@ -1,13 +1,7 @@
 const {models} = require('vn-loopback/server/server');
 
 describe('itemShelving getAlternative()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        ctx = {
-            req: {
-                headers: {origin: 'http://localhost'},
-            }
-        };
-    });
 
     it('should return a list of items without alternatives', async() => {
         const shelvingFk = 'HEJ';

modules/item/back/methods/item-shelving/specs/upsertItem.spec.js

@@ -1,25 +1,13 @@
 const {models} = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('ItemShelving upsertItem()', () => {
     const warehouseFk = 1;
-    let ctx;
+
+    const ctx = beforeAll.getCtx();
     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;

modules/item/back/methods/item/specs/clone.spec.js

@@ -1,21 +1,8 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('item clone()', () => {
     let nextItemId;
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     beforeEach(async() => {
         let query = `SELECT i1.id + 1 as id FROM vn.item i1

modules/item/back/methods/item/specs/getBalance.spec.js

@@ -2,17 +2,14 @@ const models = require('vn-loopback/server/server').models;
 const LoopBackContext = require('loopback-context');
 
 describe('item getBalance()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
     it('should return the balance lines of a client type loses in which one has highlighted true', async() => {
         const tx = await models.Item.beginTransaction({});
         const options = {transaction: tx};
 
         try {
-            const activeCtx = {
-                accessToken: {userId: 9},
-            };
             spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-                active: activeCtx
+                active: ctx
             });
             const losesClientId = 1111;
             const ticket = await models.Ticket.findById(7, null, options);

modules/item/back/methods/item/specs/new.spec.js

@@ -1,21 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('item new()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
-    beforeAll(async() => {
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should create a new item, adding the name as a tag', async() => {
         const tx = await models.Item.beginTransaction({});

modules/item/back/methods/item/specs/regularize.spec.js

@@ -1,27 +1,13 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('regularize()', () => {
-    beforeAll(async() => {
+    const ctx = beforeAll.mockLoopBackContext(18);
-        const activeCtx = {
-            accessToken: {userId: 18},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should create a new ticket and add a line', async() => {
         const tx = await models.Item.beginTransaction({});
         const options = {transaction: tx};
 
         try {
-            const ctx = {req: {accessToken: {userId: 18}}};
             const itemId = 1;
             const warehouseId = 1;
             const quantity = 11;

modules/item/back/methods/item/specs/updateTaxes.spec.js

@@ -1,20 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('item updateTaxes()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should throw an error if the taxClassFk is blank', async() => {
         const tx = await models.Item.beginTransaction({});

modules/item/back/methods/tag/specs/onSubmit.spec.js

@@ -1,20 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('tag onSubmit()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should delete a tag', async() => {
         const tx = await models.Item.beginTransaction({});

modules/order/back/methods/order-row/specs/addToOrder.spec.js

@@ -1,7 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('order addToOrder()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
     const orderId = 8;
     it('should add a row to a given order', async() => {
         const tx = await models.Order.beginTransaction({});

modules/order/back/methods/order/specs/filter.spec.js

@@ -1,11 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('order filter()', () => {
-    const ctx = {
+    const ctx = beforeAll.getCtx();
-        req: {accessToken: {userId: 9}},
-        args: {},
-        params: {}
-    };
 
     it('should call the filter method with a basic search', async() => {
         const myCtx = Object.assign({}, ctx);

modules/order/back/methods/order/specs/new.spec.js

@@ -2,7 +2,7 @@ const models = require('vn-loopback/server/server').models;
 const UserError = require('vn-loopback/util/user-error');
 
 describe('order new()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
     it('should throw an error if the client isnt active', async() => {
         const tx = await models.Order.beginTransaction({});
 

modules/order/back/methods/order/specs/newFromTicket.spec.js

@@ -1,7 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('order newFromTicket()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
     it('should create a new order from an existing ticket', async() => {
         const tx = await models.Order.beginTransaction({});
 

modules/route/back/methods/agency-term/specs/createInvoiceIn.spec.js

@@ -1,21 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('AgencyTerm createInvoiceIn()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
-    beforeAll(async() => {
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
     const rows = [
         {
             routeFk: 2,

modules/route/back/methods/roadmapStop/getPalletMatchState.js

@@ -0,0 +1,62 @@
+module.exports = Self => {
+    Self.remoteMethod('getPalletMatchState', {
+        description: 'Get list of pallet from truckFk with true or false if state is matched',
+        accessType: 'WRITE',
+        accepts: [{
+            arg: 'truckFk',
+            type: 'number',
+            required: true,
+            description: 'The truckFk id'
+        },
+        {
+            arg: 'state',
+            type: 'string',
+            required: true,
+            description: 'State code'
+        }],
+        returns: {
+            type: 'object',
+            root: true
+        },
+        http: {
+            path: `/getPalletMatchState`,
+            verb: 'GET'
+        }
+    });
+
+    Self.getPalletMatchState = async(truckFk, state, options) => {
+        const myOptions = {};
+
+        if (typeof options == 'object')
+            Object.assign(myOptions, options);
+
+        const result = await Self.rawSql(`
+                        WITH tPallet AS(
+                            SELECT ep.id pallet, e.id expedition, e.stateTypeFk
+                                FROM vn.expeditionPallet ep
+                                    JOIN vn.expeditionScan es ON es.palletFk = ep.id
+                                    JOIN expedition e ON e.id = es.expeditionFk
+                                WHERE ep.truckFk = ?
+                        ),totalPalletExpedition AS(
+                                SELECT t.*, COUNT(expedition) totalPalletExpedition
+                                    FROM tPallet t
+                                    GROUP BY expedition
+                        ),totalPalletExpeditionCode AS(
+                                SELECT t.*, COUNT(expedition) totalPalletExpeditionCode
+                                    FROM tPallet t
+                                        JOIN vn.expeditionStateType est ON est.id = t.stateTypeFk
+                                    WHERE code = ?
+                                    GROUP BY expedition
+                            )
+                        SELECT  t.pallet,
+                                tpe.totalPalletExpedition = tpec.totalPalletExpeditionCode hasMatchStateCode
+                            FROM tPallet t
+                                LEFT JOIN totalPalletExpedition tpe ON tpe.expedition = t.expedition
+                                LEFT JOIN totalPalletExpeditionCode tpec ON tpec.expedition = t.expedition
+                            GROUP BY t.pallet;`,
+        [truckFk, state],
+        myOptions);
+
+        return result;
+    };
+};

modules/route/back/methods/roadmapStop/specs/getPalletMatchState.spec.js

@@ -0,0 +1,11 @@
+const {models} = require('vn-loopback/server/server');
+
+describe('roadMapStop getPalletMatchState()', () => {
+    it('should return list of pallet with true or false if state is matched', async() => {
+        const truckFk = 1;
+        const state = 'ON DELIVERY';
+        const result = await models.RoadmapStop.getPalletMatchState(truckFk, state);
+
+        expect(result[0].hasMatchStateCode).toBe(1);
+    });
+});

modules/route/back/methods/route/specs/clone.spec.js

@@ -1,21 +1,7 @@
 const app = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('route clone()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     const createdDate = Date.vnNew();
     it('should throw an error if the amount of ids pased to the clone function do no match the database', async() => {

modules/route/back/methods/route/specs/guessPriority.spec.js

@@ -1,20 +1,9 @@
 const app = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('route guessPriority()', () => {
     const targetRouteId = 7;
     let routeTicketsToRestore;
-
+    const ctx = beforeAll.getCtx();
-    const activeCtx = {
-        accessToken: {userId: 9},
-        __: () => {}
-    };
-
-    beforeAll(() => {
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     afterAll(async() => {
         let restoreFixtures = [];
@@ -25,9 +14,6 @@ describe('route guessPriority()', () => {
     });
 
     it('should call guessPriority() then check all tickets in that route have their priorities defined', async() => {
-        const ctx = {
-            req: activeCtx
-        };
         routeTicketsToRestore = await app.models.Ticket.find({where: {routeFk: targetRouteId}});
 
         await app.models.Route.guessPriority(ctx, targetRouteId);

modules/route/back/methods/route/specs/updateWorkCenter.spec.js

@@ -1,20 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('route updateWorkCenter()', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
     const routeId = 1;
 
     it('should set the commission work center if the worker has workCenter', async() => {

modules/route/back/models/roadmap.json

@@ -3,7 +3,7 @@
     "base": "VnModel",
     "options": {
         "mysql": {
-        "table": "roadmap"
+            "table": "roadmap"
         }
     },
     "properties": {
@@ -41,10 +41,22 @@
         },
         "driverName": {
             "type": "string"
+        },
+        "kmStart": {
+            "type": "number"
+        },
+        "kmEnd": {
+            "type": "number"
+        },
+        "started": {
+            "type": "date"
+        },
+        "finished": {
+            "type": "date"
         }
     },
-	"relations": {
+    "relations": {
-		"worker": {
+        "worker": {
             "type": "belongsTo",
             "model": "Worker",
             "foreignKey": "id"
@@ -59,5 +71,5 @@
             "model": "RoadmapStop",
             "foreignKey": "roadmapFk"
         }
-	}
+    }
 }

modules/route/back/models/roadmapStop.js

@@ -0,0 +1,3 @@
+module.exports = Self => {
+    require('../methods/roadmapStop/getPalletMatchState')(Self);
+};

modules/route/back/models/roadmapStop.json

@@ -3,7 +3,7 @@
     "base": "VnModel",
     "options": {
         "mysql": {
-        "table": "roadmapStop"
+            "table": "roadmapStop"
         }
     },
     "properties": {
@@ -28,16 +28,16 @@
             "type": "number"
         }
     },
-	"relations": {
+    "relations": {
-		"roadmap": {
+        "roadmap": {
             "type": "belongsTo",
             "model": "Roadmap",
             "foreignKey": "roadmapFk"
         },
         "address": {
             "type": "belongsTo",
-            "model": "Address",
+            "model": "RoadmapAddress",
             "foreignKey": "addressFk"
         }
-	}
+    }
 }

modules/supplier/back/methods/supplier/specs/consumption.spec.js

@@ -1,8 +1,8 @@
 const app = require('vn-loopback/server/server');
 
 describe('supplier consumption() filter', () => {
+    const ctx = beforeAll.getCtx();
     it('should return a list of entries from the supplier 2', async() => {
-        const ctx = {req: {accessToken: {userId: 9}}, args: {}};
         const filter = {
             where: {
                 supplierFk: 2

modules/supplier/back/methods/supplier/specs/updateFiscalData.spec.js

@@ -3,7 +3,6 @@ const LoopBackContext = require('loopback-context');
 
 describe('Supplier updateFiscalData()', () => {
     const supplierId = 1;
-    const administrativeId = 5;
     const buyerId = 35;
 
     const name = 'NEW PLANTS';

modules/supplier/back/models/specs/supplier.spec.js

@@ -1,25 +1,13 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('loopback model Supplier', () => {
     let supplierOne;
     let supplierTwo;
 
+    beforeAll.mockLoopBackContext();
     beforeAll(async() => {
         supplierOne = await models.Supplier.findById(1);
         supplierTwo = await models.Supplier.findById(442);
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
     });
 
     describe('payMethodFk', () => {

modules/ticket/back/methods/expedition-state/specs/addExpeditionState.spec.js

@@ -1,7 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('expeditionState addExpeditionState()', () => {
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
     it('should update the expedition states', async() => {
         const tx = await models.ExpeditionState.beginTransaction({});
         try {

modules/ticket/back/methods/expedition/specs/deleteExpeditions.spec.js

@@ -1,19 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('ticket deleteExpeditions()', () => {
-    let ctx;
+    const ctx = beforeAll.getCtx();
-    beforeAll(async() => {
-        ctx = {
-            accessToken: {userId: 9},
-            req: {
-                headers: {origin: 'http://localhost'}
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: ctx
-        });
-    });
 
     it('should delete the selected expeditions', async() => {
         const tx = await models.Expedition.beginTransaction({});

modules/ticket/back/methods/expedition/specs/moveExpeditions.spec.js

@@ -1,29 +1,11 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('ticket moveExpeditions()', () => {
-    beforeAll(async() => {
+    const ctx = beforeAll.getCtx();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should move the selected expeditions to new ticket', async() => {
         const tx = await models.Expedition.beginTransaction({});
-        const ctx = {
+        const myCtx = ctx;
-            req: {accessToken: {userId: 9}},
-            args: {},
-            params: {}
-        };
-        const myCtx = Object.assign({}, ctx);
 
         try {
             const options = {transaction: tx};
@@ -38,7 +20,7 @@ describe('ticket moveExpeditions()', () => {
 
             };
 
-            const ticket = await models.Expedition.moveExpeditions(myCtx, options);
+            const ticket = await models.Expedition.moveExpeditions(ctx, options);
 
             const newestTicketIdInFixtures = 27;
 

modules/ticket/back/methods/sale/specs/canEdit.spec.js

@@ -1,21 +1,8 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('sale canEdit()', () => {
     const employeeId = 1;
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     describe('sale not exists', () => {
         it('should return error if sale not exists', async() => {

modules/ticket/back/methods/sale/specs/clone.spec.js

@@ -1,24 +1,11 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
-
 describe('Ticket cloning - clone function', () => {
-    let ctx;
+    const ctx = beforeAll.getCtx();
+    beforeAll.mockLoopBackContext();
     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;

modules/ticket/back/methods/sale/specs/deleteSales.spec.js

@@ -1,20 +1,8 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('sale deleteSales()', () => {
-    beforeAll(async() => {
+    const ctx = beforeAll.getCtx();
-        const activeCtx = {
+    beforeAll.mockLoopBackContext();
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should throw an error if the ticket of the given sales is not editable', async() => {
         const tx = await models.Sale.beginTransaction({});
@@ -23,14 +11,6 @@ describe('sale deleteSales()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'localhost:5000'},
-                    __: () => {}
-                }
-            };
-
             const sales = [{id: 1, instance: 0}, {id: 2, instance: 1}];
             const ticketId = 2;
 
@@ -51,13 +31,6 @@ describe('sale deleteSales()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'localhost:5000'},
-                    __: () => {}
-                }
-            };
             const sale = await models.Sale.findOne({where: {id: 9}}, options);
             sale.id = null;
             const newSale = await models.Sale.create(sale, options);

modules/ticket/back/methods/sale/specs/recalculatePrice.spec.js

@@ -1,6 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('sale recalculatePrice()', () => {
+    const ctx = beforeAll.getCtx();
     it('should update the sale price', async() => {
         const tx = await models.Sale.beginTransaction({});
         const sales = [
@@ -10,7 +11,6 @@ describe('sale recalculatePrice()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}};
             const response = await models.Sale.recalculatePrice(ctx, sales, options);
 
             expect(response[0].affectedRows).toBeDefined();
@@ -30,7 +30,6 @@ describe('sale recalculatePrice()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}};
             const immutableSale = [{id: 1, ticketFk: 1}];
             await models.Sale.recalculatePrice(ctx, immutableSale, options);
 

modules/ticket/back/methods/sale/specs/reserve.spec.js

@@ -1,5 +1,4 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('sale reserve()', () => {
     const ctx = {
@@ -10,19 +9,7 @@ describe('sale reserve()', () => {
         }
     };
 
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should throw an error if the ticket can not be modified', async() => {
         const tx = await models.Sale.beginTransaction({});

modules/ticket/back/methods/sale/specs/updateConcept.spec.js

@@ -1,22 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('sale updateConcept()', () => {
-    beforeAll(async() => {
+    const ctx = beforeAll.getCtx();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
-
-    const ctx = {req: {accessToken: {userId: 9}}};
     const saleId = 25;
 
     it('should throw if ID was undefined', async() => {

modules/ticket/back/methods/ticket-request/specs/confirm.spec.js

@@ -1,29 +1,8 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('ticket-request confirm()', () => {
-    beforeAll(async() => {
+    const ctx = beforeAll.getCtx();
-        const activeCtx = {
+    beforeAll.mockLoopBackContext();
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
-    let ctx = {
-        req: {
-            accessToken: {userId: 9},
-            headers: {origin: 'http://localhost'}
-        }
-    };
-    ctx.req.__ = value => {
-        return value;
-    };
 
     it(`should throw an error if the item doesn't exist`, async() => {
         const tx = await models.TicketRequest.beginTransaction({});

modules/ticket/back/methods/ticket-request/specs/deny.spec.js

@@ -1,39 +1,15 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('ticket-request deny()', () => {
-    beforeAll(async() => {
+    const ctx = beforeAll.getCtx();
-        const activeCtx = {
+    beforeAll.mockLoopBackContext();
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should return the denied ticket request', async() => {
         const tx = await models.TicketRequest.beginTransaction({});
 
         try {
             const options = {transaction: tx};
-
+            ctx.args = {id: 4, observation: 'my observation'};
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'http://localhost'}
-                },
-                args: {id: 4, observation: 'my observation'},
-            };
-
-            ctx.req.__ = value => {
-                return value;
-            };
-
             const result = await models.TicketRequest.deny(ctx, options);
 
             expect(result.id).toEqual(4);

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.getCtx();
     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;

modules/ticket/back/methods/ticket/specs/componentUpdate.spec.js

@@ -1,8 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('ticket componentUpdate()', () => {
-    const userID = 1101;
+    const ctx = beforeAll.getCtx(1101);
     const ticketID = 11;
     const today = Date.vnNew();
     const tomorrow = Date.vnNew();
@@ -16,18 +15,8 @@ describe('ticket componentUpdate()', () => {
     let componentOfSaleEight;
     let componentValue;
 
+    beforeAll.mockLoopBackContext();
     beforeAll(async() => {
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
         const deliveryComponenet = await models.Component.findOne({where: {code: 'delivery'}});
         deliveryComponentId = deliveryComponenet.id;
         componentOfSaleSeven = `SELECT value
@@ -64,16 +53,7 @@ describe('ticket componentUpdate()', () => {
                 isWithoutNegatives: false
             };
 
-            let ctx = {
+            ctx.args = args;
-                args: args,
-                req: {
-                    accessToken: {userId: userID},
-                    headers: {origin: 'http://localhost'},
-                    __: value => {
-                        return value;
-                    }
-                }
-            };
             await models.Ticket.componentUpdate(ctx, options);
 
             [componentValue] = await models.SaleComponent.rawSql(componentOfSaleSeven, null, options);
@@ -113,16 +93,7 @@ describe('ticket componentUpdate()', () => {
                 isWithoutNegatives: false
             };
 
-            const ctx = {
+            ctx.args = args;
-                args: args,
-                req: {
-                    accessToken: {userId: userID},
-                    headers: {origin: 'http://localhost'},
-                    __: value => {
-                        return value;
-                    }
-                }
-            };
             const observationTypeDelivery = await models.ObservationType.findOne({
                 where: {code: 'delivery'}
             }, options);
@@ -152,6 +123,7 @@ describe('ticket componentUpdate()', () => {
     });
 
     it('should change warehouse and without negatives', async() => {
+        const ctx = beforeAll.getCtx(9);
         const tx = await models.SaleComponent.beginTransaction({});
 
         try {
@@ -178,17 +150,7 @@ describe('ticket componentUpdate()', () => {
                 option: 'renewPrices',
                 isWithoutNegatives: true
             };
-
+            ctx.args = args;
-            const ctx = {
-                args: args,
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'http://localhost'},
-                    __: value => {
-                        return value;
-                    }
-                }
-            };
 
             const oldTicket = await models.Ticket.findById(ticketID, null, options);
 
@@ -210,6 +172,7 @@ describe('ticket componentUpdate()', () => {
     });
 
     describe('componentUpdate() keepPrice', () => {
+        const ctx = beforeAll.getCtx();
         it('should change shipped and keep price', async() => {
             const tx = await models.Ticket.beginTransaction({});
 
@@ -239,16 +202,7 @@ describe('ticket componentUpdate()', () => {
                     keepPrice: true
                 };
 
-                const ctx = {
+                ctx.args = args;
-                    args: args,
-                    req: {
-                        accessToken: {userId: 9},
-                        headers: {origin: 'http://localhost'},
-                        __: value => {
-                            return value;
-                        }
-                    }
-                };
 
                 const beforeSale = await models.Sale.findById(saleId, null, options);
                 await models.Ticket.componentUpdate(ctx, options);
@@ -292,16 +246,7 @@ describe('ticket componentUpdate()', () => {
                     keepPrice: false
                 };
 
-                const ctx = {
+                ctx.args = args;
-                    args: args,
-                    req: {
-                        accessToken: {userId: 9},
-                        headers: {origin: 'http://localhost'},
-                        __: value => {
-                            return value;
-                        }
-                    }
-                };
 
                 const beforeSale = await models.Sale.findById(saleId, null, options);
                 await models.Ticket.componentUpdate(ctx, options);

modules/ticket/back/methods/ticket/specs/filter.spec.js

@@ -1,6 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('ticket filter()', () => {
+    const ctx = beforeAll.getCtx();
     it('should return the tickets matching the filter', async() => {
         const tx = await models.Ticket.beginTransaction({});
 
@@ -31,11 +32,12 @@ describe('ticket filter()', () => {
             const today = Date.vnNew();
             today.setHours(23, 59, 59, 59);
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {
+            const args = {
                 problems: true,
                 from: yesterday,
                 to: today
-            }};
+            };
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -60,11 +62,12 @@ describe('ticket filter()', () => {
             const today = Date.vnNew();
             today.setHours(23, 59, 59, 59);
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {
+            const args = {
                 problems: false,
                 from: yesterday,
                 to: today
-            }};
+            };
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -83,7 +86,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {problems: null}};
+            const args = {problems: null};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -102,7 +106,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {orderFk: 11}};
+            const args = {orderFk: 11};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
             const firstRow = result[0];
@@ -123,7 +128,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {pending: true}};
+            const args = {pending: true};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -146,7 +152,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {pending: false}};
+            const args = {pending: false};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
             const firstRow = result[0];
@@ -167,11 +174,13 @@ describe('ticket filter()', () => {
 
     it('should return the tickets from the worker team', async() => {
         const tx = await models.Ticket.beginTransaction({});
+        const ctx = beforeAll.getCtx(18);
 
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 18}}, args: {myTeam: true}};
+            const args = {myTeam: true};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -185,12 +194,14 @@ describe('ticket filter()', () => {
     });
 
     it('should return the tickets that are not from the worker team', async() => {
+        const ctx = beforeAll.getCtx(18);
         const tx = await models.Ticket.beginTransaction({});
 
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 18}}, args: {myTeam: false}};
+            const args = {myTeam: false};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -204,12 +215,14 @@ describe('ticket filter()', () => {
     });
 
     it('should return the tickets belonging to the collection id 1', async() => {
+        const ctx = beforeAll.getCtx(18);
         const tx = await models.Ticket.beginTransaction({});
 
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 18}}, args: {collectionFk: 1}};
+            const args = {collectionFk: 1};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -228,7 +241,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {hasRoute: true}};
+            const args = {hasRoute: true};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -247,7 +261,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {hasRoute: false}};
+            const args = {hasRoute: false};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -266,7 +281,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {hasRoute: null}};
+            const args = {hasRoute: null};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -285,7 +301,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {hasInvoice: true}};
+            const args = {hasInvoice: true};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 
@@ -304,7 +321,8 @@ describe('ticket filter()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}, args: {hasInvoice: null}};
+            const args = {hasInvoice: null};
+            ctx.args = args;
             const filter = {};
             const result = await models.Ticket.filter(ctx, filter, options);
 

modules/ticket/back/methods/ticket/specs/getTicketsAdvance.spec.js

@@ -1,6 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('TicketFuture getTicketsAdvance()', () => {
+    const ctx = beforeAll.getCtx();
     const today = Date.vnNew();
     today.setHours(0, 0, 0, 0);
     let tomorrow = Date.vnNew();
@@ -18,7 +19,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
                 warehouseFk: 1,
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsAdvance(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -42,7 +43,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
                 isFullMovable: true
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsAdvance(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -67,7 +68,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
                 isFullMovable: false
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsAdvance(ctx, options);
 
             expect(result.length).toEqual(0);
@@ -92,7 +93,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
                 ipt: 'V'
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsAdvance(ctx, options);
 
             expect(result.length).toBeGreaterThan(5);
@@ -117,7 +118,7 @@ describe('TicketFuture getTicketsAdvance()', () => {
                 tfIpt: 'V'
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsAdvance(ctx, options);
 
             expect(result.length).toBeGreaterThan(5);

modules/ticket/back/methods/ticket/specs/getTicketsFuture.spec.js

@@ -1,6 +1,7 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('ticket getTicketsFuture()', () => {
+    const ctx = beforeAll.getCtx();
     const today = Date.vnNew();
     today.setHours(0, 0, 0, 0);
 
@@ -16,7 +17,7 @@ describe('ticket getTicketsFuture()', () => {
                 warehouseFk: 1,
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -40,7 +41,7 @@ describe('ticket getTicketsFuture()', () => {
                 problems: true
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -65,7 +66,7 @@ describe('ticket getTicketsFuture()', () => {
                 problems: false
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toEqual(0);
@@ -90,7 +91,7 @@ describe('ticket getTicketsFuture()', () => {
                 problems: null
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -115,7 +116,7 @@ describe('ticket getTicketsFuture()', () => {
                 state: 'OK'
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -140,7 +141,7 @@ describe('ticket getTicketsFuture()', () => {
                 futureState: 'OK'
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -165,7 +166,7 @@ describe('ticket getTicketsFuture()', () => {
                 ipt: null
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -190,7 +191,7 @@ describe('ticket getTicketsFuture()', () => {
                 ipt: 'H'
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -215,7 +216,7 @@ describe('ticket getTicketsFuture()', () => {
                 futureIpt: null
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -240,7 +241,7 @@ describe('ticket getTicketsFuture()', () => {
                 futureIpt: 'H'
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -265,7 +266,7 @@ describe('ticket getTicketsFuture()', () => {
                 id: 13
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);
@@ -290,7 +291,7 @@ describe('ticket getTicketsFuture()', () => {
                 futureId: 12
             };
 
-            const ctx = {req: {accessToken: {userId: 9}}, args};
+            ctx.args = args;
             const result = await models.Ticket.getTicketsFuture(ctx, options);
 
             expect(result.length).toBeGreaterThan(0);

modules/ticket/back/methods/ticket/specs/isEditableOrThrow.spec.js

@@ -1,14 +1,12 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('ticket isEditableOrThrow()', () => {
+    const ctx = beforeAll.getCtx();
     it('should throw an error as the ticket does not exist', async() => {
         const tx = await models.Ticket.beginTransaction({});
         let error;
         try {
             const options = {transaction: tx};
-            const ctx = {
-                req: {accessToken: {userId: 9}}
-            };
 
             await models.Ticket.isEditableOrThrow(ctx, 9999, options);
             await tx.rollback();

modules/ticket/back/methods/ticket/specs/merge.spec.js

@@ -1,7 +1,7 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('ticket merge()', () => {
+    const ctx = beforeAll.getCtx();
     const tickets = {
         originId: 13,
         destinationId: 12,
@@ -10,30 +10,6 @@ describe('ticket merge()', () => {
         workerFk: 1
     };
 
-    beforeAll(async() => {
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
-
-    const ctx = {
-        req: {
-            accessToken: {userId: 9},
-            headers: {origin: 'http://localhost:5000'},
-        }
-    };
-    ctx.req.__ = value => {
-        return value;
-    };
-
     it('should merge two tickets', async() => {
         const tx = await models.Ticket.beginTransaction({});
 

modules/ticket/back/methods/ticket/specs/recalculateComponents.spec.js

@@ -3,6 +3,7 @@ const ForbiddenError = require('vn-loopback/util/forbiddenError');
 
 describe('ticket recalculateComponents()', () => {
     const ticketId = 11;
+    const ctx = beforeAll.getCtx();
 
     it('should update the ticket components', async() => {
         const tx = await models.Ticket.beginTransaction({});
@@ -10,7 +11,6 @@ describe('ticket recalculateComponents()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}};
             const response = await models.Ticket.recalculateComponents(ctx, ticketId, options);
 
             expect(response.affectedRows).toBeDefined();
@@ -29,7 +29,6 @@ describe('ticket recalculateComponents()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}};
             const immutableTicketId = 1;
             await models.Ticket.recalculateComponents(ctx, immutableTicketId, options);
 

modules/ticket/back/methods/ticket/specs/sendSms.spec.js

@@ -1,13 +1,13 @@
 const models = require('vn-loopback/server/server').models;
 
 describe('ticket sendSms()', () => {
+    const ctx = beforeAll.getCtx();
     it('should send a message and log it', async() => {
         const tx = await models.Ticket.beginTransaction({});
 
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: {accessToken: {userId: 9}}};
             const id = 11;
             const destination = 222222222;
             const message = 'this is the message created in a test';

modules/ticket/back/methods/ticket/specs/setDeleted.spec.js

@@ -1,17 +1,8 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('ticket setDeleted()', () => {
-    const userId = 1106;
+    const ctx = beforeAll.getCtx();
-    const activeCtx = {
+    beforeAll.mockLoopBackContext();
-        accessToken: {userId: userId},
-    };
-
-    beforeEach(() => {
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
 
     it('should throw an error if the given ticket has a claim', async() => {
         const tx = await models.Ticket.beginTransaction({});
@@ -20,7 +11,6 @@ describe('ticket setDeleted()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {req: activeCtx};
             const ticketId = 16;
 
             await models.Ticket.setDeleted(ctx, ticketId, options);
@@ -40,15 +30,6 @@ describe('ticket setDeleted()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'http://localhost:5000'},
-                }
-            };
-            ctx.req.__ = value => {
-                return value;
-            };
             const ticketId = 24;
             const [sectorCollectionBefore] = await models.Ticket.rawSql(
                 `SELECT COUNT(*) numberRows
@@ -75,15 +56,6 @@ describe('ticket setDeleted()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'http://localhost:5000'},
-                }
-            };
-            ctx.req.__ = value => {
-                return value;
-            };
             const [ticketCollectionOld] = await models.Ticket.rawSql(
                 `SELECT COUNT(*) numberRows
                     FROM vn.ticketCollection`, [], options);
@@ -110,16 +82,6 @@ describe('ticket setDeleted()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'http://localhost:5000'},
-                }
-            };
-            ctx.req.__ = value => {
-                return value;
-            };
-
             const ticketId = 8;
             await models.Ticket.setDeleted(ctx, ticketId, options);
 

modules/ticket/back/methods/ticket/specs/state.spec.js

@@ -2,14 +2,10 @@ const models = require('vn-loopback/server/server').models;
 const LoopBackContext = require('loopback-context');
 
 xdescribe('ticket state()', () => {
+    const ctx = beforeAll.getCtx();
     const salesPersonId = 18;
     const employeeId = 1;
     const productionId = 49;
-    const activeCtx = {
-        accessToken: {userId: 9},
-        __: value => value
-    };
-    const ctx = {req: activeCtx};
     const now = Date.vnNew();
     const sampleTicket = {
         shipped: now,
@@ -33,7 +29,7 @@ xdescribe('ticket state()', () => {
 
     beforeAll(async() => {
         spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
+            active: ctx
         });
     });
 
@@ -45,7 +41,7 @@ xdescribe('ticket state()', () => {
         try {
             const options = {transaction: tx};
 
-            activeCtx.accessToken.userId = salesPersonId;
+            ctx.req.accessToken.userId = salesPersonId;
 
             await models.Ticket.state(ctx, {ticketFk: 2, stateFk: 3}, options);
 
@@ -66,7 +62,7 @@ xdescribe('ticket state()', () => {
         try {
             const options = {transaction: tx};
 
-            activeCtx.accessToken.userId = employeeId;
+            ctx.req.accessToken.userId = employeeId;
 
             await models.Ticket.state(ctx, {ticketFk: 11, stateFk: 13}, options);
 
@@ -88,7 +84,7 @@ xdescribe('ticket state()', () => {
 
             const ticket = await models.Ticket.create(sampleTicket, options);
 
-            activeCtx.accessToken.userId = productionId;
+            ctx.req.accessToken.userId = productionId;
             const stateOk = await models.State.findOne({where: {code: 'OK'}}, options);
             const params = {ticketFk: ticket.id, stateFk: stateOk.id};
 
@@ -115,7 +111,7 @@ xdescribe('ticket state()', () => {
 
             spyOn(models.Chat, 'sendCheckingPresence').and.callThrough();
             const ticket = await models.Ticket.create(sampleTicket, options);
-            activeCtx.accessToken.userId = salesPersonId;
+            ctx.req.accessToken.userId = salesPersonId;
             const assignedState = await models.State.findOne({where: {code: 'PICKER_DESIGNED'}}, options);
             const paramsAssigned = {ticketFk: ticket.id, stateFk: assignedState.id, userFk: 1};
             const resAssigned = await models.Ticket.state(ctx, paramsAssigned, options);
@@ -127,7 +123,7 @@ xdescribe('ticket state()', () => {
             expect(resAssigned.id).toBeDefined();
             expect(models.Chat.sendCheckingPresence).not.toHaveBeenCalled();
 
-            activeCtx.accessToken.userId = productionId;
+            ctx.req.accessToken.userId = productionId;
             const packedState = await models.State.findOne({where: {code: 'PACKED'}}, options);
             const paramsPacked = {ticketFk: ticket.id, stateFk: packedState.id, userFk: salesPersonId};
             const resPacked = await models.Ticket.state(ctx, paramsPacked, options);
@@ -149,7 +145,7 @@ xdescribe('ticket state()', () => {
             const options = {transaction: tx};
 
             const ticket = await models.Ticket.create(sampleTicket, options);
-            activeCtx.accessToken.userId = salesPersonId;
+            ctx.req.accessToken.userId = salesPersonId;
 
             const sampleSale = {
                 ticketFk: ticket.id,
@@ -169,7 +165,7 @@ xdescribe('ticket state()', () => {
             expect(resAssigned.userFk).toBe(1);
             expect(resAssigned.id).toBeDefined();
 
-            activeCtx.accessToken.userId = productionId;
+            ctx.req.accessToken.userId = productionId;
             const packedState = await models.State.findOne({where: {code: 'PACKED'}}, options);
             const paramsPacked = {ticketFk: ticket.id, stateFk: packedState.id, userFk: salesPersonId};
             const resPacked = await models.Ticket.state(ctx, paramsPacked, options);

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.getCtx();
     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;

modules/ticket/back/methods/ticket/specs/updateDiscount.spec.js

@@ -1,20 +1,8 @@
 const models = require('vn-loopback/server/server').models;
-const LoopBackContext = require('loopback-context');
 
 describe('sale updateDiscount()', () => {
-    beforeAll(async() => {
+    const ctx = beforeAll.getCtx();
-        const activeCtx = {
+    beforeAll.mockLoopBackContext();
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
     const originalSaleId = 8;
 
     it('should throw an error if no sales were selected', async() => {
@@ -24,13 +12,6 @@ describe('sale updateDiscount()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'localhost:5000'},
-                    __: () => {}
-                }
-            };
             const ticketId = 11;
             const sales = [];
             const newDiscount = 10;
@@ -53,13 +34,6 @@ describe('sale updateDiscount()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'localhost:5000'},
-                    __: () => {}
-                }
-            };
             const ticketId = 11;
             const sales = [1, 14];
             const newDiscount = 10;
@@ -82,13 +56,6 @@ describe('sale updateDiscount()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'localhost:5000'},
-                    __: () => {}
-                }
-            };
             const ticketId = 1;
             const sales = [1];
             const newDiscount = 100;
@@ -106,17 +73,11 @@ describe('sale updateDiscount()', () => {
 
     it('should update the discount if the salesPerson has mana and manaCode = "mana"', async() => {
         const tx = await models.Ticket.beginTransaction({});
+        const ctx = beforeAll.getCtx(18);
 
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 18},
-                    headers: {origin: 'localhost:5000'},
-                    __: () => {}
-                }
-            };
             const ticketId = 11;
             const sales = [originalSaleId];
             const newDiscount = 100;
@@ -151,17 +112,11 @@ describe('sale updateDiscount()', () => {
 
     it('should update the discount if the salesPerson has mana and manaCode = "manaClaim"', async() => {
         const tx = await models.Ticket.beginTransaction({});
+        const ctx = beforeAll.getCtx(18);
 
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 18},
-                    headers: {origin: 'localhost:5000'},
-                    __: () => {}
-                }
-            };
             const ticketId = 11;
             const sales = [originalSaleId];
             const newDiscount = 100;
@@ -200,13 +155,6 @@ describe('sale updateDiscount()', () => {
         try {
             const options = {transaction: tx};
 
-            const ctx = {
-                req: {
-                    accessToken: {userId: 9},
-                    headers: {origin: 'localhost:5000'},
-                    __: () => {}
-                }
-            };
             const ticketId = 11;
             const sales = [originalSaleId];
             const newDiscount = 100;

modules/ticket/back/models/specs/ticket-packaging.spec.js

@@ -1,20 +1,7 @@
 const app = require('vn-loopback/server/server');
-const LoopBackContext = require('loopback-context');
 
 describe('ticket model TicketTracking', () => {
-    beforeAll(async() => {
+    beforeAll.mockLoopBackContext();
-        const activeCtx = {
-            accessToken: {userId: 9},
-            http: {
-                req: {
-                    headers: {origin: 'http://localhost'}
-                }
-            }
-        };
-        spyOn(LoopBackContext, 'getCurrentContext').and.returnValue({
-            active: activeCtx
-        });
-    });
     let ticketTrackingId;
 
     afterAll(async() => {

modules/ticket/back/models/ticket-log.json

@@ -40,6 +40,9 @@
         },
         "changedModelValue": {
             "type": "string"
+        },
+        "summaryId": {
+            "type":"string"
         }
     }
 }

modules/ticket/back/models/ticket-request.js

@@ -19,7 +19,7 @@ module.exports = function(Self) {
             instance.requesterFk = worker.id;
 
             const httpCtx = {req: loopBackContext.active};
-            const httpRequest = httpCtx.req.http .req;
+            const httpRequest = httpCtx.req.http.req;
             const $t = httpRequest.__;
 
             if (attenderFk) {

modules/travel/back/methods/thermograph/specs/createThermograph.spec.js

@@ -5,7 +5,7 @@ describe('Termograph createThermograph()', () => {
     const model = 'DISPOSABLE';
     const temperatureFk = 'COOL';
     const warehouseId = 1;
-    const ctx = {req: {accessToken: {userId: 9}}};
+    const ctx = beforeAll.getCtx();
 
     it(`should create a thermograph which is saved in both thermograph and travelThermograph`, async() => {
         const tx = await models.Thermograph.beginTransaction({});