Merge pull request #65 from amadeus4dev/pois-retrieve

Add support for Points of Interest retrieve

Author: tsolakoua

README.md

@@ -330,6 +330,9 @@ PointOfInterest[] pointsOfInterest = amadeus.referenceData.locations.pointsOfInt
     .and("south", "41.394582")
     .and("east", "2.177181"));
 
+// Returns a single Point of Interest from a given id
+PointOfInterest pointOfInterest = amadeus.referenceData.locations.pointOfInterest("9CB40CB5D0").get();
+
 // What's the likelihood flights from this airport will leave on time?
 Prediction AirportOnTime = amadeus.airport.predictions.onTime.get(Params
     .with("airportCode", "NCE")

src/main/java/com/amadeus/referenceData/Locations.java

@@ -5,10 +5,10 @@
 import com.amadeus.Response;
 import com.amadeus.exceptions.ResponseException;
 import com.amadeus.referenceData.locations.Airports;
+import com.amadeus.referenceData.locations.PointOfInterest;
 import com.amadeus.referenceData.locations.PointsOfInterest;
 import com.amadeus.resources.Location;
 import com.amadeus.resources.Resource;
-import com.google.gson.Gson;
 
 /**
  * <p>
@@ -49,6 +49,13 @@ public class Locations {
    */
   public PointsOfInterest pointsOfInterest;
 
+  /**
+   * <p>
+   *   A namespaced client for the
+   *   <code>/v2/reference-data/locations/pois</code> endpoints.
+   * </p>
+   */
+  public PointOfInterest pointOfInterest;
 
   /**
    * Constructor.
@@ -86,4 +93,8 @@ public Location[] get(Params params) throws ResponseException {
   public Location[] get() throws ResponseException {
     return get(null);
   }
+
+  public PointOfInterest pointOfInterest(String id) {
+    return new PointOfInterest(client, id);
+  }
 }

src/main/java/com/amadeus/referenceData/locations/PointOfInterest.java

@@ -0,0 +1,63 @@
+package com.amadeus.referenceData.locations;
+
+import com.amadeus.Amadeus;
+import com.amadeus.Params;
+import com.amadeus.Response;
+import com.amadeus.exceptions.ResponseException;
+import com.amadeus.resources.Resource;
+
+/**
+ * <p>
+ *   A namespaced client for the
+ *   <code>/v1/reference-data/locations/pois</code> endpoints.
+ * </p>
+ *
+ * <p>
+ *   Access via the Amadeus client object.
+ * </p>
+ *
+ * <pre>
+ * Amadeus amadeus = Amadeus.builder("clientId", "secret").build();
+ * amadeus.referenceData.locations.pointOfInterest;</pre>
+ */
+public class PointOfInterest {
+  private Amadeus client;
+  private String id;
+
+
+  /**
+   * Constructor.
+   * @hide
+   */
+  public PointOfInterest(Amadeus client, String id) {
+    this.client = client;
+    this.id = id;
+  }
+
+  /**
+   * <p>
+   *   Returns a single Point of Interest from a given id.
+   * </p>
+   *
+   * <pre>
+   * amadeus.referenceData.locations.pointOfInterest("9CB40CB5D0").get();</pre>
+   *
+   * @param params the parameters to send to the API
+   * @return an API response object
+   * @throws ResponseException when an exception occurs
+   */
+  public com.amadeus.resources.PointOfInterest get(Params params) throws ResponseException {
+    String path = String.format("/v1/reference-data/locations/pois/%s", id);
+    Response response = client.get(path, params);
+    return (com.amadeus.resources.PointOfInterest) Resource.fromObject(
+      response, com.amadeus.resources.PointOfInterest.class);
+  }
+
+  /**
+   * Convenience method for calling <code>get</code> without any parameters.
+   * @see PointOfInterest#get()
+   */
+  public com.amadeus.resources.PointOfInterest get() throws ResponseException {
+    return get(null);
+  }
+}

src/main/java/com/amadeus/resources/PointOfInterest.java

@@ -4,7 +4,7 @@
 import lombok.ToString;
 
 /**
- * An PointOfInterest object as returned by the Location API.
+ * A PointOfInterest object as returned by the Location API.
  * @see com.amadeus.referenceData.locations.PointOfInterest#get()
  */
 @ToString

src/test/java/com/amadeus/NamespaceTest.java

@@ -51,6 +51,7 @@ public void testAllNamespacesExist() {
     TestCase.assertNotNull(client.referenceData.locations.airports);
     TestCase.assertNotNull(client.referenceData.locations.pointsOfInterest);
     TestCase.assertNotNull(client.referenceData.locations.pointsOfInterest.bySquare);
+    TestCase.assertNotNull(client.referenceData.locations.pointOfInterest("XXX"));
     TestCase.assertNotNull(client.referenceData.location("123"));
     TestCase.assertNotNull(client.referenceData.airlines);
     TestCase.assertNotNull(client.travel.analytics.airTraffic.traveled);
@@ -149,6 +150,16 @@ public void testGetMethods() throws ResponseException {
     TestCase.assertNotNull(poisSquare.get(params));
     TestCase.assertEquals(poisSquare.get().length, 2);
 
+    // Testing retrieving point of interest
+    Mockito.when(client.get("/v1/reference-data/locations/pois/XXX", null))
+        .thenReturn(multiResponse);
+    Mockito.when(client.get("/v1/reference-data/locations/pois/XXX", params))
+        .thenReturn(multiResponse);
+    PointsOfInterest poi = new PointsOfInterest(client);
+    TestCase.assertNotNull(poi.get());
+    TestCase.assertNotNull(poi.get(params));
+    TestCase.assertEquals(poi.get().length, 2);
+
     // Testing fetching a single location
     Mockito.when(client.get("/v1/reference-data/locations/ALHR", null))
         .thenReturn(singleResponse);