Skip to content

Gx 29736/test/radio buttons #457

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 9 commits into from
Sep 2, 2023
Merged

Conversation

Missael1494
Copy link

Se llevaron a cabo 3 casos de prueba los cuales pasaron exitosamente

3 Test Cases were added to the Story Covergae
TC1: Validar visualizar el label YES cuando se selecciona el radio button Yes
TC2: Validar visualizar el label Impressive cuando se selecciona el radio button Impressive
TC3: Validar que el radio button con label "NO" este deshabilitado y no se pueda seleccionar

On branch GX-29736/test/radio-buttons
Changes to be committed:
	modified:   cypress/e2e/Tests/Elements/GX-29736_RadioButtons.cy.js
3 Tests cases were added to the story Coverage

TC1: Validar visualizar el label YES cuando se selecciona el radio button Yes
TC2: Validar visualizar el label Impressive cuando se selecciona el radio button Impressive
TC3: Validar que el radio button con label "NO" este deshabilitado y no se pueda seleccionar

On branch GX-29736/test/radio-buttons
Changes to be committed:
	modified:   cypress/e2e/Tests/Elements/GX-29736_RadioButtons.cy.js
Changes to be committed:
	modified:   .github/workflows/CI-Suite.yml
	modified:   cypress/e2e/Tests/Elements/GX-29736_RadioButtons.cy.js
@Missael1494 Missael1494 added Needs Review Ready for review by the team. In Review Currently being evaluated by the reviewer. labels Aug 30, 2023
@Missael1494 Missael1494 requested a review from apscecilia August 30, 2023 15:40
@Missael1494 Missael1494 self-assigned this Aug 30, 2023
@apscecilia
Copy link
Contributor

Hola Missael, espero que estés bien. Estoy contenta por tu esfuerzo y dedicación al completar los casos de prueba en Cypress. Quiero compartirte algunos comentarios que te ayudarán a mejorar tus habilidades en la automatización de pruebas y hacer que tus futuros trabajos sean aún más sólidos.

Puntos Positivos:

  • Manejaste correctamente el CI.suite.
  • Respetaste el formato de la creación de la US, así como su estructura, lo que ayuda a entender rápidamente qué está siendo probado en cada caso.
  • Agregaste el archivo Markdown (.md) con los detalles de tu US, eso facilita el contexto y analisis de tu US.

Áreas de Mejora:

  • Elimina los comentarios que no sean informativos para que quede más limpio el código

Sugerencias:

  • Puedes ir ampliando tus commit para darle más detalle de tu trabajo, asi como tambien la descripción de tu PR

Estoy segura de que con tu dedicación y esfuerzo, seguirás mejorando tus habilidades en la automatización de pruebas.
¡Sigue adelante vas bien!

@apscecilia apscecilia added Approved Approved and pending merge. Tutor Assigned I have a tutor assigned and removed Needs Review Ready for review by the team. In Review Currently being evaluated by the reviewer. labels Sep 1, 2023
@ZwitterKaneda
Copy link
Contributor

Hola @Missael1494 te dejaré esto que le dejé como comentario a otro compañero.

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.

Continua avanzando 🚀

@ZwitterKaneda ZwitterKaneda merged commit 48519b5 into QA Sep 2, 2023
@ZwitterKaneda ZwitterKaneda deleted the GX-29736/test/radio-buttons branch January 9, 2025 13:00
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.

3 participants