Skip to content

GX-22811 Test(DynamicsProps): primera cobertura sobre elementos dinamicos #365

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

Conversation

vertonepa
Copy link

Se realizaron las coberturas de la US https://upexgalaxy18.atlassian.net/browse/GX-22811
Usé por primera vez los comandos clock() y tick() para probar que la interacción con los elementos dinámicos sea diferente en distintos rangos de tiempo.
Se dieron complicaciones al probar el elemento con ID dinámico.

@vertonepa vertonepa added the Needs Review Ready for review by the team. label Jul 2, 2023
@vertonepa vertonepa requested a review from ZwitterKaneda July 2, 2023 12:17
@vertonepa vertonepa self-assigned this Jul 2, 2023
@ZwitterKaneda ZwitterKaneda added Requirements Not Met Does not meet specified requirements. and removed Needs Review Ready for review by the team. labels Jul 2, 2023
@ZwitterKaneda
Copy link
Contributor

Hola @vertop390 cómo estas?, revisaré tu PR hoy.
La cobertura de tu test no es correcta tienes 4 items por validar, necesitas minimamente 4 casos de prueba

I want to test the Dynamic Properties WHEN:

Element has Dynamic ID in DOM

Element is enable after 5 sec

Element is visible after 5 sec

Element CSS color is changed

Algo que tienes que tener en cuenta es que la US tiene como objetivo a que aprendas a usar el {timeout}. La opción {timeout} se utiliza para especificar un tiempo de espera personalizado para un comando en particular. Puedes proporcionar el tiempo de espera en milisegundos como un número entero o una cadena que representa un período de tiempo, como "2000" para 2 segundos o "30s" para 30 segundos. Aquí te dejaré información sobre él

@vertonepa vertonepa reopened this Jul 3, 2023
@vertonepa
Copy link
Author

Actualización:

  • Se expandió la cobertura de pruebas, se pasó de 2 TC a 4 TC
  • Se cambiaron detalles de la validación sobre el cambio de estado de los elementos

@ZwitterKaneda ZwitterKaneda added Approved Approved and pending merge. and removed Requirements Not Met Does not meet specified requirements. labels Jul 3, 2023
@ZwitterKaneda
Copy link
Contributor

Excelente @vertop390 buen trabajo! continuemos avanzando!

@ZwitterKaneda ZwitterKaneda merged commit de619ae into QA Jul 3, 2023
@ZwitterKaneda ZwitterKaneda deleted the GX-22811-✅-tools-qa-elements-dynamic-properties 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.

2 participants