8422-createVehicleDms #1614

Open
jtubau wants to merge 36 commits from 8422-createVehicleDms into dev
Member

PR BACK: verdnatura/salix#3569
PR BACK (CERRADA): verdnatura/salix#3435

✅PR BACK: https://gitea.verdnatura.es/verdnatura/salix/pulls/3569 ❎PR BACK **(CERRADA)**: _https://gitea.verdnatura.es/verdnatura/salix/pulls/3435_
jtubau added 13 commits 2025-03-20 11:19:00 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
8f0a4da245
feat: refs #8422 add Vehicle DMS import functionality and routing
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
b259988f90
test: refs #8422 add test for vehicleDms
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
7ab2e28dc7
fix: refs #8422 update filter structure in VnDmsList component
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
5d356428d3
refactor: refs #8422 enhance error handling in save function and update selectors in tests
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
de9f1bca9a
Merge branch 'dev' into 8422-createVehicleDms
jtubau added 1 commit 2025-03-20 12:34:24 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
cb99c50f33
fix: refs #8422 vehicleDms tests
jtubau added 1 commit 2025-03-24 09:36:13 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
1b86d34976
Merge branch 'dev' into 8422-createVehicleDms
jtubau requested review from jorgep 2025-03-24 09:48:41 +00:00
Author
Member

Cree una nueva PR, lo comente con Alex y la otra PR se ve que como lleva mucho tiempo creada, como que jenkins "la borra" y ya no le lanza los test.

Esta es la PR original: #1377

