[android] test tweaks based on Lukasz reviewupstream/ls-builder-pattern-for-android-LocationComponent-activation

2 files changed, 3 insertions, 16 deletions

diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationComponentActivationOptions.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationComponentActivationOptions.java
index 8bcfd11367..3f71209d10 100644
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationComponentActivationOptions.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationComponentActivationOptions.java
@@ -198,6 +198,9 @@ public class LocationComponentActivationOptions {
      * @param useDefaultLocationEngine true if you want to initialize and use the
      *                                 built-in location engine or false if there
      *                                 should be no location engine initialized
+     *                                 This is ignored when null is set as the parameter
+     *                                 for {@link LocationComponentActivationOptions#builder
+     *                                 (Context, Style)#locationEngine()}.
      * @return the {@link Builder} object being constructed
      */
     public Builder useDefaultLocationEngine(boolean useDefaultLocationEngine) {
diff --git a/platform/android/MapboxGLAndroidSDK/src/test/java/com/mapbox/mapboxsdk/location/LocationComponentActivationOptionsTest.java b/platform/android/MapboxGLAndroidSDK/src/test/java/com/mapbox/mapboxsdk/location/LocationComponentActivationOptionsTest.java
index f1448a60c6..2660c819d4 100644
--- a/platform/android/MapboxGLAndroidSDK/src/test/java/com/mapbox/mapboxsdk/location/LocationComponentActivationOptionsTest.java
+++ b/platform/android/MapboxGLAndroidSDK/src/test/java/com/mapbox/mapboxsdk/location/LocationComponentActivationOptionsTest.java
@@ -77,7 +77,6 @@ public class LocationComponentActivationOptionsTest {
     LocationComponentOptions locationComponentOptions = LocationComponentOptions.builder(context)
       .accuracyAlpha(0.5f)
       .build();
-    assertNotNull(locationComponentOptions);
 
     LocationComponentActivationOptions.builder(context, style)
       .locationComponentOptions(locationComponentOptions)
@@ -90,11 +89,6 @@ public class LocationComponentActivationOptionsTest {
     thrown.expect(NullPointerException.class);
     thrown.expectMessage("Context in LocationComponentActivationOptions is null.");
 
-    LocationComponentOptions locationComponentOptions = LocationComponentOptions.builder(context)
-      .accuracyAlpha(0.5f)
-      .build();
-    assertNotNull(locationComponentOptions);
-
     LocationComponentActivationOptions.builder(null, style)
       .build();
   }
@@ -105,11 +99,6 @@ public class LocationComponentActivationOptionsTest {
     thrown.expectMessage("Style in LocationComponentActivationOptions is null. Make sure the Style object isn't null."
       + " Wait for the map to fully load before passing the Style object to LocationComponentActivationOptions.");
 
-    LocationComponentOptions locationComponentOptions = LocationComponentOptions.builder(context)
-      .accuracyAlpha(0.5f)
-      .build();
-    assertNotNull(locationComponentOptions);
-
     LocationComponentActivationOptions.builder(context, null)
       .build();
   }
@@ -124,11 +113,6 @@ public class LocationComponentActivationOptionsTest {
         + "map to fully load before passing the Style object to "
         + "LocationComponentActivationOptions.");
 
-    LocationComponentOptions locationComponentOptions = LocationComponentOptions.builder(context)
-        .accuracyAlpha(0.5f)
-        .build();
-    assertNotNull(locationComponentOptions);
-
     LocationComponentActivationOptions.builder(context, style)
         .build();
   }