Gx 23427 ✅ tools qa forms practice form #410

vertonepa · 2023-07-23T15:43:14Z

Se desarrolló la cobertura completa del formulario de estudiantes. Se realizaron 6 TC's de los cuales 5 (cinco) pasaron la prueba y 1 (uno) falló.

Para esta cobertura se desarrollaron un POM y un archivo JSON con datos de ingreso para el fixture.
El archivo "fixtures\data\GX-23427-PracticeForm.json" se estructura:

Datos inválidos de email;
Datos de archivos;
Datos para validar bordes;
Datos de número telefónico

AVISO: El TC02 falló por defecto encontrado por lo que fue salteada su ejecución. En breve será reportado.

…e Fixture

…oped

…rst TC was created

…enero en el popup

…completo

LauraMont

Hola Pablo! Te comento algunos puntos que nos interesan para esta review:

US en Jira: Te quedo pendiente además de reportar el bug ,cerrar el TX y cambiar el estado a "Test Failed" en la US así se indica que hay un bug.
Cobertura: Muy bien la cobertura, aunque la US te pide solo el HP no esta de mas realiza por los casos negativos a fines didácticos
Código: el archivo GX-23133-BrokenLinks.cy.js tiene cambios que no son necesarios para esta US. Te dejo revertir estos cambios para corregir. Tambien en este otro archivo GX-22038-TextBox.Page.js realizaste cambios pero son de otra US. Puede ser algo de que se mergearon cambios a QA ,por lo que antes de modificar esto que te pido realizar un fetch y merge a QA. Si no se han realizado los cambios puedes revertir estos. Los archivos correspondientes a tu ejecución te los comento mas abajo en las líneas que nos interesen revisar.
Otros: Bien editado el CI. El archivo del markdown (.md) debe estar en el sprint donde se diseño en este caso el sprint-21

Quedo al pendiente de las correcciones, un saludo!

LauraMont · 2023-07-23T20:45:46Z

cypress/fixtures/data/GX-23427-PracticeForm.json

