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

Test desafios con parametros #1519

Merged
merged 5 commits into from
Nov 21, 2023
Merged

Test desafios con parametros #1519

merged 5 commits into from
Nov 21, 2023

Conversation

danielferro69
Copy link
Contributor

@danielferro69 danielferro69 commented Nov 16, 2023

Resolves #1351

Por alguna razon (que no pude descubrir a pesar de poder revisar el codigo) estos test que reciben parametros funcionan cuando el valor que se pasa, se pasa asi:

           <value name="direccion">
                <block type="param_get" id="26">
                    <field name="VAR">direccion</field>
                </block>
           </value>

y no asi:

            <value name="direccion">
                  <block type="variables_get" id="A$I=sXn2/C/-w-dC{cM$">
                       <mutation var="direccion" parent="126"></mutation>
                  </block>
             </value>

Como que la interpretacion de mutation no estaria funcionando de la misma forma que lo hace cuando se utiliza field name.
luego, si el bloque es de tipo variables_get o param_get da igual.

Cambiando esto en las soluciones, resolvió los tests

Parecia que el test se freezaba pero en realidad no se podia ejecutar (y por ello el timeout) por falta de bloques.
Cuando se ejecutan los tests si se destilda la opcion que oculta el contenedor del test (hide container), se pueden observar los bloques en pantalla y no solo la escena y alli se podia observar que faltaban bloques, aunque en realidad estaban, pero bajo la "definicion" de <mutation var>

image

y por otro lado, el error de los 2 test en el test:CI esta dado por el highlighter de procedimientos al analizar el campo id (que no tiene que ver con el id que le colocamos o no en la solucion). Supongo (sin mucho mas análisis realmente) que es a raiz de intentar highlightear (valga la expresion) la definicion del procedimiento y no el momento de la ejecucion. Pero no explica por qué no falla en TitoRecargado o en el CangrejoAguafiestas SALVO por el hecho de que estos desafios tienen una escena propia definida en exercises y los dos nuevos se construyen a partir de la escena generica de cada personaje (EscenaCapy/EscenaYvoty)
La alternativa o contraprueba para verificar esto (que no hice aun) sería hacer para al menos uno de estos nuevos desafios, una clase en exercises como TitoRecargado, digamos un InifinitosTrofeos especifico y ver si a partir de alli, el test (volviendo el highlighter a como estaba) no falla. Porque es el unico argumento que puedo esgrimir para explicar la diferencia y por qué falla en uno y en otro no siendo que ambos tienen la misma solucion (porque salvo un par de bloques, las soluciones son identicas).

@danielferro69 danielferro69 requested a review from a team as a code owner November 16, 2023 12:38
@danielferro69 danielferro69 marked this pull request as draft November 16, 2023 16:24
@danielferro69 danielferro69 marked this pull request as ready for review November 16, 2023 22:48
@danielferro69 danielferro69 merged commit 6bcfd2b into develop Nov 21, 2023
8 checks passed
@danielferro69 danielferro69 deleted the testWithParameters branch November 21, 2023 15:24
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.

Tests desafios con parámetros
2 participants