Skip to content

Gx2 8536 � tools qa interactions selectable #523

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
Oct 31, 2023

Conversation

Eritolosa
Copy link
Contributor

En este pull request se implementa la funcionalidad de Interactions - Selectable en el proyecto.

Se crearon tres casos de prueba donde validamos la funcionalidad de los elementos indicados en la historia de usuario.

El patrón de diseño de POM se utilizó para organizar el código de los casos de prueba de manera que sea fácil de entender y mantener.

Los fixtures se utilizaron para facilitar la ejecución de los casos de prueba, proporcionando datos de prueba predefinidos.

Creamos el pull request

@Eritolosa Eritolosa requested a review from saiotest October 30, 2023 22:33
@Eritolosa Eritolosa self-assigned this Oct 30, 2023
@Eritolosa Eritolosa added the Needs Review Ready for review by the team. label Oct 30, 2023
@saiotest
Copy link
Contributor

Revisando!

@saiotest saiotest added the Tutor Assigned I have a tutor assigned label Oct 31, 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.

TUTTO BENE

selectablePage.defaultColorList();
selectablePage.get.grid().should('have.attr', data.attribute, data.valueFalse);
});
it('GX2-8537 | TC2 Validar poder seleccionar y des-seleccionar los elementos de la pestaña “List“', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Erika, me encanta cómo vas! solo que los métodos usados (la nomenclatura) no se entiende bien qué pasos hiciste ahí con los métodos del POM, la idea es que se pueda super leer para que cualquier QA o Dev lo entienda jaja Los pasos del Test a través de los nombres de los métodos.

@@ -0,0 +1,9 @@
{
"List": ["Cras justo odio", "Dapibus ac facilisis in", "Morbi leo risus", "Porta ac consectetur ac"],
Copy link
Contributor

Choose a reason for hiding this comment

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

El nombre de "List" no es muy descriptivo que digamos, pero entiendo para qué se usará. Podemos mejorar más la nomenclatura general 🗡️

Copy link
Contributor

Choose a reason for hiding this comment

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

Puedes intentar usar "Lista selectable" es decir: "selectableList"

Copy link
Contributor

Choose a reason for hiding this comment

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

Muy ingenioso de usar Fixtures en esta US!!

list: () => cy.get('#demo-tab-list'),
grid: () => cy.get('#demo-tab-grid'),
};
clickList() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Cuando te refieres a "clickList" te refieres a abrir la Lista? seleccionar la Lista?
Lo bonito de ser un Automation, es que a pesar de automatizar, también debemos ser como unos analistas escribiendo código entendible <3 mientras más descriptiva eres, más enamorarás a los recruiters y tus reviewers del futuro! jaja

Copy link
Contributor

Choose a reason for hiding this comment

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

Te daré un super consejo con los métodos del POM: La nomenclatura es clave, la forma como nombramos a los métodos parece mentira pero ayuda a desarrollar más rápido sabiendo lo que quieres y hace tus métodos si pasan unos meses sin verlos jaja. En este caso, es el uso de los "Getters" y "Settters" son dos formas de llamar a los métodos.

Hay métodos que se usan para obtener una data o generar una data, suelen colocarse "get" al principio, y seguido de lo que vas a obtener!.

@saiotest saiotest merged commit 0de9a0c into QA Oct 31, 2023
@saiotest saiotest added Approved Approved and pending merge. and removed Needs Review Ready for review by the team. labels Oct 31, 2023
@ZwitterKaneda ZwitterKaneda deleted the GX2-8536-�-tools-qa-interactions-selectable 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. Tutor Assigned I have a tutor assigned
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants