From 554c09155fd9b10bd14caaec5e2ecffdc63ed96d Mon Sep 17 00:00:00 2001
From: alexandre <alexandre@verdnatura.es>
Date: Wed, 26 Apr 2023 12:04:55 +0200
Subject: [PATCH 1/3] refs #5255

---
 src/components/UserPanel.vue                  |   2 +-
 src/composables/useRole.js                    |   2 +-
 src/pages/Ticket/Card/TicketBoxing.vue        | 221 +++++++++---------
 .../pages/Tickets/TicketBoxing.spec.js        |   2 +-
 4 files changed, 113 insertions(+), 114 deletions(-)

diff --git a/src/components/UserPanel.vue b/src/components/UserPanel.vue
index 6bc00d5ce..a7504705a 100644
--- a/src/components/UserPanel.vue
+++ b/src/components/UserPanel.vue
@@ -70,7 +70,7 @@ async function saveDarkMode(value) {
 }
 
 async function saveLanguage(value) {
-    const query = `/Accounts/${user.value.id}`;
+    const query = `/VnUsers/${user.value.id}`;
     await axios.patch(query, {
         lang: value,
     });
diff --git a/src/composables/useRole.js b/src/composables/useRole.js
index b8da153d3..a781785a0 100644
--- a/src/composables/useRole.js
+++ b/src/composables/useRole.js
@@ -5,7 +5,7 @@ export function useRole() {
     const state = useState();
 
     async function fetch() {
-        const { data } = await axios.get('Accounts/acl');
+        const { data } = await axios.get('VnUsers/acl');
         const roles = data.roles.map((userRoles) => userRoles.role.name);
 
         const userData = {
diff --git a/src/pages/Ticket/Card/TicketBoxing.vue b/src/pages/Ticket/Card/TicketBoxing.vue
index 9d46ddba2..a398d82c9 100644
--- a/src/pages/Ticket/Card/TicketBoxing.vue
+++ b/src/pages/Ticket/Card/TicketBoxing.vue
@@ -27,16 +27,20 @@ const time = ref({
 });
 
 async function fetch() {
-    const filter = {
-        where: {
-            ticketFk: entityId.value,
-        },
-    };
-    const { data } = await axios.get(`/Expeditions/filter`, {
-        params: { filter },
-    });
+    try {
+        const filter = {
+            where: {
+                ticketFk: entityId.value,
+            },
+        };
+        const { data } = await axios.get(`/Expeditions/filter`, {
+            params: { filter },
+        });
 
-    if (data) expeditions.value = data;
+        if (data) expeditions.value = data;
+    } catch (e) {
+        //
+    }
 }
 
 async function getVideoList(expeditionId, timed) {
@@ -46,115 +50,110 @@ async function getVideoList(expeditionId, timed) {
     };
 
     if (timed) {
+        if (timed.max == timed.min) {
+            if (timed.max != 24) timed.max += 1;
+            else timed.min -= 1;
+        }
         Object.assign(params, { from: timed.min, to: timed.max });
     }
-    const { data } = await axios.get(`/Boxings/getVideoList`, { params: params });
+    await axios.get(`/Boxings/getVideoList`, { params: params }).then((res) => {
+        const data = res.data;
+        if (!data.length) {
+            return quasar.notify({
+                message: t('ticket.boxing.notFound'),
+                type: 'negative',
+            });
+        }
+        const list = [];
+        for (const video of data) {
+            const videoName = video.split('.')[0].split('T')[1].replace(/-/g, ':');
+            list.push({
+                label: videoName,
+                value: video,
+                url: `api/Boxings/getVideo?id=${expeditionId}&filename=${video}`,
+            });
+        }
 
-    const list = [];
-    for (const video of data) {
-        const videName = video.split('.')[0].split('T')[1].replaceAll('-', ':');
-        list.push({
-            label: videName,
-            value: video,
-            url: `api/Boxings/getVideo?id=${expeditionId}&filename=${video}`,
-        });
-    }
-
-    videoList.value = list.reverse();
-    if (list[0]) {
-        slide.value = list[0].value;
-        time.value = {
-            min: parseInt(list[0].label.split(':')[0]),
-            max: parseInt(list[list.length - 1].label.split(':')[0]),
-        };
-    }
-
-    if (!data.length) {
-        return quasar.notify({
-            message: t('ticket.boxing.notFound'),
-            type: 'negative',
-        });
-    }
+        videoList.value = list.reverse();
+        if (list[0]) {
+            slide.value = list[0].value;
+            time.value = {
+                min: parseInt(list[0].label.split(':')[0]),
+                max: parseInt(list[list.length - 1].label.split(':')[0]) + 1,
+            };
+        }
+    });
 }
 </script>
 
 <template>
-    <teleport to=".q-layout">
-        <QDrawer show-if-above side="right">
-            <QScrollArea class="fit">
-                <QList bordered separator style="max-width: 318px">
-                    <QItem v-if="lastExpedition && videoList.length">
-                        <QItemSection>
-                            <QItemLabel class="text-h6">
-                                {{ t('ticket.boxing.selectTime') }} ({{ time.min }}-{{
-                                    time.max
-                                }})
-                            </QItemLabel>
-                            <QRange
-                                v-model="time"
-                                @change="getVideoList(lastExpedition, time)"
-                                :min="0"
-                                :max="24"
-                                :step="1"
-                                :left-label-value="time.min + ':00'"
-                                :right-label-value="time.max + ':00'"
-                                label
-                                markers
-                                snap
-                                color="orange"
-                            />
-                        </QItemSection>
-                    </QItem>
-                    <QItem v-if="lastExpedition && videoList.length">
-                        <QItemSection>
-                            <QSelect
-                                color="orange"
-                                v-model="slide"
-                                :options="videoList"
-                                :label="t('ticket.boxing.selectVideo')"
-                                emit-value
-                                map-options
-                            >
-                                <template #prepend>
-                                    <QIcon name="schedule" />
-                                </template>
-                            </QSelect>
-                        </QItemSection>
-                    </QItem>
-                    <QItem
-                        v-for="expedition in expeditions"
-                        :key="expedition.id"
-                        @click="getVideoList(expedition.id)"
-                        clickable
-                        v-ripple
-                    >
-                        <QItemSection>
-                            <QItemLabel class="text-h6">#{{ expedition.id }}</QItemLabel>
-                        </QItemSection>
-                        <QItemSection>
-                            <QItemLabel caption>{{
-                                t('ticket.boxing.created')
-                            }}</QItemLabel>
-                            <QItemLabel>
-                                {{
-                                    date.formatDate(
-                                        expedition.created,
-                                        'YYYY-MM-DD HH:mm:ss'
-                                    )
-                                }}
-                            </QItemLabel>
-                            <QItemLabel caption>{{ t('ticket.boxing.item') }}</QItemLabel>
-                            <QItemLabel>{{ expedition.packagingItemFk }}</QItemLabel>
-                            <QItemLabel caption>{{
-                                t('ticket.boxing.worker')
-                            }}</QItemLabel>
-                            <QItemLabel>{{ expedition.userName }}</QItemLabel>
-                        </QItemSection>
-                    </QItem>
-                </QList>
-            </QScrollArea>
-        </QDrawer>
-    </teleport>
+    <QDrawer show-if-above side="right">
+        <QScrollArea class="fit">
+            <QList bordered separator style="max-width: 318px">
+                <QItem v-if="lastExpedition && videoList.length">
+                    <QItemSection>
+                        <QItemLabel class="text-h6">
+                            {{ t('ticket.boxing.selectTime') }} ({{ time.min }}-{{
+                                time.max
+                            }})
+                        </QItemLabel>
+                        <QRange
+                            v-model="time"
+                            @change="getVideoList(lastExpedition, time)"
+                            :min="0"
+                            :max="24"
+                            :step="1"
+                            :left-label-value="time.min + ':00'"
+                            :right-label-value="time.max + ':00'"
+                            label
+                            markers
+                            snap
+                            color="orange"
+                        />
+                    </QItemSection>
+                </QItem>
+                <QItem v-if="lastExpedition && videoList.length">
+                    <QItemSection>
+                        <QSelect
+                            color="orange"
+                            v-model="slide"
+                            :options="videoList"
+                            :label="t('ticket.boxing.selectVideo')"
+                            emit-value
+                            map-options
+                        >
+                            <template #prepend>
+                                <QIcon name="schedule" />
+                            </template>
+                        </QSelect>
+                    </QItemSection>
+                </QItem>
+                <QItem
+                    v-for="expedition in expeditions"
+                    :key="expedition.id"
+                    @click="getVideoList(expedition.id)"
+                    clickable
+                    v-ripple
+                >
+                    <QItemSection>
+                        <QItemLabel class="text-h6">#{{ expedition.id }}</QItemLabel>
+                    </QItemSection>
+                    <QItemSection>
+                        <QItemLabel caption>{{ t('ticket.boxing.created') }}</QItemLabel>
+                        <QItemLabel>
+                            {{
+                                date.formatDate(expedition.created, 'YYYY-MM-DD HH:mm:ss')
+                            }}
+                        </QItemLabel>
+                        <QItemLabel caption>{{ t('ticket.boxing.item') }}</QItemLabel>
+                        <QItemLabel>{{ expedition.packagingItemFk }}</QItemLabel>
+                        <QItemLabel caption>{{ t('ticket.boxing.worker') }}</QItemLabel>
+                        <QItemLabel>{{ expedition.userName }}</QItemLabel>
+                    </QItemSection>
+                </QItem>
+            </QList>
+        </QScrollArea>
+    </QDrawer>
 
     <QCard>
         <QCarousel animated v-model="slide" height="max-content">
diff --git a/test/vitest/__tests__/pages/Tickets/TicketBoxing.spec.js b/test/vitest/__tests__/pages/Tickets/TicketBoxing.spec.js
index b731cf2cf..8fd62d8c2 100644
--- a/test/vitest/__tests__/pages/Tickets/TicketBoxing.spec.js
+++ b/test/vitest/__tests__/pages/Tickets/TicketBoxing.spec.js
@@ -4,7 +4,7 @@ import TicketBoxing from 'pages/Ticket/Card/TicketBoxing.vue';
 
 // #4836 - Investigate how to test q-drawer outside
 // q-layout or how to teleport q-drawer inside
-describe.skip('TicketBoxing', () => {
+describe('TicketBoxing', () => {
     let vm;
     beforeAll(() => {
         vm = createWrapper(TicketBoxing).vm;

From 0096851894ef3f1fe53d842ef5fa5c310c7e9d40 Mon Sep 17 00:00:00 2001
From: alexandre <alexandre@verdnatura.es>
Date: Wed, 26 Apr 2023 13:52:51 +0200
Subject: [PATCH 2/3] refs #5255 changed time to reactive

---
 src/pages/Ticket/Card/TicketBoxing.vue | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/src/pages/Ticket/Card/TicketBoxing.vue b/src/pages/Ticket/Card/TicketBoxing.vue
index a398d82c9..05bde8977 100644
--- a/src/pages/Ticket/Card/TicketBoxing.vue
+++ b/src/pages/Ticket/Card/TicketBoxing.vue
@@ -1,9 +1,9 @@
 <script setup>
-import { useI18n } from 'vue-i18n';
-import { computed, ref, onMounted } from 'vue';
-import { useRouter } from 'vue-router';
 import axios from 'axios';
 import { date, useQuasar } from 'quasar';
+import { computed, onMounted, reactive, ref } from 'vue';
+import { useI18n } from 'vue-i18n';
+import { useRouter } from 'vue-router';
 
 const router = useRouter();
 const { t } = useI18n();
@@ -21,11 +21,10 @@ const expeditions = ref({});
 const lastExpedition = ref();
 const slide = ref(null);
 const videoList = ref([]);
-const time = ref({
+const time = reactive({
     min: 0,
     max: 24,
 });
-
 async function fetch() {
     try {
         const filter = {
@@ -77,10 +76,8 @@ async function getVideoList(expeditionId, timed) {
         videoList.value = list.reverse();
         if (list[0]) {
             slide.value = list[0].value;
-            time.value = {
-                min: parseInt(list[0].label.split(':')[0]),
-                max: parseInt(list[list.length - 1].label.split(':')[0]) + 1,
-            };
+            time.min = parseInt(list[0].label.split(':')[0]);
+            time.max = parseInt(list[list.length - 1].label.split(':')[0]) + 1;
         }
     });
 }

From cacd36a854b100c9fb55466ea4a976ebc55ae5b9 Mon Sep 17 00:00:00 2001
From: alexandre <alexandre@verdnatura.es>
Date: Thu, 27 Apr 2023 10:40:00 +0200
Subject: [PATCH 3/3] refs #5255 tests fixed

---
 src/boot/axios.js                             |  5 ++-
 src/pages/Login/LoginMain.vue                 | 34 +++++++++++--------
 test/cypress/integration/claimPhoto.spec.js   |  4 +--
 test/cypress/integration/ticketBoxing.spec.js |  8 ++---
 test/cypress/integration/workerList.spec.js   |  5 ++-
 .../cypress/integration/workerSummary.spec.js |  4 +--
 test/vitest/__tests__/boot/axios.spec.js      | 12 +++----
 7 files changed, 41 insertions(+), 31 deletions(-)

diff --git a/src/boot/axios.js b/src/boot/axios.js
index 2f454188e..cbd5e7213 100644
--- a/src/boot/axios.js
+++ b/src/boot/axios.js
@@ -58,9 +58,12 @@ const onResponseError = (error) => {
             break;
     }
 
-    if (session.isLoggedIn && response.status === 401) {
+    if (session.isLoggedIn() && response.status === 401) {
         session.destroy();
         Router.push({ path: '/login' });
+    } else if(!session.isLoggedIn())
+    {
+        message = 'login.loginError';
     }
 
     Notify.create({
diff --git a/src/pages/Login/LoginMain.vue b/src/pages/Login/LoginMain.vue
index 7fb0ee408..747fe8b14 100644
--- a/src/pages/Login/LoginMain.vue
+++ b/src/pages/Login/LoginMain.vue
@@ -48,25 +48,29 @@ const password = ref('');
 const keepLogin = ref(true);
 
 async function onSubmit() {
-    const { data } = await axios.post('Accounts/login', {
-        user: username.value,
-        password: password.value,
-    });
+    try {
+        const { data } = await axios.post('Accounts/login', {
+            user: username.value,
+            password: password.value,
+        });
 
-    if (!data) return;
+        if (!data) return;
 
-    await session.login(data.token, keepLogin.value);
+        await session.login(data.token, keepLogin.value);
 
-    quasar.notify({
-        message: t('login.loginSuccess'),
-        type: 'positive',
-    });
+        quasar.notify({
+            message: t('login.loginSuccess'),
+            type: 'positive',
+        });
 
-    const currentRoute = router.currentRoute.value;
-    if (currentRoute.query && currentRoute.query.redirect) {
-        router.push(currentRoute.query.redirect);
-    } else {
-        router.push({ name: 'Dashboard' });
+        const currentRoute = router.currentRoute.value;
+        if (currentRoute.query && currentRoute.query.redirect) {
+            router.push(currentRoute.query.redirect);
+        } else {
+            router.push({ name: 'Dashboard' });
+        }
+    } catch (e) {
+        //
     }
 }
 </script>
diff --git a/test/cypress/integration/claimPhoto.spec.js b/test/cypress/integration/claimPhoto.spec.js
index e5ea2ce25..ec41e92a3 100755
--- a/test/cypress/integration/claimPhoto.spec.js
+++ b/test/cypress/integration/claimPhoto.spec.js
@@ -43,13 +43,13 @@ describe('ClaimPhoto', () => {
         cy.get(
             '.multimediaParent > :nth-child(3) > .q-btn > .q-btn__content > .q-icon'
         ).click();
-        cy.get('.q-btn--standard > .q-btn__content > .block').click();
+        cy.get('.q-btn--unelevated > .q-btn__content > .block').click();
         cy.get('.q-notification__message').should('have.text', 'Data deleted');
 
         cy.get(
             '.multimediaParent > :nth-child(3) > .q-btn > .q-btn__content > .q-icon'
         ).click();
-        cy.get('.q-btn--standard > .q-btn__content > .block').click();
+        cy.get('.q-btn--unelevated > .q-btn__content > .block').click();
         cy.get('.q-notification__message').should('have.text', 'Data deleted');
     });
 });
diff --git a/test/cypress/integration/ticketBoxing.spec.js b/test/cypress/integration/ticketBoxing.spec.js
index 7a1ed669d..7b3bd36cc 100755
--- a/test/cypress/integration/ticketBoxing.spec.js
+++ b/test/cypress/integration/ticketBoxing.spec.js
@@ -1,5 +1,5 @@
 /// <reference types="cypress" />
-xdescribe('TicketBoxing', () => {
+describe('TicketBoxing', () => {
     beforeEach(() => {
         const ticketId = 1;
         cy.viewport(1280, 720);
@@ -26,9 +26,9 @@ xdescribe('TicketBoxing', () => {
             },
             ['2022-01-01T01-01-00.mp4', '2022-02-02T02-02-00.mp4', '2022-03-03T03-03-00.mp4']
         ).as('getVideoList');
-        cy.get('.q-list > :nth-child(3)').click();
+        cy.get('.q-list').eq(3).find('.q-item').eq(2).click();
 
-        cy.get('.q-list > :nth-child(1)').should('be.visible');
-        cy.get('.q-list > :nth-child(2)').should('be.visible');
+        cy.get('.q-list').eq(3).find('.q-item').eq(0).find('.q-range');
+        cy.get('.q-list').eq(3).find('.q-item').eq(1).find('.q-select');
     });
 });
diff --git a/test/cypress/integration/workerList.spec.js b/test/cypress/integration/workerList.spec.js
index 9f3d62448..79f2400bf 100644
--- a/test/cypress/integration/workerList.spec.js
+++ b/test/cypress/integration/workerList.spec.js
@@ -12,7 +12,10 @@ describe('WorkerList', () => {
     });
 
     it('should open the worker summary', () => {
-        cy.get('div[class="q-item__section column q-item__section--side justify-center q-pa-md"]').eq(0).click();
+        cy.get('div.text-h6')
+            .parentsUntil('div.q-card')
+            .find('div.q-card__actions')
+            .find('button').first().click();
         cy.get('div[class="header bg-primary q-pa-sm q-mb-md"').should('have.text', '1110 - Jessica Jones');
         cy.get('div[class="q-item__label q-item__label--header text-h6"]').eq(0).should('have.text', 'Basic data');
         cy.get('div[class="q-item__label q-item__label--header text-h6"]').eq(1).should('have.text', 'User data');
diff --git a/test/cypress/integration/workerSummary.spec.js b/test/cypress/integration/workerSummary.spec.js
index ae01378f6..cef26d061 100644
--- a/test/cypress/integration/workerSummary.spec.js
+++ b/test/cypress/integration/workerSummary.spec.js
@@ -6,10 +6,10 @@ describe('WorkerSummary', () => {
     });
 
     it('should load worker summary', () => {
-        cy.get('div[class="header bg-primary q-pa-sm q-mb-md"').should('have.text', '19 - salesBoss');
+        cy.get('div[class="header bg-primary q-pa-sm q-mb-md"').should('have.text', '19 - salesBoss salesBoss');
         cy.get('div[class="q-item__label q-item__label--header text-h6"]').eq(0).should('have.text', 'Basic data');
         cy.get('div[class="q-item__label q-item__label--header text-h6"]').eq(1).should('have.text', 'User data');
         cy.get('div[class="q-item__section column q-item__section--main justify-center"]').eq(0).should('have.text', 'NamesalesBossNick');
     });
 
-});
\ No newline at end of file
+});
diff --git a/test/vitest/__tests__/boot/axios.spec.js b/test/vitest/__tests__/boot/axios.spec.js
index a95c57fba..feb0d93ea 100644
--- a/test/vitest/__tests__/boot/axios.spec.js
+++ b/test/vitest/__tests__/boot/axios.spec.js
@@ -1,15 +1,15 @@
-import { vi, describe, expect, it } from 'vitest';
+import { Notify } from 'quasar';
 import { onRequest, onResponseError } from 'src/boot/axios';
-import { Notify } from 'quasar'
+import { describe, expect, it, vi } from 'vitest';
 
 vi.mock('src/composables/useSession', () => ({
     useSession: () => ({
-        getToken: () => 'DEFAULT_TOKEN'
-    }),
+        getToken: () => 'DEFAULT_TOKEN',
+        isLoggedIn: () => vi.fn(),
+        destroy: () => vi.fn(),
+    })
 }));
 
-vi.mock('src/router', () => ({}));
-
 describe('Axios boot', () => {
 
     describe('onRequest()', async () => {