search

antoniolg / androidmvp

MVP Android Example
5.94k stars 1.58k forks source link

inject LoginInteractorImpl #33

Closed Sanpetch closed 6 years ago

Sanpetch commented 6 years ago

I think in LoginPresenterImpl Class should inject LoginInteractorImpl in constructor because First, it will not make that constructor depend on another instance.Second, it will easy to mock and test

antoniolg commented 6 years ago

Agreed, sorry for taking so much time into reviewing this.