2010C2‎ > ‎Correcciones‎ > ‎

Grupo3Entrga3

Gente, les paso las cosas que vi en la entrega:

- Un detalle que vi, que no habia notado: Es una buena practica y 
convención que los paquetes llevan nombres en singular. Por ejemplo resource y no resources.
No hace falta que lo cambien, solo sepanlo.

- TestAnalisisResource esta marcado como @Ignore, a que se debe esto? Fue accidental?
Traten de no commitear código con @Ignore, y si lo hacen ponganle un //FIXME con la 
explicacion de la decision y diciendo cuando hay que quitar el @Ignore.

- FirstServerResource, lo mismo, no deberían comittear código de prueba, menos en la carpeta
main, y llegado el caso si se ven forzados a hacerlo deberian poner un //FIXME para explicar
cuando tiene que ser eliminado.

- Bien que incorporaron Junit 4, pero tengan cuidado con mezclar la version 3 y 4, 
particularmente con que los tests extiendan TestCase, porque puede traer problemas.

- AnalisisListServerResource tiene un mapa analisisMap donde guardan los analisis.
No esta mal, pero teniendo en cuenta que van a tener que implementar la capa de persistencia,
quizas les conviene empezar a manejarse con otro esquema, por ejemplo un DAO que si puede 
tener el mapa por ahora, lo cual hace que el dia que lo cambien, por ejemplo contra una base
de datos relacional, AnalisisListServerResource siga funcionando igual y no se entere de la
modificacion. Lo mismo pasa con medicos y pacientes.

- IMPORTANTE: Esta bien que restlet permita levantar el servidor "a mano" por decirlo de 
alguna forma. Pero traten de usar un webserver, por ejemplo Jetty. En el ejemplo que mando
Hernan a la lista de alumnos hay una integracion entre Spring y Restlet que corre con jetty.
Esto permite por ejemplo correrlo usando un comndo de maven (mvn jetty:run) y dejar la app
levantada para usarla. Aca hay otro link que les puede servir: 
http://www.restlet.org/documentation/1.0/faq#02 Cualquier cosa si tienen problemas con esto
avisen y lo vemos.

La entrega está bastante bien, buen trabajo.
Con respecto a los servicios rest, van por buen camino. Los tests que armaron son más de integración que unitarios, ya que levantan un servidor y le pegan para ver que fucniona. Igual prueban lo que tienen que probar, pero en futuras iteraciones se les puede complicar y van a tener que mockear la capa de persistencia para no guardar en la base de "producción" lo que hagan en los tests.

Sigamos con los pasos que veníamos hablando, y denle bola al tema de Jetty (o tomcat, cualquier web server, pero que sea una aplicación que se pueda deployar en un container)
Comments