verdnatura/salix-front
verdnatura
/
salix-front
2
2
Fork 1
Code Issues Pull Requests 69 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 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ed0f33dbc5 - Show all commits

2
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('.vn-row > .q-field > .q-field__inner > .q-field__control').type('Warehouse Two{enter}');
cy.dataCy('Warehouse_select').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);
});