Merge pull request #149 from amadeus4dev/hotel-list-api

Add support for hotel list endpoints

Author: tsolakoua

README.rst

@@ -263,6 +263,14 @@ List of supported endpoints
     # Confirm the availability of a specific offer
     offerId = amadeus.shopping.hotel_offer('8123DD9DE5102DADF5DA3B55C8C575F54114336EE718578753888747FE0652FC').get()
 
+    # Hotel List
+    # Get list of hotels by hotel id
+    amadeus.reference_data.locations.hotels.by_hotels.get(hotelIds='ADPAR001')
+    # Get list of hotels by city code
+    amadeus.reference_data.locations.hotels.by_city.get(cityCode='PAR')
+    # Get list of hotels by a geocode
+    amadeus.reference_data.locations.hotels.by_geocode.get(longitude=2.160873,latitude=41.397158)
+
     # Hotel Booking
     # The offerId comes from the hotel_offer above
     amadeus.booking.hotel_bookings.post(offerId, guests, payments)

amadeus/reference_data/_locations.py

@@ -2,13 +2,15 @@
 from amadeus.reference_data.locations._airports import Airports
 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
 
 
 class Locations(Decorator, object):
     def __init__(self, client):
         Decorator.__init__(self, client)
         self.airports = Airports(client)
         self.points_of_interest = PointsOfInterest(client)
+        self.hotels = Hotels(client)
 
     def point_of_interest(self, poi_id):
         return PointOfInterest(self.client, poi_id)

amadeus/reference_data/locations/_hotels.py

@@ -0,0 +1,12 @@
+from amadeus.client.decorator import Decorator
+from amadeus.reference_data.locations.hotels import ByCity
+from amadeus.reference_data.locations.hotels import ByGeocode
+from amadeus.reference_data.locations.hotels import ByHotels
+
+
+class Hotels(Decorator, object):
+    def __init__(self, client):
+        Decorator.__init__(self, client)
+        self.by_hotels = ByHotels(client)
+        self.by_geocode = ByGeocode(client)
+        self.by_city = ByCity(client)

amadeus/reference_data/locations/hotels/__init__.py

@@ -0,0 +1,5 @@
+from ._by_city import ByCity
+from ._by_geocode import ByGeocode
+from ._by_hotels import ByHotels
+
+__all__ = ['ByCity', 'ByGeocode', 'ByHotels']

amadeus/reference_data/locations/hotels/_by_city.py

@@ -0,0 +1,22 @@
+from amadeus.client.decorator import Decorator
+
+
+class ByCity(Decorator, object):
+    def get(self, **params):
+        '''
+        Searches for hotel in a given city.
+
+        .. code-block:: python
+
+
+            amadeus.reference_data.locations.hotels.by_city.get(
+                cityCode='PAR')
+
+        :param cityCode: the City IATA code for which to find a hotel, for
+            example ``"PAR"`` for Paris.
+
+        :rtype: amadeus.Response
+        :raises amadeus.ResponseError: if the request could not be completed
+        '''
+        return self.client.get(
+            '/v1/reference-data/locations/hotels/by-city', **params)

amadeus/reference_data/locations/hotels/_by_geocode.py

@@ -0,0 +1,26 @@
+from amadeus.client.decorator import Decorator
+
+
+class ByGeocode(Decorator, object):
+    def get(self, **params):
+        '''
+        Searches for hotel using a geocode.
+
+        .. code-block:: python
+
+
+            amadeus.reference_data.locations.hotels.by_geocode.get(
+                longitude=2.160873,
+                latitude=41.397158
+            )
+
+        :param latitude: latitude of geographic location to search around.
+            For example: ``41.397158``
+        :param longitude: longitude of geographic location to search around.
+            For example: ``2.160873``
+
+        :rtype: amadeus.Response
+        :raises amadeus.ResponseError: if the request could not be completed
+        '''
+        return self.client.get(
+            '/v1/reference-data/locations/hotels/by-geocode', **params)

amadeus/reference_data/locations/hotels/_by_hotels.py

@@ -0,0 +1,22 @@
+from amadeus.client.decorator import Decorator
+
+
+class ByHotels(Decorator, object):
+    def get(self, **params):
+        '''
+        Searches for hotel using it's unique id.
+
+        .. code-block:: python
+
+
+            amadeus.reference_data.locations.hotels.by_hotels.get(
+                hotelIds=["ADPAR001"])
+
+        :param hotelIds: Amadeus Property Codes (8 chars)
+            For example: ``["ADPAR001"]``
+
+        :rtype: amadeus.Response
+        :raises amadeus.ResponseError: if the request could not be completed
+        '''
+        return self.client.get(
+            '/v1/reference-data/locations/hotels/by-hotels', **params)

docs/index.rst

@@ -161,6 +161,18 @@ ReferenceData/RecommendedLocations
 .. autoclass:: amadeus.reference_data.RecommendedLocations
   :members: get
 
+ReferenceData/Locations/Hotels
+=======================
+
+.. autoclass:: amadeus.reference_data.hotels.ByHotels
+  :members: get
+
+.. autoclass:: amadeus.reference_data.hotels.ByCity
+  :members: get
+
+.. autoclass:: amadeus.reference_data.hotels.ByGeocode
+  :members: get
+
 Helper/Location
 ==================
 

specs/namespaces/namespaces_spec.py

@@ -24,7 +24,10 @@
         expect(
             client.reference_data.locations.points_of_interest.by_square).not_to(
             be_none)
-        expect(client.reference_data.locations.point_of_interest).not_to(be_none)
+        expect(client.reference_data.locations.hotels).not_to(be_none)
+        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.travel).not_to(be_none)
         expect(client.travel.analytics).not_to(be_none)
         expect(client.travel.analytics.air_traffic.traveled).not_to(be_none)
@@ -105,7 +108,12 @@
         expect(client.reference_data.locations.point_of_interest(
             '9CB40CB5D0').get).not_to(be_none)
         expect(client.reference_data.recommended_locations.get).not_to(be_none)
-
+        expect(
+            client.reference_data.locations.hotels.by_city.get).not_to(be_none)
+        expect(
+            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.travel.analytics.air_traffic.traveled.get).not_to(be_none)
         expect(client.travel.analytics.air_traffic.booked.get).not_to(be_none)
         expect(
@@ -509,3 +517,24 @@
             expect(self.client.get).to(have_been_called_with(
                 '/v1/duty-of-care/diseases/covid19-area-report', a='b'
             ))
+
+        with it('.reference_data.locations.hotels.by_hotels.get'):
+            self.client.reference_data.locations.hotels.by_hotels.get(
+                a='b')
+            expect(self.client.get).to(have_been_called_with(
+                '/v1/reference-data/locations/hotels/by-hotels', a='b'
+            ))
+
+        with it('.reference_data.locations.hotels.by_city.get'):
+            self.client.reference_data.locations.hotels.by_city.get(
+                a='b')
+            expect(self.client.get).to(have_been_called_with(
+                '/v1/reference-data/locations/hotels/by-city', a='b'
+            ))
+
+        with it('.reference_data.locations.hotels.by_geocode.get'):
+            self.client.reference_data.locations.hotels.by_geocode.get(
+                a='b')
+            expect(self.client.get).to(have_been_called_with(
+                '/v1/reference-data/locations/hotels/by-geocode', a='b'
+            ))