Merge pull request #155 from siddydutta/hotel-name-autocomplete

Add Support of Hotel Name Autocomplete API

Author: tsolakoua

README.rst

@@ -277,6 +277,9 @@ List of supported endpoints
     # Get list of hotels by a geocode
     amadeus.reference_data.locations.hotels.by_geocode.get(longitude=2.160873,latitude=41.397158)
 
+    # Hotel Name Autocomplete
+    amadeus.reference_data.locations.hotel.get(keyword='PARI', subType=[Hotel.HOTEL_GDS, Hotel.HOTEL_LEISURE])
+
     # Hotel Booking
     # The offerId comes from the hotel_offer above
     amadeus.booking.hotel_bookings.post(offerId, guests, payments)

amadeus/client/hotel.py

@@ -0,0 +1,22 @@
+#
+class Hotel(object):
+    '''
+    A list of hotel sub types, as used in Hotel Name Autocomplete
+
+    .. code-block:: python
+
+
+        from amadeus import Hotel
+
+        amadeus.reference_data.locations.hotel.get(
+            keyword='PARI',
+            subType=[Hotel.HOTEL_LEISURE, Hotel.HOTEL_GDS]
+        )
+
+    :cvar HOTEL_LEISURE: ``"HOTEL_LEISURE"``
+    :cvar HOTEL_GDS: ``"HOTEL_GDS"``
+    '''
+    # Hotel Leisure
+    HOTEL_LEISURE = 'HOTEL_LEISURE'
+    # Hotel GDS
+    HOTEL_GDS = 'HOTEL_GDS'

amadeus/client/request.py

@@ -138,7 +138,7 @@ def __port_matches_scheme(self):
 
     # Helper method to prepare the parameter encoding
     def _urlencode(self, d):
-        return urlencode(self._flatten_keys(d, '', {}))
+        return urlencode(self._flatten_keys(d, '', {}), doseq=True)
 
     # Flattens the hash keys, so page: { offset: 1 } becomes page[offet] = 1
     def _flatten_keys(self, d, key, out):

amadeus/reference_data/_locations.py

@@ -3,6 +3,7 @@
 from amadeus.reference_data.locations._points_of_interest import PointsOfInterest
 from amadeus.reference_data.locations._point_of_interest import PointOfInterest
 from amadeus.reference_data.locations._hotels import Hotels
+from amadeus.reference_data.locations._hotel import Hotel
 
 
 class Locations(Decorator, object):
@@ -11,6 +12,7 @@ def __init__(self, client):
         self.airports = Airports(client)
         self.points_of_interest = PointsOfInterest(client)
         self.hotels = Hotels(client)
+        self.hotel = Hotel(client)
 
     def point_of_interest(self, poi_id):
         return PointOfInterest(self.client, poi_id)

amadeus/reference_data/locations/__init__.py

@@ -1,3 +1,5 @@
 from ._airports import Airports
 from ._points_of_interest import PointsOfInterest
-__all__ = ['Airports', 'PointsOfInterest']
+from ._hotel import Hotel
+
+__all__ = ['Airports', 'PointsOfInterest', 'Hotel']

amadeus/reference_data/locations/_hotel.py

@@ -0,0 +1,26 @@
+from amadeus.client.decorator import Decorator
+
+
+class Hotel(Decorator, object):
+    def get(self, **params):
+        '''
+        Returns a list of hotels matching a given keyword.
+
+        .. code-block:: python
+
+
+            amadeus.reference_data.locations.hotel.get(
+                keyword='PARI',
+                subType=[Hotel.HOTEL_LEISURE, Hotel.HOTEL_GDS]
+            )
+
+        :param keyword: location query keyword.
+            For example: ``PARI``
+        :param subType: category of search.
+            For example: ``[Hotel.HOTEL_LEISURE, Hotel.HOTEL_GDS]``
+
+        :rtype: amadeus.Response
+        :raises amadeus.ResponseError: if the request could not be completed
+        '''
+        return self.client.get(
+            '/v1/reference-data/locations/hotel', **params)

docs/index.rst

@@ -179,6 +179,12 @@ ReferenceData/Locations/Hotels
 .. autoclass:: amadeus.reference_data.hotels.ByGeocode
   :members: get
 
+ReferenceData/Locations/Hotel
+=======================
+
+.. autoclass:: amadeus.reference_data.locations.Hotel
+  :members: get
+
 Helper/Location
 ==================
 

specs/namespaces/namespaces_spec.py

@@ -28,6 +28,7 @@
         expect(client.reference_data.locations.hotels.by_hotels).not_to(be_none)
         expect(client.reference_data.locations.hotels.by_city).not_to(be_none)
         expect(client.reference_data.locations.hotels.by_geocode).not_to(be_none)
+        expect(client.reference_data.locations.hotel).not_to(be_none)
         expect(client.travel).not_to(be_none)
         expect(client.travel.analytics).not_to(be_none)
         expect(client.travel.analytics.air_traffic.traveled).not_to(be_none)
@@ -114,6 +115,7 @@
             client.reference_data.locations.hotels.by_hotels.get).not_to(be_none)
         expect(
             client.reference_data.locations.hotels.by_geocode.get).not_to(be_none)
+        expect(client.reference_data.locations.hotel.get).not_to(be_none)
         expect(client.travel.analytics.air_traffic.traveled.get).not_to(be_none)
         expect(client.travel.analytics.air_traffic.booked.get).not_to(be_none)
         expect(
@@ -555,3 +557,9 @@
             expect(self.client.get).to(have_been_called_with(
                 '/v1/reference-data/locations/hotels/by-geocode', a='b'
             ))
+
+        with it('.reference_data.locations.hotel.get'):
+            self.client.reference_data.locations.hotel.get(a='b')
+            expect(self.client.get).to(have_been_called_with(
+                '/v1/reference-data/locations/hotel', a='b'
+            ))