Skip to content

Gx 41632 ✅ tools qa widgets dropdown select menu #524

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 4 commits into from
Nov 11, 2023

Conversation

DaryelinRossell
Copy link
Contributor

Para el desarrollo de esta US, implemente POM y Fixture.
Se analizó cada select de forma separada por lo que se ovtuvieron 5 casos de prueba. En cada uno se realizo su respectiva validacion.

@DaryelinRossell DaryelinRossell added Needs Review Ready for review by the team. Tutor Assigned I have a tutor assigned labels Oct 31, 2023
@DaryelinRossell DaryelinRossell self-assigned this Oct 31, 2023
@saiotest
Copy link
Contributor

Revisando!

@ZwitterKaneda ZwitterKaneda added Approved Approved and pending merge. and removed Needs Review Ready for review by the team. labels Nov 8, 2023
Copy link
Contributor

@saiotest saiotest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Buen código, se puede mejorar a ser más escalable y flexible pero buenas validaciones!

const randomSelectValue = faker.datatype.number({ min: 1, max: 4 });
const randomValue = faker.datatype.number({ min: 1, max: 2 });
selectMenuPage.clickSelectValue();
if (randomSelectValue === 1) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me gusta como validación exhaustiva pero sería bueno ver cómo seleccionarías una opció sin especificar que sea del grupo y opción, es decir no tener un selector por cada opción del dropdown.

Qué pasaría si hubieran 245 opciones del Dropdown? harías una para cada una? La idea es crear como una partición, una función que solo seleccione una opción del que pases por parámetro. Esto sería el mejor enfoque para una selección de Dropdown... SIN EMBARGO, considero esto que hiciste válido ya que buscaste validar cada opción como creo que indica la US.

selectMenuPage.clickOldSelectMenu(randomOldSelectMenu);
selectMenuPage.get.selectOldMenu().should('contain', data.oldStyleMenu[randomOldSelectMenu]);
});
it('41632 | TC4: validar la funcionalidad del Multiselect dropdown', () => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me gusta que el código sea iterable, pero creo que pudimos mejorarlo de alguna manera.

@@ -0,0 +1,26 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me gusta esta data! la estructura

@@ -0,0 +1,90 @@
class SelectMenu {
get = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Esto es lo que te decía de tener un montón de selectores con diferentes opciones de un Dropdown.

Yo aprobaría todo este hermoso código y esfuerzo, pero buscaría mejorarla escalabilidad del código, en tener que usar métodos que puedan ser más flexibles. Pero podemos verlo en llamada! recuerda que puedo continuar contigo como Tutor por el siguiente Sprint, y ahora que sos Mod, puedes tener más de un tutor si quieres jaja (no tan literal)

@saiotest saiotest merged commit 6945496 into QA Nov 11, 2023
@AlexanderWindecker AlexanderWindecker deleted the GX-41632-✅-tools-qa-widgets-dropdown-select-menu 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. Tutor Assigned I have a tutor assigned
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants