Gx 21524 ✅ tools qa widgets dropdown select menu #342
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jusantamaria
requested review from
ZwitterKaneda
and removed request for
mjorabdo
LauraMont
reviewed
Jun 25, 2023
There was a problem hiding this comment.
¡Hola Julian! Cómo estas
Voy a comentar algunos puntos de la ejecución de este US para la revisión:
- Cobertura de Pruebas: muy buena cobertura, pero los títulos son un poco confusos. Recuerda que en el titulo de un TC queremos notificar una situación que queremos probar en el TC1 por ejemplo puedes decir "Verificar seleccionar un valor del desplegable 'Seleccionar Valor'". Me gusta dejar por defecto el caso HP y declarar un caso negativo con el título así "Verificar intentar... con..."
- US en Jira: Genial. Me encanta que no se olviden de completar el US en Jira, ¡es realmente importante!
- Código (principalmente selectores para este caso): Para los TC1,2 y 3 se puede usar el id de cada dropdown, hay que tratar de evitar las clases porque tienden a cambiar. Lo mismo para la validacion de la opcion (en el caso de que solo se pueda elegir una opcion) se puede esperar que dentro de este elemento se encuentre la opcion seleccionada.
- Mejoras para el próximo US: ¡Lo estás haciendo muy bien! Puedes explorar seleccionar una opción aleatoria. este enfoque es muy útil para tener un código más robusto y más adaptable a los cambios que se pueden hacer si por ejemplo cambian las opciones en los desplegables.
Estas pruebas estan aprobadas, voy a poner el labbel para marcar esto y cerrar tu US en Jira. Si quieres puedes hacer las sugerencias de mejoras en otro PR
¡Gran trabajo Julian ,sigue así!
LauraMont
added
Approved
|
Hola Laura! Muchas gracias por tus correcciones, voy a intentar mejorar los puntos mencionados en las siguientes US. |
AlexanderWindecker
deleted the
GX-21524-✅-tools-qa-widgets-dropdown-select-menu
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
En esta US se han probado diferentes tipos de Dropdowns. En los TC he aprendido como hacer aserciones de inputs con más de un valor.