GitLab

- no cambiamos, nada por la solución en clase, solo ponemos unas reflexiones

en esta entraga usamos la solución de la cátedra para explorar otro punto del espacio del diseño

el commit 4f7a5758 tenía el corazón en lo cierto pero la cabeza en otro lado, no había subido el fix.

De todas maneras el commit de entrega: c8ebcde0 es válido desde cuis 4137

desde el commit c8ebcde0 todo anda.

El commit 4f7a5758 es un error, no arregla nada, no hay nada que arreglar.

pd: usamos el cuis de versión 4137

lo que fué entregado está en el branch

tus-libros-entrega-iteracion-2

lo habíamos definido para que lance error

- duplicamos catalogo y lista de precios, así que necesitamos un local que se encargue de la consistencia

MerchantProcessor -> TusLibros

no se porque usé MerchantProcessor y luego quedó ... pero es mal nombre

similar a como hacen los docentes en el branch de cátedra

- no acoplar con colaboradores internos: #cantidadDeLibros, #contarLibros:
- los tests de error al agregar, deben asegurar que no se agrego
- un test para desacoplar la colección devuelta del contenido del carrito; es decir que no se pueda modificar desde afuera

lo único que no saqué fue la redefinición de #new ... no me termina de convencer; por ahora al menos...

+ mejores comentarios en las clases de reportes

separamos los tests a una categoría

- es más facil de comparar con la alternativa
  - tiene mensajes de la misma
  - no subclasificamos reporte todavía