Skip to content

Gx 22486 ✅ tools qa forms practice form #370

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
Jul 8, 2023

Conversation

gualex89
Copy link

@gualex89 gualex89 commented Jul 4, 2023

En la presente US, se utilizó la librería faker para el llenado de los campos de typeo, y los campos de selección se hizo de manera random. También se utilizo un fixture para los campos inválidos. Todos las pruebas pasaron a pesar de que hay un problema en la página en la que el campo phone number reacciona de maneras distintas con campos inválidos cuando lo pruebas de manera manual y cuando lo pruebas de manera automática.

@gualex89 gualex89 added the Code Review Feedback Specific feedback provided by the reviewer. label Jul 4, 2023
@gualex89 gualex89 requested a review from PaolaRod July 4, 2023 13:11
@gualex89 gualex89 self-assigned this Jul 4, 2023
@ZwitterKaneda
Copy link
Contributor

Hola @gualex89 tienes un conflicto en el PR que debes de reparar.
image
Has realizado alguna modificación en otro test y lo has enviado 😅 esto deberás de restaurarlo ^^

Comentame, para que es este launch.json que has creado? Cuál es su función?
image

La imagen que deberás de utilizar para realizar subidas, se encuentra en el repositorio en la carpeta fixture
image

Como buena practica, como siempre menciono, es realizar los commits por cada avance y con la nomenclatura de la US. Checa que en el único commit donde usaste la nomenclatura quedó bien realizada la trazabilidad de este. Para tus próximos commit ten eso en cuenta, siempre será git commit -m "GX-1234 <mensaje>" de hecho en el mismo Jira te lo indica
image

Con respecto al test, me gustó mucho este enredo que te hiciste, me da gusto ver como buscaste una solución y te salió. Bien por eso!
image
Lograste lo que buscabas 👏 me hizo gracia porque la selección del gender aleatorio puede ser más sensillo, te lo comentaré cuando revise todo el código. Me tomaré el tiempo luego de descargarlo y revisarlo. Ya que tienes muchos cambios solicitados por hacer, aguardaré por ellos.

@ZwitterKaneda ZwitterKaneda added Requirements Not Met Does not meet specified requirements. Suggested Improvement Suggests improvements or enhancements made in the pull request. and removed Code Review Feedback Specific feedback provided by the reviewer. labels Jul 5, 2023
@gualex89
Copy link
Author

gualex89 commented Jul 5, 2023

Listo @ZwitterKaneda creo que deje todo como me pediste.

@gualex89 gualex89 added Code Review Feedback Specific feedback provided by the reviewer. and removed Requirements Not Met Does not meet specified requirements. Suggested Improvement Suggests improvements or enhancements made in the pull request. labels Jul 5, 2023
@gualex89 gualex89 requested review from ZwitterKaneda and removed request for PaolaRod July 5, 2023 20:34
@ZwitterKaneda ZwitterKaneda added In Review Currently being evaluated by the reviewer. and removed Code Review Feedback Specific feedback provided by the reviewer. labels Jul 7, 2023
@ZwitterKaneda
Copy link
Contributor

Hola @gualex89 continuaré con la revisión!:
Primero tengamos en cuenta que el CI te esta fallando, te han fallado 5 test, debes de revisar ese inconveniente.
image
Verás que aquí te marco varias cositas antes de continuar avanzando en la revisación. El challenge de la user story es hacer el calendario aleatorio, tanto año, como mes, como el día. Este metodo que creaste selecciona uno de los 40 y pico días que se ven en el calendario, pero no selecciona año ni mes aleatorio. Deberás de re hacer este método ;) es el que tiene todo el valor del test! ánimos!.
image
Aquí veo que tienes elementos fuera de tu POM, esto debería estar en él y algo que esta mal usado son los contains()
este se utiliza para atrapar texto plano en el DOM, te dejo aquí la documentación de esta sintaxis. Las BDD assertions que usamos en nuestros test son 2 .should() y expect() podrás ver más información es su documentación
Deberas de reparar esa parte del test para finalizar con las aserciones.
image
En el TC03 fallas en el scope de este, intentas validar que no funcione sin el "@" pero no escribes en el input, deberás de acomodar tu variable para que ingrese un mail sin "@"
Por cierto, buen uso del Faker ¿Lo entiendes? ¿Alguna duda al respecto de él?
Revisando los otros test cases, en ninguno de ellos escribes el mail 👀 ¿Por qué? Son todos con objetivo en el email y este no esta escrito 🤔 toca reparar eso.
Olvidaba comentar, veo que los cambios anteriormente solicitados los hiciste todos, bien con eso.

Te serán de utilidad los formaters que tenemos configurados en las recomendaciones de las extenciones para vscode. Te recomiendo instalarlas =)

Recuerda también hacer fetch ya que se integrarán cambios mientras aguardamos la reparación de tus test. Éxito!

P/D: Puedes escribirme cualquier consulta, no es molestia, si me demoro mucho en responder, en consultas o cualquier mod falcón puede darte una mano

@ZwitterKaneda ZwitterKaneda added Requirements Not Met Does not meet specified requirements. Suggested Improvement Suggests improvements or enhancements made in the pull request. and removed In Review Currently being evaluated by the reviewer. labels Jul 7, 2023
@ZwitterKaneda ZwitterKaneda merged commit 9a7f7a5 into QA Jul 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requirements Not Met Does not meet specified requirements. Suggested Improvement Suggests improvements or enhancements made in the pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants