Program-AR / pilas-bloques-app

Pilas Bloques es una aplicación para enseñar y aprender a programar por medio de bloques de forma simple y divertida.
https://pilasbloques.program.ar/
GNU Affero General Public License v3.0
5 stars 6 forks source link

Execute tests #310

Closed dlopezalvas closed 1 month ago

dlopezalvas commented 2 months ago

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 commented 1 month ago

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 https://github.com/Program-AR/pilas-bloques-app/pull/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 commented 1 month ago

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