places bubble wrap style (#477)

Author: sarahcodes100

mapzen-places-api/src/main/java/com/mapzen/places/api/internal/PlacePickerActivity.java

@@ -58,7 +58,7 @@ public class PlacePickerActivity extends Activity implements
     }
 
     mapView = (MapView) findViewById(R.id.mz_map_view);
-    mapView.getMapAsync(this);
+    presenter.getMapAsync(mapView, this);
 
     final PlaceDialogFragment fragment =
         (PlaceDialogFragment) getFragmentManager().findFragmentByTag(PlaceDialogFragment.TAG);

mapzen-places-api/src/main/java/com/mapzen/places/api/internal/PlacePickerPresenter.java

@@ -1,5 +1,7 @@
 package com.mapzen.places.api.internal;
 
+import com.mapzen.android.graphics.MapView;
+import com.mapzen.android.graphics.OnMapReadyCallback;
 import com.mapzen.places.api.Place;
 
 import java.util.Map;
@@ -8,6 +10,13 @@
  * Interface for managing state associated with the {@link PlacePickerViewController}.
  */
 interface PlacePickerPresenter {
+
+  /**
+   * Get a reference to {@link com.mapzen.android.graphics.MapzenMap}.
+   * @param mapView
+   */
+  void getMapAsync(MapView mapView, OnMapReadyCallback callback);
+
   /**
    * Sets the presenter's controller. Must be called before any other methods in the interface
    * can be called.

mapzen-places-api/src/main/java/com/mapzen/places/api/internal/PlacePickerPresenterImpl.java

@@ -1,5 +1,7 @@
 package com.mapzen.places.api.internal;
 
+import com.mapzen.android.graphics.MapView;
+import com.mapzen.android.graphics.OnMapReadyCallback;
 import com.mapzen.places.api.Place;
 
 import android.util.Log;
@@ -25,6 +27,10 @@ class PlacePickerPresenterImpl implements PlacePickerPresenter {
     detailFetcher = fetcher;
   }
 
+  @Override public void getMapAsync(MapView mapView, OnMapReadyCallback callback) {
+    mapView.getMapAsync(new PlacesBubbleWrapStyle(), callback);
+  }
+
   @Override public void setController(PlacePickerViewController controller) {
     this.controller = controller;
   }

mapzen-places-api/src/main/java/com/mapzen/places/api/internal/PlacesBubbleWrapStyle.java

@@ -0,0 +1,13 @@
+package com.mapzen.places.api.internal;
+
+import com.mapzen.android.graphics.model.BubbleWrapStyle;
+
+/**
+ * Style used for place picker UI. Sets highest label level as default label level.
+ */
+class PlacesBubbleWrapStyle extends BubbleWrapStyle {
+
+  @Override public int getDefaultLabelLevel() {
+    return 11;
+  }
+}

mapzen-places-api/src/test/java/com/mapzen/places/api/internal/PlacePickerPresenterTest.java

@@ -1,5 +1,7 @@
 package com.mapzen.places.api.internal;
 
+import com.mapzen.android.graphics.MapView;
+import com.mapzen.android.graphics.OnMapReadyCallback;
 import com.mapzen.places.api.Place;
 
 import org.junit.Before;
@@ -8,6 +10,10 @@
 import java.util.HashMap;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.Matchers.any;
+import static org.mockito.Matchers.eq;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.verify;
 
 public class PlacePickerPresenterTest {
 
@@ -20,6 +26,13 @@ public class PlacePickerPresenterTest {
     presenter.setController(controller);
   }
 
+  @Test public void getMapAsync_shouldLoadPlacesBubbleWrapStyle() throws Exception {
+    MapView mapView = mock(MapView.class);
+    OnMapReadyCallback callback = mock(OnMapReadyCallback.class);
+    presenter.getMapAsync(mapView, callback);
+    verify(mapView).getMapAsync(any(PlacesBubbleWrapStyle.class), eq(callback));
+  }
+
   @Test public void onLabelPicked_shouldShowDialog() {
     HashMap<String, String> properties = new HashMap<>();
     properties.put("name", "Hanjan");

mapzen-places-api/src/test/java/com/mapzen/places/api/internal/PlacesBubbleWrapStyleTest.java

@@ -0,0 +1,14 @@
+package com.mapzen.places.api.internal;
+
+import org.junit.Test;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+public class PlacesBubbleWrapStyleTest {
+
+  PlacesBubbleWrapStyle bubbleWrap = new PlacesBubbleWrapStyle();
+
+  @Test public void shouldReturnCorrectLabelLevel() throws Exception {
+    assertThat(bubbleWrap.getDefaultLabelLevel()).isEqualTo(11);
+  }
+}