Cree una nueva PR, lo comente con Alex y la otra PR se ve que como lleva mucho tiempo creada, como que jenkins "la borra" y ya no le lanza los test. Esta es la PR original: https://gitea.verdnatura.es/verdnatura/salix-front/pulls/1377
jorgep reviewed 2025-03-26 09:24:45 +00:00
@ -44,6 +44,7 @@ export default defineConfig({
supportFile: 'test/cypress/support/index.js',
videosFolder: 'test/cypress/videos',
downloadsFolder: 'test/cypress/downloads',
tmpUploadFolder: '../salix/storage/tmp/dms',
Member

@alexm das el ok?

@alexm das el ok?
Member

@jorgep No, diria que hay que usar este path: test/cypress/storage/tmp/dms esta dentro de lilium

@jorgep No, diria que hay que usar este path: `test/cypress/storage/tmp/dms` esta dentro de lilium
jtubau marked this conversation as resolved
jorgep reviewed 2025-03-26 09:25:46 +00:00
@ -87,0 +89,4 @@
delete dms.value.files;
return response;
} catch (e) {
throw new Error(e.message);
Member

Muestra notificación al usuario correctamente no?

Muestra notificación al usuario correctamente no?
jtubau marked this conversation as resolved
jorgep requested changes 2025-03-26 09:36:31 +00:00
@ -261,6 +262,7 @@ function deleteDms(dmsFk) {
await axios.post(`${$props.deleteModel ?? $props.model}/${dmsFk}/removeFile`);
Member

Si falla el post, mostrarás la notificación igualmente, trycatch para no mostrar la notificación si falla.

Si falla el post, mostrarás la notificación igualmente, trycatch para no mostrar la notificación si falla.
jtubau marked this conversation as resolved
@ -0,0 +30,4 @@
dmsId.value = null;
emit('onDataSaved');
} catch (e) {
throw new Error(e.message);
Member

Revisar si se muestra el el error al usuario

Revisar si se muestra el el error al usuario
jtubau marked this conversation as resolved
@ -20,1 +20,4 @@
vehicleStateFk: Estado
dms:
import: Importar desde existente
selectDocId: Introduzca id de gestion documental
Member

falta un acento, preguntale a J. Gallego si ve bien esa traducción.

falta un acento, preguntale a J. Gallego si ve bien esa traducción.
jtubau marked this conversation as resolved
@ -370,6 +370,21 @@ Cypress.Commands.add('validateContent', (selector, expectedValue) => {
cy.get(selector).should('have.text', expectedValue);
});
Cypress.Commands.add('containContent', (selector, expectedValue) => {
Member

Hazlo más flexible. permite que se pueda evaluar value por ejemplo.

Hazlo más flexible. permite que se pueda evaluar value por ejemplo.
Author
Member

Lo mejoramos en otra tarea, de hecho ese método ya esta fusionado en otra rama que también lo tenia, y necesito para los test de la nueva sección cambios de VnDms y VnDmsList que lleva esta PR.

Lo mejoramos en otra tarea, de hecho ese método ya esta fusionado en otra rama que también lo tenia, y necesito para los test de la nueva sección cambios de VnDms y VnDmsList que lleva esta PR.
jorgep marked this conversation as resolved
@ -373,0 +379,4 @@
});
});
Cypress.Commands.add('openActionDescriptor', (opt) => {
Member

Hay un comando muy parecido ya, selectDescriptorOption o algo así.

Hay un comando muy parecido ya, selectDescriptorOption o algo así.
Author
Member

no es mío, ese comando ya estaba, solo que al poner arriba el containContent que si es mío, habrá detectado cambios en el de abajo

no es mío, ese comando ya estaba, solo que al poner arriba el containContent que si es mío, habrá detectado cambios en el de abajo
jorgep marked this conversation as resolved
jtubau added 2 commits 2025-03-27 14:10:34 +00:00
jtubau added 1 commit 2025-03-28 07:58:18 +00:00
jtubau added 1 commit 2025-04-01 04:56:06 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
f0ef9b42c9
refactor: refs #8422 removed error-related changes
jtubau added 1 commit 2025-04-01 04:57:59 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
f6360e29af
Merge branch 'dev' into 8422-createVehicleDms
jtubau added 1 commit 2025-04-02 11:53:05 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
0d056d3ed6
fix: refs #8422 router/module/route.js
jtubau added 1 commit 2025-04-03 05:17:55 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
5cc599a0e6
Merge branch 'dev' into 8422-createVehicleDms
jtubau added 2 commits 2025-04-03 08:28:58 +00:00
jtubau added 1 commit 2025-04-03 11:02:26 +00:00
jtubau added 1 commit 2025-04-03 12:11:00 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
5e2a74125f
Merge branch 'dev' into 8422-createVehicleDms
jtubau added 1 commit 2025-04-03 13:06:53 +00:00
jtubau added 1 commit 2025-04-04 10:06:37 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
6ddcf57b08
Merge branch 'dev' into 8422-createVehicleDms
jtubau added 1 commit 2025-04-07 07:25:19 +00:00
jtubau requested review from jorgep 2025-04-07 08:37:25 +00:00
jorgep requested changes 2025-04-09 08:34:44 +00:00
@ -91,0 +93,4 @@
delete dms.value.files;
return response;
} catch (e) {
throw new Error(e.message);
Member

No puedes usar throw e y ya está?

No puedes usar throw e y ya está?
@ -264,0 +267,4 @@
rows.value.splice(index, 1);
notify(t('globals.dataDeleted'), 'positive');
} catch (e) {
throw new Error(e.message);
Member

throw e?

throw e?
@ -0,0 +33,4 @@
dmsId.value = null;
emit('onDataSaved');
} catch (e) {
throw new Error(e.message);
Member

throw e?

throw e?
@ -22,0 +17,4 @@
descriptorRouteSubtitle: '[data-cy="vnDescriptor_subtitle"]',
descriptorAgencyAndSupplierTitle: '[data-cy="vnDescriptor_description"]',
descriptorInvoiceInTitle: '[data-cy="vnDescriptor_title"]',
descriptorOpenSummaryBtn: '.q-menu > .descriptor [data-cy="openSummaryBtn"]',
Member

Si vas directo al dataCy no funciona?

Si vas directo al dataCy no funciona?
@ -70,0 +83,4 @@
});
});
describe('Agency route pop-ups', () => {
Member

Veo mucha duplicidad en los tests de los popups, crea una fn que puedas reutilizar

Veo mucha duplicidad en los tests de los popups, crea una fn que puedas reutilizar
@ -71,2 +71,3 @@
case 'checkbox':
cy.get(selector).should('be.visible').click().click();
cy.get(selector).should('be.visible').click()
cy.get(selector).click();
Member

Esto porque? Tampoco lo veía bien antes pero porque haces click 2 veces? para marcar y desmarcar??

Esto porque? Tampoco lo veía bien antes pero porque haces click 2 veces? para marcar y desmarcar??
@ -0,0 +72,4 @@
cy.checkNotification('Data saved');
});
it('Should import DMS', () => {
Member

Create, import, edit y delete son casi iguales, crea una fn para no duplicar código

Create, import, edit y delete son casi iguales, crea una fn para no duplicar código
@ -0,0 +98,4 @@
cy.containContent(selectors.lastRowReference, 'Vehicle:3333-BAT');
});
it('Should redirect to worker summary from worker descriptor pop-up', () => {
Member

Lo mismo que los popup del otro archivo de tests

Lo mismo que los popup del otro archivo de tests
jtubau added 1 commit 2025-04-11 06:26:50 +00:00
jtubau added 1 commit 2025-04-11 07:01:52 +00:00
jtubau added 1 commit 2025-04-11 08:43:52 +00:00
jtubau added 1 commit 2025-04-11 10:02:50 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
2159ce0897
Merge branch 'dev' into 8422-createVehicleDms
jtubau added 2 commits 2025-04-11 10:20:06 +00:00
jtubau added 1 commit 2025-04-11 12:01:14 +00:00
Some checks reported warnings
gitea/salix-front/pipeline/pr-dev This commit is unstable
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#1614
No description provided.