Skip to content

GX-19307 | ✅ | Tools QA | Forms | Practice Form #335

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
Jun 25, 2023

Conversation

romanfenoy
Copy link

Se realizaron 18 Test los cuales todos pasaron correctamente.

image

Se testeo el formulario "Student Registration Form", al realizar los test se tardó más de lo esperado porque tuve varios inconvenientes los cuales fueron

  • Primera vez que trabaje con POM, faker y date picker
  • Las const y funciones que usaba en el POM al pasarlas a los test no era fácilmente legibles por lo cual tuve que rehacerlos
  • Fallos en las funciones del POM
  • Múltiples fallos por assetions que no funcionaban, ya sea por mal selector o no encontrar forma de que la función del POM te devuelva los valores que buscabas

Hubo una assertion que fue skipeada por falta de conocimiento, la cual fue la assertion del datepicker en el TC07 que pedía la fecha actual.

…o el POM y hay hecho un test de momento para verificar los valores
…ixture, el POM fue modificado para coincidir con los test y el fixture. Se agregaron los test, en total 18, para el formulario de estudiantes
…n los test, fixture y POM para que se puedan leer mejor
…n los elementos en string asi se puede hacer la asercion
…os test, POM y Fixture, cambiando los const del POM al test para poder hacer las assertion y sea mas sencillo de leer cada uno de los test y el fixture se elimino data estatica que no se usaba
@romanfenoy romanfenoy requested a review from ZwitterKaneda June 20, 2023 21:13
@romanfenoy romanfenoy self-assigned this Jun 20, 2023
@PaolaRod PaolaRod added the In Review Currently being evaluated by the reviewer. label Jun 21, 2023
@PaolaRod
Copy link

PaolaRod commented Jun 22, 2023

Hola Roman

Estuve revisando tu test

  • En general, lo veo bastante bien, cumpliendo con las condiciones basicas que siempre mencionamos.
  • Queria destacar el uso de faker para todas las entradas, y que no validaste por css los bordes rojos en los casos negativos (ya que siempre presenta problemas al validar por el color) Muy bien ahi.
  • En cuanto a redaccion de los TC te recomendaria mejorarla por ejemplo en vez de Validar usuario no ingresa “First Name” en el formulario puede ser : Validar no poder registrarse dejando campos vacios en el formulario.
  • En cuanto a la assertion que no pudo ser validada, hay una funciones ya establecidas en javascript para obtener la fecha actual, una de ellas se llama new Date( ) , pero te dejo un articulo mas amplio para escoger cual es la que mas te conviene: https://www.freecodecamp.org/espanol/news/javascript-date-now-como-obtener-la-fecha-actual-con-javascript/
  • En cuanto a los test debido al comportamiento que tiene el formulario, que cada input informa el error, puedes englobar los casos negativos por ejemplo los que involucran diferentes inputs en un solo test case, por ejemplo:
  1. Validar usuario no ingresa “First Name” en el formulario
  2. Validar usuario no ingresa “Last Name” en el formulario
  3. Validar usuario no ingresa “Gender” en el formulario

Puede ser un solo TC: Validar no poder registrarse ingresando campos FirsName, LastName, Gender vacios, esto para poder minimizar la cantidad de TC y que no sea tan repetitivo.

  • En cuanto a las asserciones tenemos la premisa de no dejar aserciones en el POM, por lo que mi recomendacion es no tener esas funciones en el POM, y tratar de hacerlas directamente en el test.

Eso seria todo, cualquier cosa avisame, en general buen trabajo y espero las correcciones para ser aceptado. Saludos

@PaolaRod PaolaRod added Suggested Improvement Suggests improvements or enhancements made in the pull request. Code Review Feedback Specific feedback provided by the reviewer. and removed In Review Currently being evaluated by the reviewer. labels Jun 22, 2023
@ZwitterKaneda
Copy link
Contributor

Para avanzar, requiere hacer fetch de QA

…fueron: Combinar ciertos test asi se reduce las cantidades, sacar las asserciones del POM, se hizo la assertion de la fecha actual que faltaba y cambios de algunos nombres de los TCs
@PaolaRod
Copy link

Excelente trabajo, estuviste atento a todas las observaciones hechas. Continua asi! Vamos por mas!

@PaolaRod PaolaRod added Approved Approved and pending merge. and removed Suggested Improvement Suggests improvements or enhancements made in the pull request. Code Review Feedback Specific feedback provided by the reviewer. labels Jun 25, 2023
@PaolaRod PaolaRod merged commit 5e0feb0 into QA Jun 25, 2023
@AlexanderWindecker AlexanderWindecker deleted the GX-19307-✅-tools-qa-forms-practice-form branch August 26, 2024 23:13
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