Skip to content

Gx 35657 tools qa forms practice form #488

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 7 commits into from
Oct 13, 2023

Conversation

rafo18
Copy link

@rafo18 rafo18 commented Sep 29, 2023

En esta US se realizo:

  • Se realizaron 3TC

TC1: Se valido poder enviar formulario con datos validos
TC2: Se valido no enviar formulario con inputs vacios y se verifico que todos los campos obligtorios marquen un error al estar vacios
TC3: Se valido no enviar formulario con datos no validos en los inputs de Mobile Number y email

  • Se creo un POM para el formulario donde se realizo un solo metodo que me permita mandar datos vacios si no ingreso ningun valor como argumento.

  • El uso de datos validos se realizo mediante la libreria faker

  • El uso de datos no validos de email y mobile number se realizo mediante un Fixture

@rafo18 rafo18 added the Needs Review Ready for review by the team. label Sep 29, 2023
@rafo18 rafo18 requested a review from apscecilia September 29, 2023 01:57
@rafo18 rafo18 self-assigned this Sep 29, 2023
@rafo18 rafo18 requested review from ZwitterKaneda and apscecilia and removed request for apscecilia and ZwitterKaneda September 29, 2023 22:30
@apscecilia apscecilia added the In Review Currently being evaluated by the reviewer. label Sep 30, 2023
@apscecilia
Copy link
Contributor

Estimado Rafael, me complace nuevamente revisar tu test. Haz tenido un avance progresivo y constante!!. Este es mi feedback:

  • Utilizas la biblioteca Faker.js para generar datos aleatorios, lo cual es útil para probar escenarios diversos y extremos.
  • Mantienes buena estructura de trabajo (commit, MD, suite.ci, test,fixture, POM)
  • Tu redacción y analisis de tus TC son cada vez más claro y bien estructurado

SUGERENCIAS:

  • Mantén tu código limpio de comentarios innecesarios (ej linea 177) y uso excesivo de cy.log, esto puede dificultar la lectura y comprensión del código. Se puede usar si la información del resultado es significativo. También puedes usarlo para depurar problemas, cuando sea necesario, pero luego se quita
  • Crear un solo método en el POM para llenar el formulario puede ser una buena estrategia si deseas simplificar tu código y reutilizarlo en varias pruebas. Sin embargo, también es importante tener en cuenta que un método que realiza muchas acciones diferentes puede volverse difícil de mantener y entender a medida que el formulario se complica.
    Una estrategia intermedia podría ser crear métodos separados en el POM para diferentes tipos de acciones en el formulario. Por ejemplo:
  • Un método para completar información personal (nombre, correo electrónico, género).
  • Un método para completar información de contacto (número de teléfono, dirección actual).
  • Métodos para seleccionar materias, pasatiempos y cargar imágenes por separado.
    Esto brinda un equilibrio entre la reutilización y la facilidad de mantenimiento, ya que puedes invocar estos métodos según sea necesario en tus test.

De todas formas la elección de cómo estructurar tu POM dependerá de la complejidad del formulario y tus preferencias personales. Lo más importante es mantener el código limpio, organizado y fácil de entender para que puedas mantener y ampliar tus TC con facilidad. Ya estoy hilando mas fino porque todo lo demás ya lo estas manejando super bien. Excelente trabajo!!

@apscecilia apscecilia added Approved Approved and pending merge. and removed Needs Review Ready for review by the team. In Review Currently being evaluated by the reviewer. labels Sep 30, 2023
@ZwitterKaneda
Copy link
Contributor

Hola @rafo18 espero que las correcciones que te dimos en las picadas de código sean de ayuda. Si quieres integrar los cambios sugeridos, puedes hacerlo acá mismo. Deberás de abrir nuevamente el mismo PR

@ZwitterKaneda ZwitterKaneda merged commit 6248e4a into QA Oct 13, 2023
@AlexanderWindecker AlexanderWindecker deleted the GX-35657--tools-qa-forms-practice-form 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants