Ho una discussione con un collega.
Pensa che dovremmo iniettare una classe di mappatura (per mappare il domainmodel su un viewmodel) in un controller. Pensa che ora possiamo testare la classe di mappatura (che è una classe di automapper per alcune proprietà) e possiamo iniettare un'altra implementazione quando ne abbiamo bisogno.
La mia opinione è che la logica per questo tipo di mappatura non verrà modificata con un'altra implementazione, né deve essere testata separatamente, perché non viene riutilizzata.
Pensa anche che sia più "elegante" ed è meglio perché è disaccoppiato.
Penso che non dovresti disaccoppiare il tuo codice quando è usato solo in un posto. Secondo me non è male avere un codice che è legato a una sola classe, in quella classe, e che esiste una cosa come "over-disaccoppiamento".
Apprezzerei la tua opinione su questo.