RodrigoToroIcarte / IIC2113-2023-1

6 stars 0 forks source link

Modo debug [E2] #155

Open vicente999 opened 1 year ago

vicente999 commented 1 year ago

Hola! Tengo una duda, que me fallo un test de los invalid deck pero si corro el modo debug de render me sale que esta bien, que hago en este caso? Ademas revisando el detalle del test estan iguales y no se que podria fallar :( (Los test de invalid deck los tenia todos buenos para la E1)

RodrigoToroIcarte commented 1 year ago

¿Si corres el test individualmente falla?

Screenshot from 2023-09-09 22-19-21

vicente999 commented 1 year ago

No, ahi funcionan! pero al correr todos los test fallan

RodrigoToroIcarte commented 1 year ago

Ya, lo que probablemente está ocurriendo es que estás usando cosas static que es mejor no usar. Los tests se corren en paralelo. Pero si tienes un atributo static su valor es el mismo en todos los threads. Tener métodos static da igual, pero un atributo static es complicado pues su valor puede ser cambiado por un thread de otro test... haciendo que todo falle.

Ver: https://github.com/RodrigoToroIcarte/IIC2113-2023-1/issues/49 y https://github.com/RodrigoToroIcarte/IIC2113-2023-1/issues/106.

vicente999 commented 1 year ago

Ya perfecto! gracias

vicente999 commented 1 year ago

Lo que hice fue crear los players de forma static, asi los podia llamar desde cualquier parte

image
RodrigoToroIcarte commented 1 year ago

Era una buena idea... lástima que testear cosas static sea un cacho :(

vicente999 commented 1 year ago

Y como puedo hacer para mantener el codigo limpio y no mandar como argumento a todas mis funciones arrastrando el playerOne y el playerTwo? O no hay forma jajajja

RodrigoToroIcarte commented 1 year ago

Nadie dijo que limpiar código fuera fácil :man_shrugging:

vicente999 commented 1 year ago

jajajaja esta bien, muchas gracias! (Ya se arreglo el error)