Skip builders on directions response reading when possible (#1626)

Author: jush

gradle/dependencies.gradle

@@ -2,7 +2,7 @@ ext {
 
   version = [
       autoValue        : '1.7',
-      autoValueGson    : '0.0.3',
+      autoValueGson    : '0.0.4',
       junit            : '4.12',
       annotation       : '1.0.0',
       gson             : '2.13.1',

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/Admin.java

@@ -1,16 +1,19 @@
 package com.mapbox.api.directions.v5.models;
 
 import androidx.annotation.Nullable;
+
 import com.google.auto.value.AutoValue;
 import com.google.gson.Gson;
 import com.google.gson.GsonBuilder;
 import com.google.gson.TypeAdapter;
 import com.google.gson.annotations.SerializedName;
 import com.mapbox.api.directions.v5.DirectionsAdapterFactory;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 /**
  * An objects describing the administrative boundaries the route leg travels through.
  */
+@GsonTypeAdapterConfig(useBuilderOnRead = false)
 @AutoValue
 public abstract class Admin extends DirectionsJsonObject {
 

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/Amenity.java

@@ -2,17 +2,20 @@
 
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
+
 import com.google.auto.value.AutoValue;
 import com.google.gson.Gson;
 import com.google.gson.GsonBuilder;
 import com.google.gson.TypeAdapter;
 import com.mapbox.api.directions.v5.DirectionsAdapterFactory;
 import com.mapbox.api.directions.v5.DirectionsCriteria;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 /**
  * An object containing information about amenities available at rest stops along the route.
  * Only available on the {@link DirectionsCriteria#PROFILE_DRIVING} profile.
  */
+@GsonTypeAdapterConfig(useBuilderOnRead = false)
 @AutoValue
 public abstract class Amenity extends DirectionsJsonObject {
 

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/BannerComponents.java

@@ -10,6 +10,7 @@
 import com.google.gson.TypeAdapter;
 import com.google.gson.annotations.SerializedName;
 import com.mapbox.api.directions.v5.DirectionsAdapterFactory;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 import java.lang.annotation.Retention;
 import java.lang.annotation.RetentionPolicy;
@@ -23,6 +24,7 @@
  *
  * @since 3.0.0
  */
+@GsonTypeAdapterConfig(useBuilderOnRead = false)
 @AutoValue
 public abstract class BannerComponents extends DirectionsJsonObject
   implements Comparable<BannerComponents> {

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/BannerInstructions.java

@@ -8,6 +8,7 @@
 import com.google.gson.GsonBuilder;
 import com.google.gson.TypeAdapter;
 import com.mapbox.api.directions.v5.DirectionsAdapterFactory;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 /**
  * Visual instruction information related to a particular {@link LegStep} useful for making UI
@@ -16,6 +17,7 @@
  *
  * @since 3.0.0
  */
+@GsonTypeAdapterConfig(useBuilderOnRead = false)
 @AutoValue
 public abstract class BannerInstructions extends DirectionsJsonObject {
 

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/BannerText.java

@@ -9,6 +9,7 @@
 import com.google.gson.TypeAdapter;
 import com.google.gson.annotations.SerializedName;
 import com.mapbox.api.directions.v5.DirectionsAdapterFactory;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 import java.util.List;
 
@@ -20,6 +21,7 @@
  *
  * @since 3.0.0
  */
+@GsonTypeAdapterConfig(useBuilderOnRead = false)
 @AutoValue
 public abstract class BannerText extends DirectionsJsonObject {
 

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/BannerView.java

@@ -8,6 +8,7 @@
 import com.google.gson.GsonBuilder;
 import com.google.gson.TypeAdapter;
 import com.mapbox.api.directions.v5.DirectionsAdapterFactory;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 import java.util.List;
 
@@ -19,6 +20,7 @@
  *
  * @since 5.0.0
  */
+@GsonTypeAdapterConfig(useBuilderOnRead = false)
 @AutoValue
 public abstract class BannerView extends DirectionsJsonObject {
 

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/Bearing.java

@@ -1,12 +1,18 @@
 package com.mapbox.api.directions.v5.models;
 
 import androidx.annotation.NonNull;
+
 import com.google.auto.value.AutoValue;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 /**
  * A value pair of an {@link #angle} which is a clockwise value from true north between 0 and 360,
  * and a {@link #degrees()} which is the range of degrees by which the angle can deviate.
  */
+@GsonTypeAdapterConfig(
+        // The builder sets default values so we have to use it while deserializing
+        useBuilderOnRead = true
+)
 @AutoValue
 public abstract class Bearing extends DirectionsJsonObject {
 

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/Closure.java

@@ -8,10 +8,12 @@
 import com.google.gson.TypeAdapter;
 import com.google.gson.annotations.SerializedName;
 import com.mapbox.api.directions.v5.DirectionsAdapterFactory;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 /**
  * An object indicating the geometry indexes defining a road closure.
  */
+@GsonTypeAdapterConfig(useBuilderOnRead = false)
 @AutoValue
 public abstract class Closure extends DirectionsJsonObject {
 

services-directions-models/src/main/java/com/mapbox/api/directions/v5/models/Congestion.java

@@ -5,10 +5,12 @@
 import com.google.gson.GsonBuilder;
 import com.google.gson.TypeAdapter;
 import com.mapbox.api.directions.v5.DirectionsAdapterFactory;
+import com.mapbox.auto.value.gson.GsonTypeAdapterConfig;
 
 /**
  * Quantitative descriptor of congestion.
  */
+@GsonTypeAdapterConfig(useBuilderOnRead = false)
 @AutoValue
 public abstract class Congestion extends DirectionsJsonObject {