Skip to content

Gx 23133 ✅ tools qa elements broken links images #375

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 5 commits into from
Jul 6, 2023

Conversation

vertonepa
Copy link

Se validaron la carga de imágenes y redirección a páginas por medio de links.
No se encontraron defectos, todas las pruebas pasaron.
El desarrollo de los casos de prueba tardó más de lo estimado por bloqueo al momento de validar la carga de imágenes.
Se ponen a disposición de revisión todos los cambios guardados.

@vertonepa vertonepa added the Needs Review Ready for review by the team. label Jul 6, 2023
@vertonepa vertonepa requested a review from mjorabdo July 6, 2023 09:01
@vertonepa vertonepa self-assigned this Jul 6, 2023
@mjorabdo mjorabdo added the In Review Currently being evaluated by the reviewer. label Jul 6, 2023
@mjorabdo
Copy link

mjorabdo commented Jul 6, 2023

Hola @vertop390 muy lindo trabajo! Te dejo unos comentaarios:

  • En el CI Suite debes modificar el TX ya que pusiste mal el numero.
  • Esta faltando validar el heigth (solamente validaste el weigth de las imagenes)
  • en el Describe te quedo el titulo de la US anterior al igual que en el BeforeEach

Me gustaria que empieces a implementar el uso del POM para tus siguientes US.
Muy buen trabajo! Segui asi ! y aguardo por esos cambios !!

@mjorabdo mjorabdo closed this Jul 6, 2023
@mjorabdo mjorabdo reopened this Jul 6, 2023
@mjorabdo mjorabdo merged commit e7282b4 into QA Jul 6, 2023
@AlexanderWindecker AlexanderWindecker deleted the GX-23133-✅-tools-qa-elements-broken-links-images branch August 26, 2024 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In Review Currently being evaluated by the reviewer. Needs Review Ready for review by the team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants