Generally, I would recommend that you don't change the interface of the class to implement. However, I can see the need for this constructor to help with unit tests, so I am fine with this.