Gx 32081 ✅ tools qa elements text box fill form and submit #456
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
added 4 commits
August 30, 2023 21:15
|
|
LauraMont
requested review from
LauraMont
and removed request for
ZwitterKaneda
LauraMont
reviewed
Sep 1, 2023
There was a problem hiding this comment.
Hola Leo! Te comento algunos puntos que nos interesan para esta review:
- Documentación: Bien editado el CI y agregado el markdown en la carpeta correcta
- US en Jira: En general esta muy bien ,solo te falto la trazabilidad entre la US y TX y agregar los TC restantes (al menos el TC 2 ). Otra cosa que no se si has hecho o lo han realizado por ser final de sprint es cerrar la US, por si acaso te comento que solo la enviamos a test passed de tener todos los casos aprobados
- Cobertura: Esta bastante bien la cobertura, muy bueno como abarcaste los casos negativos de email en uno solo
- Fixture: Nada que agregar realmente por aquí
- POM: Muy bien implementado, una alternativa que vi hace poco muy interesante para generar data y que te la escriba dentro del fixture es agregar un command. Otra opción también fue retornar una variable que contenga un objeto contenedor de las variables deseadas
- Código: Te comento mas abajo si es necesario
- PR: el mensaje del PR es muy importante 👀 No olvides usar la nomenclatura para los commits también
Realmente no hay mucho por corregir, mas que nada son sugerencias asique dejamos este PR como aprobado, buen trabajo Leo!
LauraMont
added
Approved
ZwitterKaneda
deleted the
GX-32081-✅-tools-qa-elements-text-box-fill-form-and-submit
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.
Mensaje motivador de lo que hicieron en la historia de usuario.