#8582 - test: e2e monitorTicket #1391

Open
jsegarra wants to merge 48 commits from e2e_monitor into dev
Member
  • Comprobar la funcionalidad de ctrl y que abra una nueva pestaña

  • Al eliminar (clear) el campo de dias en adelante,, comprobar que se ponga 0, y en el chip también

  • Validar los valores de from y to que tenga relación con scopeDays

- [x] Comprobar la funcionalidad de ctrl y que abra una nueva pestaña - [x] Al eliminar (clear) el campo de dias en adelante,, comprobar que se ponga 0, y en el chip también - [x] Validar los valores de from y to que tenga relación con scopeDays
jsegarra added 1 commit 2025-02-14 00:41:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
55841d795f
test: init e2e_monitorTicket
jsegarra changed title from test: e2e monitorTicket to WIP: test: e2e monitorTicket 2025-02-14 00:46:39 +00:00
jsegarra added 1 commit 2025-02-14 00:53:41 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a7d271bafd
test: add describe for each test
jsegarra added 2 commits 2025-03-01 02:13:17 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
97c673b5f3
test: check fields
jsegarra added 1 commit 2025-03-13 12:04:50 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
fc61c13ec4
Merge branch 'dev' into e2e_monitor
jsegarra requested review from jorgep 2025-03-14 10:49:20 +00:00
jsegarra changed title from WIP: test: e2e monitorTicket to WIP: #8582 - test: e2e monitorTicket 2025-03-14 11:56:56 +00:00
jsegarra added 7 commits 2025-03-16 11:23:26 +00:00
jsegarra added 1 commit 2025-03-16 11:36:27 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
57e80040fa
test: check params from and to
jorgep refused to review 2025-03-18 10:24:42 +00:00
Member

Pideme revisión cuando no esté en WIP

Pideme revisión cuando no esté en WIP
jsegarra added 6 commits 2025-03-24 00:43:51 +00:00
jsegarra added the
CR / Tests passed
label 2025-03-24 00:44:15 +00:00
jsegarra changed title from WIP: #8582 - test: e2e monitorTicket to #8582 - test: e2e monitorTicket 2025-03-24 00:44:19 +00:00
jsegarra added 2 commits 2025-03-24 10:15:52 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0404c00a05
Merge branch 'dev' into e2e_monitor
jsegarra requested review from jorgep 2025-03-24 10:16:08 +00:00
jorgep added 2 commits 2025-03-26 09:45:01 +00:00
jorgep requested changes 2025-03-26 10:01:03 +00:00
Dismissed
@ -707,2 +707,4 @@
:options="tableModes.filter((mode) => !mode.disable)"
/>
<QBtn
Member

Este icono solo funciona en monitores, en invoice-in, clientes... no es posible meterlo en la barra de busqueda como el resto de secciones? con un slot o algo.

Este icono solo funciona en monitores, en invoice-in, clientes... no es posible meterlo en la barra de busqueda como el resto de secciones? con un slot o algo.
Author
Member

No se por que está
No veo que yo salga en los commits y hay 26...lo quito

No se por que está No veo que yo salga en los commits y hay 26...lo quito
jsegarra marked this conversation as resolved
@ -207,6 +207,7 @@ const getLocale = (label) => {
color="primary"
style="position: fixed; z-index: 1; right: 0; bottom: 0"
icon="search"
data-cy="search"
Member

vnFilterPanel_search, veo muy general usar search

vnFilterPanel_search, veo muy general usar search
Author
Member

cambiado

cambiado
jsegarra marked this conversation as resolved
@ -0,0 +1,3 @@
Cypress.Commands.add('firstRow', (field, index = 1) =>
cy.get(`:nth-child(${index}) > [data-col-field="${field}"]`),
Member

mejor hacer .vnTable .q virtual-scroll__content resto del selector creo yo... Ser un poco más especifico y si se puede cambiar el idice de fila, ya no lo llamaría firstRow, select row o algo así.

mejor hacer .vnTable .q virtual-scroll__content resto del selector creo yo... Ser un poco más especifico y si se puede cambiar el idice de fila, ya no lo llamaría firstRow, select row o algo así.
jsegarra marked this conversation as resolved
@ -102,6 +107,13 @@ Cypress.Commands.add('waitSpinner', () => {
}
});
});
// Cypress.Commands.add('waitRightSpinner', () => {
Member

