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
Member
No description provided.
provira added 1 commit 2025-01-28 13:47:14 +00:00
gitea/salix-front/pipeline/pr-dev Build queued... Details
7748df1384
fix: refs #8423 fixed zoneWarehouse e2e test not working
provira added 1 commit 2025-01-28 13:47:20 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
1a3446dd07
Merge branch 'dev' into 8423-e2eZoneWarehouse
provira requested review from jsegarra 2025-01-28 13:54:12 +00:00
provira changed title from fix: #8423 fixed zoneWarehouse e2e test not working to fix: #8425 fixed zoneWarehouse e2e test not working 2025-01-28 14:03:16 +00:00
jsegarra requested changes 2025-01-28 22:31:28 +00:00
Dismissed
@ -15,3 +15,3 @@
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}');
Member

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?
Member

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
provira added 1 commit 2025-01-29 06:45:54 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
a013152a09
Merge branch 'dev' into 8423-e2eZoneWarehouse
provira added 2 commits 2025-01-29 07:32:39 +00:00
provira added 1 commit 2025-01-29 07:32:48 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9c790f734c
Merge branch 'dev' into 8423-e2eZoneWarehouse
provira requested review from jsegarra 2025-01-29 07:40:13 +00:00
Member

el reporte

el reporte
provira added 2 commits 2025-01-30 11:39:19 +00:00
provira added 1 commit 2025-01-30 11:40:17 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
47603d1315
Merge branch 'dev' into 8423-e2eZoneWarehouse
jsegarra approved these changes 2025-01-30 13:58:52 +00:00
provira added 1 commit 2025-01-31 04:51:55 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
e13f46613f
Merge branch 'dev' into 8423-e2eZoneWarehouse
provira scheduled this pull request to auto merge when all checks succeed 2025-01-31 04:55:20 +00:00
provira canceled auto merging this pull request when all checks succeed 2025-01-31 04:55:30 +00:00
provira added 1 commit 2025-01-31 05:21:13 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
023593eb45
Merge branch 'dev' into 8423-e2eZoneWarehouse
provira merged commit bd2db2ef58 into dev 2025-01-31 07:49:20 +00:00
provira deleted branch 8423-e2eZoneWarehouse 2025-01-31 07:49:22 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#1304
No description provided.