mapzen / eraser-map

Privacy-focused mapping application for Android
GNU General Public License v3.0
74 stars 24 forks source link

ConfidenceHandler refactor #782

Closed ecgreb closed 7 years ago

ecgreb commented 7 years ago

As a model ConfidenceHandler should not know about the presenter. This change removes MainPresenter as a dependency of ConfidenceHandler and instead uses constructor injection to provide an instance of ConfidenceHandler to the presenter when it is created.

This also refactors MainPresenter to not track reverseGeoLngLat itself but instead store that information in the ConfidenceHandler model.

Finally it adds unit tests and docs to ConfidenceHandler.