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 +3 -4
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -14,17 +14,16 @@ describe('ZoneWarehouse', () => {
it('should throw an error if the warehouse chosen is already put in the zone', () => { it('should throw an error if the warehouse chosen is already put in the zone', () => {
cy.addBtnClick(); cy.addBtnClick();
cy.selectOption('[data-cy="Warehouse_select"]', 'Warehouse Two'); 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.get(saveBtn).click();
cy.checkNotification(dataError); cy.checkNotification(dataError);
}); });
// https://redmine.verdnatura.es/issues/8425
it.skip('should create & remove a warehouse', () => { it('should create & remove a warehouse', () => {
cy.addBtnClick(); cy.addBtnClick();
cy.fillInForm(data); cy.fillInForm(data);
cy.get(saveBtn).click(); cy.get(saveBtn).click();
cy.get('.q-mt-lg > .q-btn--standard').click(); cy.get('.q-mt-lg > .q-btn--standard').click();
cy.get('tbody > :nth-child(2) > :nth-child(2) > .q-icon').click(); cy.get('tbody > :nth-child(2) > :nth-child(2) > .q-icon').click();
cy.get('[title="Confirm"]').click(); cy.get('[title="Confirm"]').click();