return the actual error coming back from nn when fetching an offline route fails

Author: Guardiola31337

libandroid-navigation/src/main/java/com/mapbox/services/android/navigation/v5/navigation/OfflineRoute.java

@@ -4,7 +4,6 @@
 import android.support.annotation.NonNull;
 import android.support.annotation.Nullable;
 
-import com.google.gson.Gson;
 import com.mapbox.api.directions.v5.models.DirectionsResponse;
 import com.mapbox.api.directions.v5.models.DirectionsRoute;
 import com.mapbox.core.exceptions.ServicesException;
@@ -14,7 +13,6 @@
 import java.util.List;
 
 import okhttp3.HttpUrl;
-import timber.log.Timber;
 
 /**
  * The {@link OfflineRoute} class wraps the {@link NavigationRoute} class with parameters which
@@ -73,10 +71,10 @@ public String buildUrl() {
   @Nullable
   DirectionsRoute retrieveOfflineRoute(@NonNull RouterResult response) {
     boolean success = response.getSuccess();
-    String jsonResponse = response.getJson();
-    if (checkOfflineRoute(success, jsonResponse)) {
+    if (!checkOfflineRoute(success)) {
       return null;
     }
+    String jsonResponse = response.getJson();
     return obtainRouteFor(jsonResponse);
   }
 
@@ -155,11 +153,8 @@ private String buildOfflineUrl(String onlineUrl) {
     return offlineUrlBuilder.build().toString();
   }
 
-  private boolean checkOfflineRoute(boolean isSuccess, String json) {
-    if (!isSuccess) {
-      Gson gson = new Gson();
-      OfflineRouteError error = gson.fromJson(json, OfflineRouteError.class);
-      Timber.e("Error occurred fetching offline route: %s - Code: %d", error.getError(), error.getErrorCode());
+  private boolean checkOfflineRoute(boolean isSuccess) {
+    if (isSuccess) {
       return true;
     }
     return false;

libandroid-navigation/src/main/java/com/mapbox/services/android/navigation/v5/navigation/OfflineRouteRetrievalTask.java

@@ -1,15 +1,20 @@
 package com.mapbox.services.android.navigation.v5.navigation;
 
 import android.os.AsyncTask;
+import android.support.annotation.NonNull;
 
+import com.google.gson.Gson;
 import com.mapbox.api.directions.v5.models.DirectionsRoute;
 import com.mapbox.navigator.Navigator;
 import com.mapbox.navigator.RouterResult;
 
+import timber.log.Timber;
+
 class OfflineRouteRetrievalTask extends AsyncTask<OfflineRoute, Void, DirectionsRoute> {
   private static final int FIRST_ROUTE = 0;
   private final Navigator navigator;
   private final OnOfflineRouteFoundCallback callback;
+  private RouterResult routerResult;
 
   OfflineRouteRetrievalTask(Navigator navigator, OnOfflineRouteFoundCallback callback) {
     this.navigator = navigator;
@@ -18,7 +23,6 @@ class OfflineRouteRetrievalTask extends AsyncTask<OfflineRoute, Void, Directions
 
   @Override
   protected DirectionsRoute doInBackground(OfflineRoute... offlineRoutes) {
-    RouterResult routerResult;
     String url = offlineRoutes[FIRST_ROUTE].buildUrl();
 
     synchronized (navigator) {
@@ -33,8 +37,20 @@ protected void onPostExecute(DirectionsRoute offlineRoute) {
     if (offlineRoute != null) {
       callback.onRouteFound(offlineRoute);
     } else {
-      OfflineError error = new OfflineError("Offline route was not found");
+      String errorMessage = generateErrorMessage();
+      OfflineError error = new OfflineError(errorMessage);
       callback.onError(error);
     }
   }
+
+  @NonNull
+  private String generateErrorMessage() {
+    String jsonResponse = routerResult.getJson();
+    Gson gson = new Gson();
+    OfflineRouteError routeError = gson.fromJson(jsonResponse, OfflineRouteError.class);
+    String errorMessage = String.format("Error occurred fetching offline route: %s - Code: %d", routeError.getError(),
+      routeError.getErrorCode());
+    Timber.e(errorMessage);
+    return errorMessage;
+  }
 }