From d6e440751b4189d7057f753151a402a89d488a7a Mon Sep 17 00:00:00 2001
From: carlosjr <carlosjr@verdnatura.es>
Date: Mon, 1 Mar 2021 17:43:34 +0100
Subject: [PATCH] e2e path for starred modules

---
 .../specs/getStarredModules.spec.js           |  1 -
 .../specs/toggleStarredModule.spec.js         |  1 -
 e2e/helpers/selectors.js                      |  5 +++
 .../{01-login => 01-salix}/01_login.spec.js   |  0
 e2e/paths/01-salix/02_starred-modules.spec.js | 43 +++++++++++++++++++
 front/salix/components/home/home.js           |  2 -
 6 files changed, 48 insertions(+), 4 deletions(-)
 rename e2e/paths/{01-login => 01-salix}/01_login.spec.js (100%)
 create mode 100644 e2e/paths/01-salix/02_starred-modules.spec.js

diff --git a/back/methods/starred-module/specs/getStarredModules.spec.js b/back/methods/starred-module/specs/getStarredModules.spec.js
index f095505f94..c39ed57e9c 100644
--- a/back/methods/starred-module/specs/getStarredModules.spec.js
+++ b/back/methods/starred-module/specs/getStarredModules.spec.js
@@ -19,7 +19,6 @@ describe('getStarredModules()', () => {
     });
 
     it(`should return the starred modules for a given user`, async() => {
-        console.log('GET');
         const newStarred = await app.models.StarredModule.create({workerFk: 9, moduleFk: 'Clients'});
         const starredModules = await app.models.StarredModule.getStarredModules(ctx);
 
diff --git a/back/methods/starred-module/specs/toggleStarredModule.spec.js b/back/methods/starred-module/specs/toggleStarredModule.spec.js
index c3faee0545..a765a29a0e 100644
--- a/back/methods/starred-module/specs/toggleStarredModule.spec.js
+++ b/back/methods/starred-module/specs/toggleStarredModule.spec.js
@@ -21,7 +21,6 @@ describe('toggleStarredModule()', () => {
     });
 
     it('should create a new starred module and then remove it by calling the method again with same args', async() => {
-        console.log('TOGGLE');
         const starredModule = await app.models.StarredModule.toggleStarredModule(ctx, 'Orders');
         let starredModules = await app.models.StarredModule.getStarredModules(ctx);
 
diff --git a/e2e/helpers/selectors.js b/e2e/helpers/selectors.js
index ca521f4454..4e512ee568 100644
--- a/e2e/helpers/selectors.js
+++ b/e2e/helpers/selectors.js
@@ -23,6 +23,11 @@ export default {
         acceptButton: '.vn-confirm.shown button[response=accept]',
         searchButton: 'vn-searchbar vn-icon[icon="search"]'
     },
+    moduleIndex: {
+        anyStarredModule: 'vn-home > div:nth-child(1) > div.modules > a',
+        firstModulePinIcon: 'vn-home a:nth-child(1) vn-icon[icon="push_pin"]',
+        firstModuleRemovePinIcon: 'vn-home a:nth-child(1) vn-icon[icon="remove_circle"]'
+    },
     clientsIndex: {
         createClientButton: `vn-float-button`
     },
diff --git a/e2e/paths/01-login/01_login.spec.js b/e2e/paths/01-salix/01_login.spec.js
similarity index 100%
rename from e2e/paths/01-login/01_login.spec.js
rename to e2e/paths/01-salix/01_login.spec.js
diff --git a/e2e/paths/01-salix/02_starred-modules.spec.js b/e2e/paths/01-salix/02_starred-modules.spec.js
new file mode 100644
index 0000000000..a64b685823
--- /dev/null
+++ b/e2e/paths/01-salix/02_starred-modules.spec.js
@@ -0,0 +1,43 @@
+import selectors from '../../helpers/selectors';
+import getBrowser from '../../helpers/puppeteer';
+
+describe('Starred modules path', async() => {
+    let browser;
+    let page;
+
+    beforeAll(async() => {
+        browser = await getBrowser();
+        page = browser.page;
+        await page.login('employee');
+    });
+
+    afterAll(async() => {
+        await browser.close();
+    });
+
+    it('should make sure there are no modules pinned yet', async() => {
+        const count = await page.countElement(selectors.moduleIndex.anyStarredModule);
+
+        expect(count).toEqual(0);
+    });
+
+    it('should set a module as favore', async() => {
+        await page.waitToClick(selectors.moduleIndex.firstModulePinIcon);
+        const message = await page.waitForSnackbar();
+        const count = await page.countElement(selectors.moduleIndex.anyStarredModule);
+
+        expect(message.text).toContain('Data saved!');
+
+        expect(count).toEqual(1);
+    });
+
+    it('should remove the module from favores', async() => {
+        await page.waitToClick(selectors.moduleIndex.firstModuleRemovePinIcon);
+        const message = await page.waitForSnackbar();
+        const count = await page.countElement(selectors.moduleIndex.anyStarredModule);
+
+        expect(message.text).toContain('Data saved!');
+
+        expect(count).toEqual(0);
+    });
+});
diff --git a/front/salix/components/home/home.js b/front/salix/components/home/home.js
index 8a317cb57c..1e34a2e0cd 100644
--- a/front/salix/components/home/home.js
+++ b/front/salix/components/home/home.js
@@ -25,8 +25,6 @@ export default class Controller extends Component {
             if (module.starred) this.starredCount ++;
             else this.regularCount ++;
         });
-        console.log('this.starredCount', this.starredCount);
-        console.log('this.reg', this.regularCount);
     }
 
     getStarredModules() {