Correcciones
Básicamente la entrega esta aprobada. Solamente queria comentarles un par de cosas que agregan valor a las tareas que hagan en el día a día.
Porque el directorio donde esta la DB se llama tmp??? ya les estaba criticando que me comitearon cosas de trabajo.
No hace falta que commiteen el .project, el .classpath y el .settings, con mvn eclipse:eclipse lo generan limpitos y no rompen nada. Lo ideal es ponerlos en ignore list. Y así no pelean con la gente, que tiene problemitas y usa otra cosa que el eclipse.
Los test de hibernate tienen efecto de lado, no me gusta.
Si la db no existe o cualquier cosa, los test siguen dando verde. Manejen bien la exception. Igual un punto a favor de que usaron Hibernate Utils. Muy pro. Pueden pensar como hacerlo ustedes y mantenerlo como en un test abstracto o algo así.
Lo de UTF-8 sigue siendo molesto..... aparecen caracterres fruta, pero por lo menos no tiraron nombres con caracteres especiales en los identificadores de java.