Quitar

Quitar
jsegarra marked this conversation as resolved
@ -68,6 +68,7 @@ const waitForApiReady = (url, maxRetries = 20, delay = 1000) => {
};
before(() => {
cy.viewport(1920, 720);
Member

Una resolución un poco rara no? Supongo que querías poner 1080

Una resolución un poco rara no? Supongo que querías poner 1080
Author
Member

Lo debí copiar de algún sitio.
Aprovecho para quitarlo

Lo debí copiar de algún sitio. Aprovecho para quitarlo
jsegarra marked this conversation as resolved
jsegarra added 4 commits 2025-03-27 10:23:06 +00:00
jsegarra added 2 commits 2025-03-28 07:54:12 +00:00
jsegarra added 1 commit 2025-03-31 10:37:00 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
789b0d22cc
Merge branch 'dev' into e2e_monitor
jsegarra added 2 commits 2025-03-31 23:19:16 +00:00
jsegarra added 1 commit 2025-04-01 14:41:23 +00:00
jsegarra added 1 commit 2025-04-01 16:11:15 +00:00
jsegarra added 1 commit 2025-04-01 16:38:49 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
20ccdb133c
refactor: add spinner wait to monitor ticket row count
jsegarra removed the
CR / Tests passed
label 2025-04-02 10:14:13 +00:00
jsegarra added 1 commit 2025-04-02 10:29:14 +00:00
gitea/salix-front/pipeline/pr-dev Build queued... Details
d29a27999a
test: fix monitorTicket
jsegarra added 1 commit 2025-04-02 10:29:47 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
c2d56ee002
Merge branch 'dev' into e2e_monitor
jsegarra added 2 commits 2025-04-02 12:26:18 +00:00
jsegarra requested review from jorgep 2025-04-02 17:11:01 +00:00
jsegarra added the
CR / Tests passed
label 2025-04-02 17:11:06 +00:00
jsegarra added 1 commit 2025-04-02 17:18:52 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2afa0765b9
refactor: remove unused fields from monitorTicket test cases
jorgep requested changes 2025-04-04 09:41:26 +00:00
Dismissed
@ -0,0 +40,4 @@
cy.intercept('GET', '**/Departments*').as('filterDepartment');
cy.wait('@filterDepartment').then(() => {
cy.clickOption();
cy.countTableRows('eq', 13);
Member

Creo que es mejor validar el contenido de las columnas. puedes usar validateVnTableCol creo que se llama el comando

Creo que es mejor validar el contenido de las columnas. puedes usar validateVnTableCol creo que se llama el comando
Author
Member

Porque es mucho trabajo y porque esa responsabilidad de devolver los datos correctos debe ser del back, de si las relaciones y los wheres estan bien configurados

Porque es mucho trabajo y porque esa responsabilidad de devolver los datos correctos debe ser del back, de si las relaciones y los wheres estan bien configurados
Member

Si te cambian las fixtures , fallará el test, lo digo por eso. Con la fn que te digo no es mucho trabajo, si te refieres a que se gasta mucho tiempo en lanzar el test, eso sí. Pregunta a @juan

Si te cambian las fixtures , fallará el test, lo digo por eso. Con la fn que te digo no es mucho trabajo, si te refieres a que se gasta mucho tiempo en lanzar el test, eso sí. Pregunta a @juan
Author
Member

Lo hablamos el lunes

Lo hablamos el lunes
Member

Cómo has quedado con Juan/Javi?

Cómo has quedado con Juan/Javi?
@ -0,0 +43,4 @@
cy.countTableRows('eq', 13);
});
cy.get(`${clientFk} [data-cy="_select"]`).type('Bruce Banner');
Member

Porque no lo separas en its? o cambia el nombre del test load layout and filter

Porque no lo separas en its? o cambia el nombre del test load layout and filter
Author
Member

Not working para its
cy.get(${clientFk}).its('[data-cy="_select"]').type('Bruce Banner');

Not working para its cy.get(`${clientFk}`).its('[data-cy="_select"]').type('Bruce Banner');
Member

Me refería a separarlo en varios tests.

Me refería a separarlo en varios tests.
Author
Member

ahhh vale, en it
Porque buscamos comprimir la logica en el menor numero de iteracones o its para ganar en performance y evitar logins + accesos a a la ruta innecesarios, y asi ahorrar tiempo y recursos.

ahhh vale, en it Porque buscamos comprimir la logica en el menor numero de iteracones o its para ganar en performance y evitar logins + accesos a a la ruta innecesarios, y asi ahorrar tiempo y recursos.
Member

Vale, pero cambia el nombre del test.

Vale, pero cambia el nombre del test.
Author
Member

Lo he cambiado a Should filter by field

Lo he cambiado a Should filter by field
jorgep marked this conversation as resolved
@ -0,0 +108,4 @@
cy.url().should('not.include', 'pending');
});
it('Cols', () => {
Member

should open the descriptorProxy and SummaryPopup. (o algo así)

should open the descriptorProxy and SummaryPopup. (o algo así)
jsegarra marked this conversation as resolved
jorgep requested changes 2025-04-04 09:45:39 +00:00
Dismissed
@ -0,0 +29,4 @@
cy.clickOption();
});
});
it('should filter by filter panel', () => {
Member

Juan dijo que no testeáramos los filtros de cada sección , de todas maneras aquí solo estas comprobando que se han añadido/quitado los parámetros en la url(Parece más de VnFilterPanel). Sería ver que los datos han sido filtrados. El de daysAgo como tiene lógica si se podría testear.

Juan dijo que no testeáramos los filtros de cada sección , de todas maneras aquí solo estas comprobando que se han añadido/quitado los parámetros en la url(Parece más de VnFilterPanel). Sería ver que los datos han sido filtrados. El de daysAgo como tiene lógica si se podría testear.
Author
Member

En la reunión dijimos de no crear una tarea para hacer el test de todos los filtros, pero eso no quita que se haga donde haga falta no?
De todas maneras, veo correcto lo que comentas de que esa comprobación debería ser de MonitorTicketFilter usando vitest.
Si te parece, creo tarea para esto ultimo

En la reunión dijimos de no crear una tarea para hacer el test de todos los filtros, pero eso no quita que se haga donde haga falta no? De todas maneras, veo correcto lo que comentas de que esa comprobación debería ser de MonitorTicketFilter usando vitest. Si te parece, creo tarea para esto ultimo
Member

Lo que te diga @juan

Lo que te diga @juan
Author
Member

Lo he quitado, asi que no haria falta revisar nada mas

Lo he quitado, asi que no haria falta revisar nada mas
Member

Sigues validando filtros, lo que te diga Juan

Sigues validando filtros, lo que te diga Juan
jsegarra added 2 commits 2025-04-04 11:08:19 +00:00
jsegarra requested review from jorgep 2025-04-04 11:08:20 +00:00
jorgep requested changes 2025-04-04 11:12:41 +00:00
jorgep left a comment
Member

El tema de los filtros prefiero que lo hables con Juan.

El tema de los filtros prefiero que lo hables con Juan.
jsegarra added 1 commit 2025-04-07 13:36:02 +00:00
gitea/salix-front/pipeline/pr-dev This commit is unstable Details
0440115d5a
Merge branch 'dev' into e2e_monitor
jsegarra added 5 commits 2025-04-09 20:44:19 +00:00
jsegarra requested review from jorgep 2025-04-09 21:20:22 +00:00
jorgep requested changes 2025-04-10 16:16:10 +00:00
@ -0,0 +1,6 @@
Cypress.Commands.add('checkVisibleDescriptor', (alias) =>
cy
.get(`[data-cy="${alias}Descriptor"] [data-cy="vnDescriptor"] > .header`)
Member

[data-cy="${alias}Descriptor"] [data-cy="vnDescriptor"] no vale así?

[data-cy="${alias}Descriptor"] [data-cy="vnDescriptor"] no vale así?
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
2 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#1391
No description provided.