+    "email": {
+        "invalid": {
+            "noAtSign": "valid_emailgmail.com", 
+            "noMinimChar": {


Te recuerdo por si acaso el formato de un mail: <user>@<org>.<tipo/extension>
Teniendo en cuenta lo anterior puedes llamar a tus variables :

noUser

noOrg

noExt o noTipo

LauraMont · 2023-07-23T21:10:16Z

cypress/support/pages/Forms/GX-23427-PracticeForm.page.js

+	}
+
+	//seleccionar un elemento de un dropdown list dinámico
+	subjectsSelector(matching) {


Muy buena la manera de seleccionar las opciones de este elemento ✨

LauraMont · 2023-07-23T21:38:32Z

cypress/e2e/Tests/Suites/GX-23427-PracticeForm.cy.js

+		form.get.address().should('have.css', 'border-color', field.greenBorder);
+	});
+
+	it.skip('23428 | TC02: Validar NO enviar formulario al ingresar datos inválidos', () => {


Por aquí falto la validación de subject, entiendo que en el SUT no pasa que al fallar pero los criterios indican que este campo no debe estar vacío

No, es lo que estaba por reportar de este bug antes del mantenimiento. El error me lo trae el input Mobile, este campo no puede tener menos de 10 dígitos y en la página de DemoQA me lo marca así, pero al ejecutar el test en Cypress esta prueba no pasa porque el campo me lo toma como correcto si la cantidad de dígitos está entre 1 y 9.

Perfecto ,me parece correcto que sea bug. Lo que te comentaba era por que no estas validando que el campo subject este en rojo ya que en los criterios de aceptacion indica que es un campo que no puede ir vacio,siguiendo esa logica su comportamiento deberia ser igual a los otros campos requeridos

Ahora que lo revisé dice que ese campo no puede estar vacío aunque no menciona que se torne rojo al dejarlo así, también es distinto el comportamiento porque no se muestra de ningún color después de enviar datos. Y ahora me doy cuenta que el hecho de que se puedan enviar datos aunque este campo esté vacío también es un bug porque se especifica que no se despliega la info con los datos enviados en este caso.

LauraMont · 2023-07-23T21:46:51Z

Cuando tengas los cambios corregidos en su totalidad no olvides agregar el tag "Needs Review". Además puedes quitar el tag "Code Review Feedback" una vez leas estos mensajes este es mas que nada indicativo que dejamos un mensaje por el hilo

…es, reubicación del md a sprint21

LauraMont · 2023-07-26T00:10:00Z

Hola Pablo! Espero que estés bien
No estoy viendo los cambios en los archivos, que tal te fue con el pull a QA? Te actualizo algo o no modifico nada?
Voy viendo el bug, te comento en el mismo cualquier cosa.
Un saludo!

ZwitterKaneda · 2023-07-30T13:59:12Z

Hola @vertop390 estoy cerrando los PR de la semana, veo que el tuyo tiene un conflicto el cual resolver y tareas pendientes.

el conflicto del CI-Suite.yml siempre estará, no le des importancia, el de GX-23133-BrokenLinks.cy.js debes de corregirlo tu. Ten en cuenta que si es de otra historia de usuario no deberías de modificarlo, sino que importarlo a tu POM creado

vertonepa · 2023-07-31T14:17:23Z

restaurado el archivo GX-23133

vertonepa · 2023-07-31T14:20:17Z

cypress/support/pages/Elements/GX-22038-TextBox.Page.js

Se cambiaron las comillas dobles a simples por configuración del repositorio

ZwitterKaneda · 2023-08-02T13:19:49Z

Buen trabajo @vertop390 👏 continua avanzando así! 🚀

vertonepa added 7 commits July 10, 2023 18:21

GX-23427 Test(PracticeForm): fixture in progress

8a6e624

GX-23427 Test(PracticeForm): POM developed and some corrections in th…

b0597e3

…e Fixture

GX-23427 Test(PracticeForm): POM was modified and TCs are being devel…

00d8f06

…oped

GX-23427 Test(PracticeForm): POM and Fixture were modified and the fi…

f2ac90d

…rst TC was created

GX-23427 Test(PracticeForm): intentando validar valor de checkbox y g…

dd07f57

…enero en el popup

GX-23427 Test(PracticeForm): fallas de diseño de pom corregidas, TC1 …

d13b6f0

…completo

GX-23427 Test(PracticeForm): Todos los tc fueron desarrollados

d7398f5

vertonepa added the Needs Review Ready for review by the team. label Jul 23, 2023

vertonepa requested a review from ZwitterKaneda July 23, 2023 15:43

vertonepa self-assigned this Jul 23, 2023

LauraMont requested review from LauraMont and removed request for ZwitterKaneda July 23, 2023 19:45

LauraMont added In Review Currently being evaluated by the reviewer. and removed Needs Review Ready for review by the team. labels Jul 23, 2023

LauraMont reviewed Jul 23, 2023

View reviewed changes

LauraMont added Requirements Not Met Does not meet specified requirements. Code Review Feedback Specific feedback provided by the reviewer. and removed In Review Currently being evaluated by the reviewer. labels Jul 23, 2023

vertonepa added 2 commits July 23, 2023 22:36

GX-23427 Test(PracticeForm): cambios en funciones del POM y agarrador…

c0ecf5b

…es, reubicación del md a sprint21

GX-23427 Test(PracticeForm): .

5a30ee1

vertonepa added Needs Review Ready for review by the team. and removed Code Review Feedback Specific feedback provided by the reviewer. Requirements Not Met Does not meet specified requirements. labels Jul 24, 2023

LauraMont added In Review Currently being evaluated by the reviewer. and removed Needs Review Ready for review by the team. labels Jul 25, 2023

LauraMont added Minor Tweaks Needed Requires minor adjustments before merging. Code Review Feedback Specific feedback provided by the reviewer. and removed In Review Currently being evaluated by the reviewer. labels Jul 26, 2023

ZwitterKaneda closed this Jul 30, 2023

ZwitterKaneda added Requirements Not Met Does not meet specified requirements. and removed Minor Tweaks Needed Requires minor adjustments before merging. Code Review Feedback Specific feedback provided by the reviewer. labels Jul 30, 2023

GX-23427 Restaurando archivo 23133

3b972f6

vertonepa reopened this Jul 31, 2023

vertonepa commented Jul 31, 2023

View reviewed changes

Merge branch 'QA' into GX-23427-✅-tools-qa-forms-practice-form

c09580e

ZwitterKaneda added Approved Approved and pending merge. and removed Requirements Not Met Does not meet specified requirements. labels Aug 2, 2023

ZwitterKaneda merged commit 01a7dd4 into QA Aug 2, 2023

ZwitterKaneda unassigned vertonepa Sep 17, 2023

AlexanderWindecker deleted the GX-23427-✅-tools-qa-forms-practice-form branch August 26, 2024 23:31

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Gx 23427 ✅ tools qa forms practice form #410

Gx 23427 ✅ tools qa forms practice form #410

vertonepa commented Jul 23, 2023

LauraMont left a comment •

edited

Loading

LauraMont Jul 23, 2023

LauraMont Jul 23, 2023

LauraMont Jul 23, 2023 •

edited

Loading

vertonepa Jul 23, 2023 •

edited

Loading

LauraMont Jul 25, 2023

vertonepa Jul 26, 2023

LauraMont commented Jul 23, 2023 •

edited

Loading

LauraMont commented Jul 26, 2023

ZwitterKaneda commented Jul 30, 2023

vertonepa commented Jul 31, 2023

vertonepa Jul 31, 2023

ZwitterKaneda commented Aug 2, 2023

Gx 23427 ✅ tools qa forms practice form #410

Gx 23427 ✅ tools qa forms practice form #410

Conversation

vertonepa commented Jul 23, 2023

LauraMont left a comment • edited Loading

Choose a reason for hiding this comment

LauraMont Jul 23, 2023

Choose a reason for hiding this comment

LauraMont Jul 23, 2023

Choose a reason for hiding this comment

LauraMont Jul 23, 2023 • edited Loading

Choose a reason for hiding this comment

vertonepa Jul 23, 2023 • edited Loading

Choose a reason for hiding this comment

LauraMont Jul 25, 2023

Choose a reason for hiding this comment

vertonepa Jul 26, 2023

Choose a reason for hiding this comment

LauraMont commented Jul 23, 2023 • edited Loading

LauraMont commented Jul 26, 2023

ZwitterKaneda commented Jul 30, 2023

vertonepa commented Jul 31, 2023

vertonepa Jul 31, 2023

Choose a reason for hiding this comment

ZwitterKaneda commented Aug 2, 2023

LauraMont left a comment •

edited

Loading

LauraMont Jul 23, 2023 •

edited

Loading

vertonepa Jul 23, 2023 •

edited

Loading

LauraMont commented Jul 23, 2023 •

edited

Loading