verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 77 Releases Wiki Activity

#8582 - test: e2e monitorTicket #1391

Merged
jsegarra merged 51 commits from e2e_monitor into dev 2025-04-15 21:04:49 +00:00
Conversation 2 Commits 51 Files Changed 21 +5 -83
2 changed files with 5 additions and 83 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 0a90c62369 - Show all commits

8
test/cypress/integration/monitor/clientActions.spec.js
View File

@ -8,7 +8,7 @@ describe('Monitor Clients actions', () => {
cy.intercept('GET', '**/SalesMonitors/ordersFilter*').as('ordersFilter');
cy.intercept('GET', '**/SalesMonitors/clientsFilter*').as('clientsFilter');
});
it('Should load layout', () => {
it('Should filter by field', () => {
cy.get('.q-page').should('be.visible');
cy.dataCy('recentOrderActions').within(() => {
@ -31,19 +31,19 @@ describe('Monitor Clients actions', () => {
.should('have.text', 'Recent order actions');
cy.dataCy('From_inputDate').should('have.value', '01/01/2001');
cy.dataCy('To_inputDate').should('have.value', '01/01/2001');
cy.get(`${clientFk} [data-cy="_select"]`).click();
cy.get(`${clientFk}`).its('[data-cy="_select"]').click();
cy.clickOption();
cy.countTableRows('eq', 0);
cy.get(`${clientFk} .q-field .q-field__control > :nth-child(2)`).click();
jorgep commented 2025-04-14 12:37:58 +00:00
Review
Copy Link

Estos dos tests que comprueban el título no le veo sentido.

Estos dos tests que comprueban el título no le veo sentido.
jsegarra commented 2025-04-14 14:01:21 +00:00
Review
Copy Link

Validan que las tablas tiene titulo de cabecera.
Me pareció interesante y es una validación que no ralentiza los tests.
Es cierto que no lo validamos pero creo que no hay mas casos como este.
Por darle más valor al test

Validan que las tablas tiene titulo de cabecera. Me pareció interesante y es una validación que no ralentiza los tests. Es cierto que no lo validamos pero creo que no hay mas casos como este. Por darle más valor al test
cy.get(`${departmentFk} [data-cy="_select"]`).type('VIP').trigger('enter');
cy.get(`${departmentFk}`).its('[data-cy="_select"]').type('VIP').trigger('enter');
cy.intercept('GET', '**/Departments*').as('filterDepartment');
cy.wait('@filterDepartment').then(() => {
cy.clickOption();
cy.countTableRows('eq', 13);
jorgep marked this conversation as resolved Outdated
jorgep commented 2025-04-04 09:41:15 +00:00
Outdated
Review
Copy Link

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
jsegarra commented 2025-04-04 11:00:45 +00:00
Outdated
Review
Copy Link

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
jorgep commented 2025-04-04 11:09:07 +00:00
Outdated
Review
Copy Link

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
jsegarra commented 2025-04-04 12:13:24 +00:00
Outdated
Review
Copy Link

Lo hablamos el lunes

Lo hablamos el lunes
jorgep commented 2025-04-10 16:13:38 +00:00
Outdated
Review
Copy Link

Cómo has quedado con Juan/Javi?

Cómo has quedado con Juan/Javi?
jsegarra commented 2025-04-14 08:19:06 +00:00
Outdated
Review
Copy Link

Te he pegado una captura de lo que he hablado con Javi/Alex

Te he pegado una captura de lo que he hablado con Javi/Alex
});
cy.get(`${clientFk} [data-cy="_select"]`).type('Bruce Banner');
cy.get(`${clientFk}`).its('[data-cy="_select"]').type('Bruce Banner');
jorgep marked this conversation as resolved Outdated
jorgep commented 2025-04-04 09:13:17 +00:00
Outdated
Review
Copy Link

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
jsegarra commented 2025-04-04 11:07:02 +00:00
Outdated
Review
Copy Link

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');
jorgep commented 2025-04-04 11:10:50 +00:00
Outdated
Review
Copy Link

Me refería a separarlo en varios tests.

Me refería a separarlo en varios tests.
jsegarra commented 2025-04-04 12:13:06 +00:00
Outdated
Review
Copy Link

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.
jorgep commented 2025-04-04 12:22:31 +00:00
Outdated
Review
Copy Link

Vale, pero cambia el nombre del test.

Vale, pero cambia el nombre del test.
jsegarra commented 2025-04-07 07:44:37 +00:00
Outdated
Review
Copy Link

Lo he cambiado a Should filter by field

Lo he cambiado a Should filter by field
👍 1
cy.intercept('GET', '**/Clients*').as('filterClient');
cy.wait('@filterClient').then(() => {
cy.clickOption();

80
test/cypress/integration/monitor/monitorTicket.spec.js
View File

@ -29,86 +29,8 @@ describe('Monitor Tickets Table', () => {
cy.clickOption();
});
});
it('should filter by filter panel', () => {
const testCases = [
{
name: 'Client id',
selector: 'Client id_input',
value: '1101',
urlParam: 'clientFk',
},
{
name: 'Agency',
selector: 'Agency_select',
value: 'inhouse pickup',
urlParam: 'agencyModeFk',
isSelect: true,
},
{
name: 'State',
selector: 'State_select',
value: 'Libre',
urlParam: 'state',
isSelect: true,
},
{
name: 'Department',
selector: 'Department_select',
value: 'COMPRAS',
urlParam: 'departmentFk',
isSelect: true,
},
];
testCases.forEach(
({
selector,
value,
urlParam,
intercept = interceptSalesFilter,
isSelect,
}) => {
if (intercept) intercept();
jorgep marked this conversation as resolved Outdated
jorgep commented 2025-04-04 09:45:36 +00:00
Outdated
Review
Copy Link

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.
jsegarra commented 2025-04-04 10:57:45 +00:00
Outdated
Review
Copy Link

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
jorgep commented 2025-04-04 11:06:51 +00:00
Outdated
Review
Copy Link

Lo que te diga @juan

Lo que te diga @juan
jsegarra commented 2025-04-04 11:47:07 +00:00
Outdated
Review
Copy Link

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

Lo he quitado, asi que no haria falta revisar nada mas
jorgep commented 2025-04-10 16:14:35 +00:00
Outdated
Review
Copy Link

Sigues validando filtros, lo que te diga Juan

Sigues validando filtros, lo que te diga Juan
jsegarra commented 2025-04-14 08:18:38 +00:00
Outdated
Review
Copy Link

He hablado con Javi y Juan al respecto
Te adjunto una foto
Pero ya no deberían estar validándose

He hablado con Javi y Juan al respecto Te adjunto una foto Pero ya no deberían estar validándose
jsegarra commented 2025-04-14 08:19:59 +00:00
Outdated
Review
Copy Link

Solo hay 1, creo que es asumible

Solo hay 1, creo que es asumible
jorgep commented 2025-04-14 09:04:04 +00:00
Outdated
Review
Copy Link

Alex dice en la foto que si es de vnTable no se validen. Juan y Javi que han dicho?

Alex dice en la foto que si es de vnTable no se validen. Juan y Javi que han dicho?
if (isSelect) {
cy.dataCy(selector).click();
cy.get('.q-item__label').contains(value).click();
} else {
cy.dataCy(selector).type(value);
}
cy.searchInFilterPanel();
cy.url().should('include', urlParam);
cy.cleanFilterPanel();
},
);
// Null to true
cy.get('[data-cy="vnCheckboxMy team"]').click();
cy.get('[data-cy="vnCheckboxWith problems"]').click();
cy.get('[data-cy="vnCheckboxPending"]').click();
cy.searchInFilterPanel();
cy.url().should('include', 'myTeam');
cy.url().should('include', 'problems');
cy.url().should('include', 'pending');
// True to false
cy.get('[data-cy="vnCheckboxMy team"]').click();
cy.get('[data-cy="vnCheckboxWith problems"]').click();
cy.get('[data-cy="vnCheckboxPending"]').click();
cy.searchInFilterPanel();
cy.url().should('include', 'myTeam');
cy.url().should('include', 'problems');
cy.url().should('include', 'pending');
// False to Null
cy.get('[data-cy="vnCheckboxMy team"]').click();
cy.get('[data-cy="vnCheckboxWith problems"]').click();
cy.get('[data-cy="vnCheckboxPending"]').click();
cy.searchInFilterPanel();
cy.url().should('not.include', 'myTeam');
cy.url().should('not.include', 'problems');
cy.url().should('not.include', 'pending');
});
it('Cols', () => {
it('should open the descriptorProxy and SummaryPopup', () => {
cy.getRowCol('totalProblems');
cy.getRowCol('id').find('span').should('have.class', 'link').click();