guidopj / rentauto

Base para todos los TP de la materia
0 stars 13 forks source link

Implementación EnviadorMail #4

Open aitrusgit opened 9 years ago

aitrusgit commented 9 years ago

¿Por qué esta implementación?

image

Está mal que exista. Si ustedes la necesitan para testear entonces eso es un mock (una implementación dummy, digamos) y debería ir en el folder de los tests.

Además, no está bien que su servicio ya esté acoplado a ese implementación:

image

En cambio quedaría mas prolijo que la recibiese por parametro (inyección de dependencia)

image

aitrusgit commented 8 years ago

Esto no esta arreglado