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

Tests desafios con parámetros #1351

Closed
dlopezalvas opened this issue Jun 7, 2023 · 4 comments · Fixed by #1519
Closed

Tests desafios con parámetros #1351

dlopezalvas opened this issue Jun 7, 2023 · 4 comments · Fixed by #1519
Assignees

Comments

@dlopezalvas
Copy link
Contributor

Viene de #1337, quedaron skipeados los tests de Mariposas Encuadradas e Infinitos Trofeos. Están tirando timeout, pero localmente se ve que ni se mueven los personajes (queda trabado y no se ejecuta nada, por eso el timeout).

@dlopezalvas dlopezalvas self-assigned this Jun 7, 2023
@dlopezalvas dlopezalvas mentioned this issue Jun 7, 2023
1 task
@dlopezalvas dlopezalvas moved this from Backlog to In Progress in Creador de Desafíos Jun 7, 2023
@dlopezalvas dlopezalvas moved this from In Progress to Backlog in Creador de Desafíos Jun 16, 2023
@dlopezalvas
Copy link
Contributor Author

Resumen de lo que llegué a ver:

Al guardar la solución como xml, si se la usa para cargarla en un ejercicio anda perfectamente, pero en el test nunca se ejecuta por este checkeo:
https://github.com/Program-AR/pilas-bloques/blob/b30e0dc6a483235d65c83f83694f1dd097d178b6/app/components/pilas-blockly.js#L358-L363

Los bloques de las llamadas a procedimientos cuando tienen parámetros, solamente en los tests fallan en la condición de allInputsFilled (de Blockly)

https://github.com/Program-AR/pilas-bloques/blob/b30e0dc6a483235d65c83f83694f1dd097d178b6/app/services/blocks-gallery.js#L45-L47

Sintacticamente el xml está igual que los demás tests con procedimientos parametrizados. Si se usa un id numérico (iguales al de Tito, o diferentes de todos los demás) como en TitoCuadrado, al correr el test solo (Mariposas encuadradas) corre todo bien, pero si se corren todos los tests juntos, tira este error: https://github.com/Program-AR/pilas-bloques/actions/runs/5193026878/jobs/9363001235#step:6:8895
Cambiando los ids para que no sean números, pasa lo mismo que antes, no se ejecutan por el allInputsFilled.

@asanzo
Copy link
Contributor

asanzo commented Jun 22, 2023

Laburazo Diaaaaaaaa!!! ¿Al final no tenía nada que ver con lo que hizo el Rasta, no? Gracias!!!

@dlopezalvas
Copy link
Contributor Author

No, es algo que pasaba siempre parece (en master estaba pasando también), lo que pasa es que nunca habíamos hecho un ejercicio nuevo con parámetros creo jejej

@asanzo asanzo added this to the 4 Post MVP milestone Jul 12, 2023
@rgonzalezt
Copy link
Contributor

Falta probar sacando los ids de los bloques.

@rgonzalezt rgonzalezt moved this from Backlog to Ready in Creador de Desafíos Oct 31, 2023
@danielferro69 danielferro69 self-assigned this Nov 16, 2023
@danielferro69 danielferro69 moved this from Ready to PR Review Needed in Creador de Desafíos Nov 16, 2023
@danielferro69 danielferro69 moved this from PR Review Needed to In Progress in Creador de Desafíos Nov 16, 2023
@danielferro69 danielferro69 moved this from In Progress to PR Review Needed in Creador de Desafíos Nov 16, 2023
@github-project-automation github-project-automation bot moved this from PR Review Needed to Done in Creador de Desafíos Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants