From 89a394a10def74e4b9c11cd3986bf7b91b24aab2 Mon Sep 17 00:00:00 2001
From: Javier Segarra <jsegarra@verdnatura.es>
Date: Tue, 2 Jul 2024 09:52:18 +0200
Subject: [PATCH] test(salix): refs #7648 #7648 entry.filter

---
 modules/entry/back/methods/entry/filter.js    |  2 +-
 .../back/methods/entry/specs/filter.spec.js   | 82 +++++++++++++++----
 2 files changed, 65 insertions(+), 19 deletions(-)

diff --git a/modules/entry/back/methods/entry/filter.js b/modules/entry/back/methods/entry/filter.js
index 2a127b496..700d251a2 100644
--- a/modules/entry/back/methods/entry/filter.js
+++ b/modules/entry/back/methods/entry/filter.js
@@ -149,7 +149,7 @@ module.exports = Self => {
         const isSupplier = await Self.app.models.Supplier.findById(userId, options);
         if (isSupplier) {
             if (!filter.where) filter.where = {};
-            filter.where.supplierFk = ctx.req.accessToken.userId;
+            filter.where[`e.supplierFk`] = ctx.req.accessToken.userId;
         }
         const stmts = [];
         let stmt;
diff --git a/modules/entry/back/methods/entry/specs/filter.spec.js b/modules/entry/back/methods/entry/specs/filter.spec.js
index e116d1c5f..76dc7b786 100644
--- a/modules/entry/back/methods/entry/specs/filter.spec.js
+++ b/modules/entry/back/methods/entry/specs/filter.spec.js
@@ -1,6 +1,6 @@
 const models = require('vn-loopback/server/server').models;
 
-describe('Entry filter()', () => {
+fdescribe('Entry filter()', () => {
     it('should return the entry matching "search"', async() => {
         const tx = await models.Entry.beginTransaction({});
         const options = {transaction: tx};
@@ -48,27 +48,73 @@ describe('Entry filter()', () => {
         }
     });
 
-    it('should return the entry matching the supplier', async() => {
-        const tx = await models.Entry.beginTransaction({});
-        const options = {transaction: tx};
+    describe('should return the entry matching the supplier', () => {
+        it('when userId is supplier ', async() => {
+            const tx = await models.Entry.beginTransaction({});
+            const options = {transaction: tx};
 
-        try {
-            const ctx = {
-                args: {
-                    supplierFk: 2
-                },
-                req: {accessToken: {userId: 9}}
-            };
+            try {
+                const ctx = {
+                    args: {},
+                    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();
-        } catch (e) {
-            await tx.rollback();
-            throw e;
-        }
+                await tx.rollback();
+            } catch (e) {
+                await tx.rollback();
+                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() => {