Added unit tests and toBuilder method

mapbox · Feb 12, 2019 · ad63326 · ad63326
1 parent a6e16fc
commit ad63326
Show file tree

Hide file tree

Showing 2 changed files with 41 additions and 6 deletions.
diff --git a/...ns-refresh/src/main/java/com/mapbox/api/directionsrefresh/v5/MapboxDirectionsRefresh.java b/...ns-refresh/src/main/java/com/mapbox/api/directionsrefresh/v5/MapboxDirectionsRefresh.java
@@ -65,12 +65,11 @@ protected GsonBuilder getGsonBuilder() {
   }
 
   /**
-   * Convert the current {@link MapboxDirectionsRefresh} to its builder holding the currently assigned
-   * values. This allows you to modify a single property and then rebuild the object resulting in
-   * an updated and modified {@link MapboxDirectionsRefresh}.
+   * Convert the current {@link MapboxDirectionsRefresh} to its builder holding the currently
+   * assigned values. This allows you to modify a single property and then rebuild the object
+   * resulting in an updated and modified {@link MapboxDirectionsRefresh}.
    *
-   * @return a {@link MapboxDirectionsRefresh.Builder} with the same values set to match the ones defined
-   *   in this {@link MapboxDirectionsRefresh}
+   * @return a {@link MapboxDirectionsRefresh.Builder} with the same values
    * @since 4.4.0
    */
   public abstract Builder toBuilder();

diff --git a/...efresh/src/test/java/com/mapbox/api/directionsrefresh/v5/MapboxDirectionsRefreshTest.java b/...efresh/src/test/java/com/mapbox/api/directionsrefresh/v5/MapboxDirectionsRefreshTest.java
@@ -1,5 +1,6 @@
 package com.mapbox.api.directionsrefresh.v5;
 
+import com.mapbox.api.directionsrefresh.v5.models.DirectionsRefreshResponse;
 import com.mapbox.core.TestUtils;
 
 import org.hamcrest.junit.ExpectedException;
@@ -15,6 +16,10 @@
 import okhttp3.mockwebserver.MockResponse;
 import okhttp3.mockwebserver.MockWebServer;
 import okhttp3.mockwebserver.RecordedRequest;
+import retrofit2.Response;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
 
 public class MapboxDirectionsRefreshTest extends TestUtils {
 
@@ -31,7 +36,6 @@ public void setUp() throws IOException {
       @Override
       public MockResponse dispatch(RecordedRequest request) throws InterruptedException {
 
-        // Switch response on geometry parameter (only false supported, so nice and simple)
         String resource = DIRECTIONS_REFRESH_V5_FIXTURE;
 
         try {
@@ -63,4 +67,36 @@ public void sanity() throws Exception {
 
     Assert.assertNotNull(mapboxDirectionsRefresh);
   }
+
+  @Test
+  public void testResponse() throws IOException {
+    MapboxDirectionsRefresh mapboxDirectionsRefresh = MapboxDirectionsRefresh.builder()
+      .accessToken(ACCESS_TOKEN)
+      .requestId("")
+      .baseUrl(mockUrl.toString())
+      .build();
+
+    Response<DirectionsRefreshResponse> response = mapboxDirectionsRefresh.executeCall();
+    DirectionsRefreshResponse directionsRefreshResponse = response.body();
+
+    assertEquals(200, response.code());
+    assertEquals("Ok", directionsRefreshResponse.code());
+    assertNotNull(directionsRefreshResponse.route().legs());
+    assertNotNull(directionsRefreshResponse.route().legs().get(0).annotation());
+  }
+
+  @Test
+  public void testRoute() throws IOException {
+    MapboxDirectionsRefresh mapboxDirectionsRefresh = MapboxDirectionsRefresh.builder()
+      .accessToken(ACCESS_TOKEN)
+      .requestId("")
+      .baseUrl(mockUrl.toString())
+      .build();
+
+    Response<DirectionsRefreshResponse> response = mapboxDirectionsRefresh.executeCall();
+    DirectionsRefreshResponse directionsRefreshResponse = response.body();
+
+    assertNotNull(directionsRefreshResponse.route().legs());
+    assertNotNull(directionsRefreshResponse.route().legs().get(0).annotation());
+  }
 }