Skip to content

Gx3 845 ⚡️ tools qa widgets auto complete #570

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 10 commits into from
Dec 26, 2023

Conversation

arjusgit
Copy link

@arjusgit arjusgit commented Dec 19, 2023

  • Para testear la funcionalidad de Auto-Complete se diseñaron 8 casos de prueba. Todos ellos fueron ejecutados de manera exitosa.
  • Se realizó un Full Update del Repositorio Local con la rama QA antes del Push
  • Se verificó que no hay conflictos en el código cuando se realizó el merge local

@arjusgit arjusgit self-assigned this Dec 19, 2023
@arjusgit arjusgit added the Needs Review Ready for review by the team. label Dec 19, 2023
@arjusgit arjusgit added Needs Review Ready for review by the team. and removed Needs Review Ready for review by the team. labels Dec 19, 2023
@apscecilia apscecilia requested review from apscecilia and removed request for ZwitterKaneda December 21, 2023 21:01
@apscecilia apscecilia added In Review Currently being evaluated by the reviewer. and removed Needs Review Ready for review by the team. labels Dec 21, 2023
@apscecilia
Copy link
Contributor

apscecilia commented Dec 21, 2023

@arjusgit, espero que te guste incursionar en el área de automation. Éste es mi feedback sobre tu trabajo;

  • Tu trabajo tiene una muy buena cobertura, dividiste las funcionalidades y eso hace que sea facil saber que se intenta probar.
  • Esta vez si incorporastes los archivos mínimos solicitados en automation, como la edición del CI, test, y el archivo MarkDown.
  • Tu código es limpio, ordenado y claro. Respetas la nomenclatura en cierta forma. No es necesario colocar el "GX3-" solo el número, igual a los TC del TS en jira.

Sugerencia de mejora:

  • Comienza incorporar otras herramientas como fixture cuando necesites usar datos estáticos, POM para manejar de forma eficiente los selectores y tener un test funcional, mantenible y escalable.
  • Luego puedes utilizar variables con datos random, haciendo uso de la función random de JS o Cypress.

Vas muy bien. No te olvides de completar el flujo en JIRA!!

@apscecilia apscecilia added Approved Approved and pending merge. and removed In Review Currently being evaluated by the reviewer. labels Dec 21, 2023
Copy link
Contributor

@ZwitterKaneda ZwitterKaneda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hola @arjusgit el test en general esta muy bien, lo que me llama la atención es que tienes el mismo error que en los anteriores.
image
Deberás de realizar practicas de las aserciones a realizar. En tu caso de prueba validas 1 sola letra en un SUT donde la funcionalidad es otra 😅

En tu próxima US busquemos validar una funcionalidad. ¿Lo vemos en la picada? Buen trabajo 👏

@ZwitterKaneda ZwitterKaneda merged commit df786ed into QA Dec 26, 2023
@ZwitterKaneda ZwitterKaneda deleted the GX3-845-⚡️-tools-qa-widgets-auto-complete branch January 9, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Approved and pending merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants