I have a discussion with a colleague.
He thinks we should inject a mapping class (for mapping the domainmodel to a viewmodel) in a controller. He thinks we now can test the mapping class (which is an automapper class for some properties) and we can inject another implementation when we need it.
My opinion is that the logic for this kind of mapping will not be changed with another implementation nor does it need to be tested separately, because it is not reused.
He also thinks it is more 'elegant' and it's better because it's decoupled.
I think you should not decouple your code when it's only used in one place. In my opinion it is not bad to have code which is tied to one class only, in that class, and that there is a such thing as 'over-decoupling'.
I would appreciate your opinion on this.