mapzen / eraser-map

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

Improve test coverage #791

Closed sarahsnow1 closed 7 years ago

sarahsnow1 commented 7 years ago

Overview

This PR moves pure java code out of MainActivity and into MainPresenterImpl as well as adds more test coverage for the moved logic.

Proposed Changes

Map listener handling (for onMapPressed and onMapDoubleTapped) has been moved into the presenter. In addition, showSearchResults and addSearchResultsToMap have both been moved into the presenter and replaced by smaller methods in the MainViewController.