Skip to content

GX-29819/test/radio-buttons #450

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Aug 26, 2023
Merged

GX-29819/test/radio-buttons #450

merged 6 commits into from
Aug 26, 2023

Conversation

igflores
Copy link

Los 3 TC pasaron correctamente

 Se actualizaron los 3 TC:
 GX-29820 | TC01: Validar visualizar el label "Yes" cuando se selecciona el radio-button "Yes"
 GX-29820 | TC02: Validar visualizar el label “Impressive” cuando se selecciona el radio-button "Impressive"
 GX-29820 | TC03: Validar que esté deshabilitado y no se pueda seleccionar el radio-button con label "No"

 On branch GX-29819/test/radio-buttons
 Your branch is up to date with 'origin/GX-29819/test/radio-buttons'.
@igflores igflores added Needs Review Ready for review by the team. Tutor Assigned I have a tutor assigned labels Aug 25, 2023
@igflores igflores requested a review from ZwitterKaneda August 25, 2023 05:30
@igflores igflores self-assigned this Aug 25, 2023
 On branch GX-29819/test/radio-buttons
 Your branch is up to date with 'origin/GX-29819/test/radio-buttons'.

 No se mostraba la ejecución de los 3 TC en el TX, era porque tenía el ID incorrecto
 Hice el cambio del ID "28921" (incorrecto), por el ID del TX correcto:"28921"
@igflores
Copy link
Author

Hola,
Hice un commit porque me di cuenta que en el fichero CI-Suite.yml pose mal el ID de mi TX, así que coloqué el ID correcto:
testExecKey: 'GX-29821'

@ZwitterKaneda ZwitterKaneda added In Review Currently being evaluated by the reviewer. and removed Needs Review Ready for review by the team. labels Aug 26, 2023
@ZwitterKaneda
Copy link
Contributor

Hola @igflores hoy revisaré tu trabajo =)

Como hablamos en las meeting que estuvimos realizando, te dejaré un comentario que le realicé (todo motivado) a otro compañero con respecto a lo que te explicaba sobre el uso del { force: true }

Donde usas el { force: true} me gustaría comentarte algo que iré comentandole a todos que lo están usando. En Cypress, esto básicamente fuerza la acción (click, check, etc.) incluso si el elemento no es actualmente interactuable (por ejemplo, está oculto, está cubierto por otro elemento, etc.). El uso de { force: true} podría considerarse un "hack" y generalmente se desaconseja a menos que no haya otra forma de interactuar con el elemento en el estado actual de la aplicación bajo prueba. Fuerza a la acción a ocurrir sin importar si sería posible en una interacción de usuario real. Esto puede llevar a pruebas que pasan, pero no necesariamente reflejan el comportamiento del usuario. Por lo tanto, si encuentras que estás usando { force: true} con frecuencia, podría valer la pena investigar por qué los elementos no son interactuables en un estado 'normal'. Podría ser útil para identificar problemas con el layout o con la lógica de la aplicación. Podrías replantear el elemento que estas obteniendo para poder hacer la acción que estas intentando realizar.

Quiero comentarte esto para que lo tengas en cuenta en tus próximos proyectos. Continúa avanzando así, vienes genial 🚀

@ZwitterKaneda ZwitterKaneda merged commit ad74868 into QA Aug 26, 2023
@ZwitterKaneda ZwitterKaneda added Approved Approved and pending merge. and removed In Review Currently being evaluated by the reviewer. labels Aug 26, 2023
@AlexanderWindecker AlexanderWindecker deleted the GX-29819/test/radio-buttons branch August 26, 2024 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Approved and pending merge. Tutor Assigned I have a tutor assigned
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants