Skip to content

Gx2 3865 ✅ tools qa interactions sortable #366

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

Conversation

dbquiroga
Copy link

Se realizaron pruebas de listas y grillas ordenadas. Se modifico de manera aleatoria su orden.

  • Repo Actualizado con QA
  • Todas las pruebas pasaron

El metodo que hace la magia es triggerItem el cual mueve de manera aleatoria los items. Genera un random del array de Items, saca la posicion de "comienzo" y la posicion de "final" y luego se hacen las aserciones de ello en el test.

image

Lo nuevo que aprendí:

  • A usar .trigger() nunca lo había usado.
  • Usar async y await para manejar operaciones sincronizas (me pareció mejor la sintaxis que Cypress.Promise)

@dbquiroga dbquiroga requested a review from mjorabdo July 2, 2023 13:35
@dbquiroga dbquiroga self-assigned this Jul 2, 2023
@ZwitterKaneda ZwitterKaneda requested review from ZwitterKaneda and removed request for mjorabdo July 2, 2023 17:50
@ZwitterKaneda ZwitterKaneda added the In Review Currently being evaluated by the reviewer. label Jul 2, 2023
@ZwitterKaneda
Copy link
Contributor

Hola @dbquiroga cómo estas? Revisaré tu código hoy.
¿Qué sucedió aquí? 🤣 hay dos commits que no son claros y sin nomenclatura 🩴
image
Para que tengas en cuenta en un próximo test, código que no utilizas, codígo que no agregas en tu test (pom, fixture, etc) por ejemplo:
image
los hooks si no los usas no deberían de ir, y los comentarios, solo informativos de funcionamiento. Se entiende que el it() contiene el test e inicia allí 😅
Con respecto al test, me gustó como lo has trabajado, el uso del trigger en el POM y la devolución de los datos necesarios para la aserción haciendolo asincrono para evitar bloquear el código con el return me gustó 😉 like it
image

Buen trabajo 👏

@ZwitterKaneda ZwitterKaneda merged commit aa4ca29 into QA Jul 2, 2023
@ZwitterKaneda ZwitterKaneda added Approved Approved and pending merge. and removed In Review Currently being evaluated by the reviewer. labels Jul 2, 2023
@ZwitterKaneda ZwitterKaneda deleted the GX2-3865-✅-ToolsQA-Interactions-Sortable 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