IIC2143 / Syllabus

IIC2143 Ingeniería de Software 2024-2 🚀✨
66 stars 0 forks source link

Duda test 2C 24 #52

Open fschiappacasse opened 2 months ago

fschiappacasse commented 2 months ago

Holaaa, me falla el paso 24 del test 2C, el cual es: 24: (False, "validacion incorrecta") if isinstance(get_game_reviews(bad_games[0]), tuple) and get_game_reviews(bad_games[0])[0] else (True, "validacion correcta"),

No me queda claro que hace este test, ya que no podría sacar una review de un juego que no puedo subir (bad_games), entonces que se hace específicamente en este test?

Muchas gracias

andry-19 commented 2 months ago

Tengo el mismo problema :( Es el único test case que me falla y no lo estoy logrando entender tampoco

waknaudt commented 2 months ago

A mi tmb me fallaba, dps de leer los tests y los logs del server, al parecer para python bad_games[0].id es None, probablemente pq este juego no se acepto al tener datos erroneos asi que no tiene id. Entonces el test de python hace la request get a /games/None/reviews. Prueba hacer el get /games/None/reviews en postman, si te responde con [] entonces es por eso que el test falla, ya que el test de python get_game_reviews retornara True. Dicho test retorna True pq [] es una respuesta valida para /games/:game_id/reviews si no se encuentran reviews para el game con id :game_id. Como no hay juegos con id None, la respuesta [] tiene sentido. Una solucion seria validar que :game_id sea un int en el metodo correspondiente, y si no lo es simplemente no dar respuesta, aunque no se si esto es lo ideal o lo que se espera :s

noamsito commented 2 months ago

En efecto, no se debe de generar una response porque finalmente estamos realizando un get de un game que no existe, por lo que retornar algo sería lo incorrecto.