Skip to content
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

Execute tests #310

Merged
merged 20 commits into from
Sep 30, 2024
Merged

Execute tests #310

merged 20 commits into from
Sep 30, 2024

Conversation

dlopezalvas
Copy link
Contributor

@dlopezalvas dlopezalvas commented Sep 27, 2024

Resolves #251

Con esto culminaría el arco del botón ejecutar (creo). Se agregó Cypress para tests de integracion, esto porque necesitamos testear lo que sucede dentro el iframe y para eso es necesario correrlos en un navegador.

Están skipeados los tests referidos a los bloques porque las soluciones las tomé de ember y por lo mencionado en el issue #312 no corren. Lo ideal sería dejarlos asi, para que cuando esté resuelto este otro issue se puedan sacar el skip.

Para correr los tests de cypress se puede correr: npm run test:cy

@dlopezalvas dlopezalvas marked this pull request as ready for review September 27, 2024 18:16
@dlopezalvas dlopezalvas requested a review from a team as a code owner September 27, 2024 18:16
Copy link
Contributor

@danielferro69 danielferro69 left a comment

Choose a reason for hiding this comment

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

Hola, desde el punto de vista del codigo, MARAVILLOSO LO QUE HICISTE CON ESTO !!!
y que se puedan correr los test en el iframe tra-or-di-na-rio!!
Sobre el punto de la ejecución, sin ahondar mucho en el tema, puse a correr el test y tengo en el specs al challengeview.cy.tsx. y ejecuta ese test. Hay alguna otra operacion que debemos hacer ? hay que loguearse para hacer alguna otra accion ? O es correcto que sea asi ? (veo la ejecucion del test de lita ok...)
Bueno, lo que consideres aclarar... gracias!

@dlopezalvas
Copy link
Contributor Author

Sobre el punto de la ejecución, sin ahondar mucho en el tema, puse a correr el test y tengo en el specs al challengeview.cy.tsx. y ejecuta ese test. Hay alguna otra operacion que debemos hacer ? hay que loguearse para hacer alguna otra accion ? O es correcto que sea asi ? (veo la ejecucion del test de lita ok...) Bueno, lo que consideres aclarar... gracias!

@danielferro69 No hace falta hacer login, con el comando de npm run test:cy se abre una pantallita que te va guiando, por ahora solo tenemos tests de componentes asi que hay que clickear ahi (los de E2E no estan configurados ni siquiera) y te abre la pantalla para correr los tests en el navegador que quieras. Si sale eso, esta todo OK.

En challengeview.cy.tsx estan todos los tests (sin contar los de #309 que mergee recien) pero por el tema de que faltan pasar bloques a react, las soluciones para los tests fallan, por eso están skipeados, asi que solamente debería correr uno solo que es el que testea la ejecución.

@danielferro69
Copy link
Contributor

Sobre el punto de la ejecución, sin ahondar mucho en el tema, puse a correr el test y tengo en el specs al challengeview.cy.tsx. y ejecuta ese test. Hay alguna otra operacion que debemos hacer ? hay que loguearse para hacer alguna otra accion ? O es correcto que sea asi ? (veo la ejecucion del test de lita ok...) Bueno, lo que consideres aclarar... gracias!

@danielferro69 No hace falta hacer login, con el comando de npm run test:cy se abre una pantallita que te va guiando, por ahora solo tenemos tests de componentes asi que hay que clickear ahi (los de E2E no estan configurados ni siquiera) y te abre la pantalla para correr los tests en el navegador que quieras. Si sale eso, esta todo OK.

En challengeview.cy.tsx estan todos los tests (sin contar los de #309 que mergee recien) pero por el tema de que faltan pasar bloques a react, las soluciones para los tests fallan, por eso están skipeados, asi que solamente debería correr uno solo que es el que testea la ejecución.

Genial !!! si, eso lo vi y ejecutó. Solo que no sabia si era suficiente.
Gracias

@dlopezalvas dlopezalvas merged commit ccaf5b4 into develop Sep 30, 2024
10 checks passed
@dlopezalvas dlopezalvas deleted the execute-tests branch September 30, 2024 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Botón Ejecutar
2 participants