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

fix: #8425 fixed zoneWarehouse e2e test not working #1304

Merged
provira merged 11 commits from 8423-e2eZoneWarehouse into dev 2025-01-31 07:49:21 +00:00
Conversation 1 Commits 11 Files Changed 1 +1 -2
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 95f96d7ebd - Show all commits

3
test/cypress/integration/zone/zoneWarehouse.spec.js
View File

@ -14,7 +14,7 @@ describe('ZoneWarehouse', () => {
it('should throw an error if the warehouse chosen is already put in the zone', () => {
cy.addBtnClick();
cy.get('[data-cy="Warehouse_select"]').type('Warehouse Two{enter}');
cy.get('.vn-row > .q-field > .q-field__inner > .q-field__control').type('Warehouse Two{enter}');
jsegarra commented 2025-01-28 22:31:25 +00:00
Outdated
Review
Copy Link

revisa el proyecto si esa es la manera correcta.
Imagino que habrán 2 estilos, una pista, esta es la manera antigua, hemos evolucionado
Sin mirar el código se te ocurre alguna manera de porque está mal? Por ejemplo, es engorroso estar escribiendo siempre data-cy, que pasa si mañana decidimos data-test-cy, un montón de reemplazos no?

revisa el proyecto si esa es la manera correcta. Imagino que habrán 2 estilos, una pista, esta es la manera antigua, hemos evolucionado Sin mirar el código se te ocurre alguna manera de porque está mal? Por ejemplo, es engorroso estar escribiendo siempre data-cy, que pasa si mañana decidimos data-test-cy, un montón de reemplazos no?
jsegarra commented 2025-01-29 07:53:13 +00:00
Outdated
Review
Copy Link

Esto está bien
cy.get('[data-cy="Warehouse_select"]').type('Warehouse Two{enter}');`
Pero puede estar mejor, por eso digo que revisa como usamos el cy.get para seleccionar atributos data-cy
Ahora te adjunto el report con mi propuesta para validar que pasa

Esto está bien `cy.get('[data-cy="Warehouse_select"]').type('Warehouse` Two{enter}');` Pero puede estar mejor, por eso digo que revisa como usamos el cy.get para seleccionar atributos data-cy Ahora te adjunto el report con mi propuesta para validar que pasa
cy.get(saveBtn).click();
cy.checkNotification(dataError);
});
@ -24,7 +24,6 @@ describe('ZoneWarehouse', () => {
cy.fillInForm(data);
cy.get(saveBtn).click();
cy.get('.q-mt-lg > .q-btn--standard').click();
cy.get('tbody > :nth-child(2) > :nth-child(2) > .q-icon').click();
cy.get('[title="Confirm"]').click();