Add isLayerReady check before adding compass.

mapbox · Oct 22, 2019 · c80ae81 · c80ae81
1 parent db02638
commit c80ae81
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 3 deletions.
diff --git a/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationComponent.java b/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationComponent.java
@@ -1289,7 +1289,7 @@ private void updateCompassListenerState(boolean canListen) {
         return;
       }
 
-      if (!isComponentInitialized || !isComponentStarted || !isEnabled) {
+      if (!isComponentInitialized || !isComponentStarted || !isEnabled || !isLayerReady) {
         return;
       }
 

diff --git a/MapboxGLAndroidSDK/src/test/java/com/mapbox/mapboxsdk/location/LocationComponentTest.kt b/MapboxGLAndroidSDK/src/test/java/com/mapbox/mapboxsdk/location/LocationComponentTest.kt
@@ -386,6 +386,23 @@ class LocationComponentTest {
     verify(compassEngine, never()).addCompassListener(any(CompassListener::class.java))
   }
 
+  @Test
+  fun compass_notAddListenerWhenLayerNotReady() {
+    locationComponent.activateLocationComponent(context, mock(Style::class.java), locationEngine, locationEngineRequest, locationComponentOptions)
+    locationComponent.onStart()
+    locationComponent.isLocationComponentEnabled = true
+    `when`(mapboxMap.cameraPosition).thenReturn(CameraPosition.DEFAULT)
+    `when`(locationLayerController.isConsumingCompass).thenReturn(true)
+    locationComponent.renderMode = RenderMode.COMPASS
+
+    verify(compassEngine, times(1)).addCompassListener(any(CompassListener::class.java))
+
+    locationComponent.onStartLoadingMap()
+    // Layer should be disabled at this point
+    locationComponent.setCameraMode(CameraMode.TRACKING_COMPASS)
+    verify(compassEngine, times(1)).addCompassListener(any(CompassListener::class.java))
+  }
+
   @Test
   fun developerAnimationCalled() {
     locationComponent.activateLocationComponent(context, mock(Style::class.java), locationEngine, locationEngineRequest, locationComponentOptions)

diff --git a/...p/src/main/java/com/mapbox/mapboxsdk/testapp/activity/location/LocationModesActivity.java b/...p/src/main/java/com/mapbox/mapboxsdk/testapp/activity/location/LocationModesActivity.java
@@ -233,8 +233,10 @@ private void toggleMapStyle() {
       return;
     }
 
-    String styleUrl = Style.DARK.equals(mapboxMap.getStyle().getUri()) ? Style.LIGHT : Style.DARK;
-    mapboxMap.setStyle(new Style.Builder().fromUri(styleUrl));
+    mapboxMap.getStyle(style -> {
+      String styleUrl = Style.DARK.equals(style.getUri()) ? Style.LIGHT : Style.DARK;
+      mapboxMap.setStyle(new Style.Builder().fromUri(styleUrl));
+    });
   }
 
   private void disableGesturesManagement() {