search

UNIZAR-30226-2024-02 / Back-end

MIT License
1 stars 2 forks source link

Tests unitarios lógica de partida #74

Open Juanguppy opened 6 months ago

737275 commented 6 months ago

¿Han sido probadas las funciones? Estaba lanzando peticiones a la ruta de realizarManiobra y su función correspondiente me está dando errores que hacen creer que este código nunca se ha ejecutado, por ejemplo la función comprobarTerritorio se está llamando con un número incorrecto de parámetros y la función numJugador no está esperando la resolución de la promesa y está generando fallos debido a no tener los valores a tiempo durante la ejecución de las comprobaciones...

Si este es el caso, se ha de reabrir el issue correspondiente a la creación de la lógica de la partida.

Juanguppy commented 6 months ago

¿Han sido probadas las funciones? Estaba lanzando peticiones a la ruta de realizarManiobra y su función correspondiente me está dando errores que hacen creer que este código nunca se ha ejecutado, por ejemplo la función comprobarTerritorio se está llamando con un número incorrecto de parámetros y la función numJugador no está esperando la resolución de la promesa y está generando fallos debido a no tener los valores a tiempo durante la ejecución de las comprobaciones...

Si este es el caso, se ha de reabrir el issue correspondiente a la creación de la lógica de la partida.

El issue de lógica de partida nunca se ha cerrado, porque no está terminada ( #25 ). Respecto a lo que hay, por lo que he debuggeado parece que está literalmente hecho con chatGPT 3.5 en español y que ni se ha probado, lo cual es una práctica que no me parece correcta como encargado de calidad del proyecto. Si se usa chatGPT que se use como auxiliar y si se utiliza para hacer el esqueleto de algo que al menos se pruebe y se debuggee. Y de todas formas está copilot gratis con el plan de estudiante que funciona bastante mejor que gpt 3.5 el cual últimamente se fuma buenos porros.

FabiMur commented 6 months ago

No, no esta hecho con ChatGPT. Un poco de cuaidado antes de ponerse a acusar a la gente. El codigo ese lo escribi simplemente como primer boceto, fui parcheando cosas sobre la marcha y no he podido ponerme a probarlo, por eso hay errores del estilo. Si no os gusta como esta diseñado sois libres de hacerlo de cualquier otra forma.

Juanguppy commented 6 months ago

No, no esta hecho con ChatGPT. Un poco de cuaidado antes de ponerse a acusar a la gente. El codigo ese lo escribi simplemente como primer boceto, fui parcheando cosas sobre la marcha y no he podido ponerme a probarlo, por eso hay errores del estilo. Si no os gusta como esta diseñado sois libres de hacerlo de cualquier otra forma.

Disculpa si te ha molestado lo de chatGPT, era una broma. Una cmoparación de que el código parecía ser de chatGPT debido a la cantidad de errores de "sintaxis". Sin embargo la lógica estaba casi perfecta y chatGPT ni de coña lo hace. El diseño estaba perfecto salvo pequeños detalles (que he tocado porque facilitan la conexión del front con el back, pero nada más), lo que estaba mal era que no cumplía la sintaxis de javaScript y ya, pero eso se solucinoa con un IDE con las extensiones necesarias o haciendo simplemente una prueba tonta.

Los test unitarios osn necesarios y creo q obligatorios, pero en mi opinión prefiero que lo pruebes aunque sea por encima antes de ponerte con los tests. Disculpa de nuevo si te ha molestado, no era mi intención más que darte un aviso de que debes asegurarte de que por lo menos funcionan las cosas. Las normas que pusimos fueron no subir nada que no compile a ni nguna de las ramas, y no subir nada que no esté terminado al main.