IIC2143 / Syllabus

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

Sobre el test 2A 19 #28

Closed dvillas28 closed 5 days ago

dvillas28 commented 5 days ago

Hola, estoy revisando que está pasando con los tests que aún no lo logro pasar en mi tarea, y al analizar el test 19, que es el que revisa el endpoint PATCH reviews/:review_id.

Segun el enunciado este endpoint recibe como request body un json de este estilo image

El tema es que en el tester (test_2A.py) lo están probando con un json nada que ver, ya que ahi hay un campo "rating" que no sale en ningun lado de la tarea image

Esto provoca que mi código tiré una excepción ya que está esperando por el campo "title" de "review"

No estoy seguro de si nuestro código debiera en realidad actuar acorde a recibir un request body invalido?, o es un test case que estara malo quizas?

Quedo atento Saludos! 🤠

jtvaldivia commented 5 days ago

Hola, s,i es un typo que se nos paso de cuando estabamos desarrollando el enunciado, lo actualizo ahora. Igualmente el tester deberia colocarlo correcto al ser un patch que actualiza la forma del modelo a nivel de bdd.

Igualmente, ojo que le patch es para actualizar cualquier atributo de las reviews, asique igualmente deberia aparecerte como correcto el test. Lo mas probable es que estas haciendo patch solo para title

dvillas28 commented 5 days ago

Entiendo, es posible que entreguen un request body con mas de un atributo a cambiar? ¿o podemos asumir que es solo un atributo el se va a cambiar?

jtvaldivia commented 5 days ago

Sip es posible que entreguemos mas de un atributo a cambiar

dvillas28 commented 5 days ago

perfecto, gracias!