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

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

@ -14,12 +14,12 @@ describe('ZoneWarehouse', () => {
it('should throw an error if the warehouse chosen is already put in the zone', () => {
cy.addBtnClick();
cy.selectOption('[data-cy="Warehouse_select"]', 'Warehouse Two');
cy.get('[data-cy="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);
});
// https://redmine.verdnatura.es/issues/8425
it.skip('should create & remove a warehouse', () => {
it('should create & remove a warehouse', () => {
cy.addBtnClick();
cy.fillInForm(data);
cy.get(saveBtn).click();