Merge pull request #150 from amadeus4dev/hotel-search-v3

Add hotel search v3

Author: minjikarin

README.rst

@@ -257,12 +257,18 @@ List of supported endpoints
     
     # Hotel Search
     # Get list of Hotels by city code
-    amadeus.shopping.hotel_offers.get(cityCode = 'LON')
+    amadeus.shopping.hotel_offers.get(cityCode='LON')
     # Get list of offers for a specific hotel
-    amadeus.shopping.hotel_offers_by_hotel.get(hotelId = 'BGLONBGB')
+    amadeus.shopping.hotel_offers_by_hotel.get(hotelId='BGLONBGB')
     # Confirm the availability of a specific offer
     offerId = amadeus.shopping.hotel_offer('8123DD9DE5102DADF5DA3B55C8C575F54114336EE718578753888747FE0652FC').get()
 
+    # Hotel Search v3
+    # Get list of available offers by hotel ids
+    amadeus.shopping.hotel_offers_search.get(hotelIds='RTPAR001', adults='2')
+    # Check conditions of a specific offer
+    amadeus.shopping.hotel_offer_search('XXX').get()
+
     # Hotel List
     # Get list of hotels by hotel id
     amadeus.reference_data.locations.hotels.by_hotels.get(hotelIds='ADPAR001')

amadeus/namespaces/_shopping.py

@@ -10,6 +10,8 @@
 from amadeus.shopping._activities import Activities
 from amadeus.shopping._activity import Activity
 from amadeus.shopping._availability import Availability
+from amadeus.shopping._hotel_offer_search import HotelOfferSearch
+from amadeus.shopping._hotel_offers_search import HotelOffersSearch
 
 
 class Shopping(Decorator, object):
@@ -24,10 +26,14 @@ def __init__(self, client):
         self.seatmaps = Seatmaps(client)
         self.activities = Activities(client)
         self.availability = Availability(client)
+        self.hotel_offers_search = HotelOffersSearch(client)
 
     def hotel_offer(self, offer_id):
         return HotelOffer(self.client, offer_id)
 
+    def hotel_offer_search(self, offer_id):
+        return HotelOfferSearch(self.client, offer_id)
+
     def activity(self, activity_id):
         return Activity(self.client, activity_id)
 

amadeus/shopping/_hotel_offer_search.py

@@ -0,0 +1,21 @@
+from amadeus.client.decorator import Decorator
+
+
+class HotelOfferSearch(Decorator, object):
+    def __init__(self, client, offer_id):
+        Decorator.__init__(self, client)
+        self.offer_id = offer_id
+
+    def get(self, **params):
+        '''
+        Returns details for a specific offer
+
+        .. code-block:: python
+
+            amadeus.shopping.hotel_offer_search('XXX').get
+
+        :rtype: amadeus.Response
+        :raises amadeus.ResponseError: if the request could not be completed
+        '''
+        return self.client.get('/v3/shopping/hotel-offers/{0}'
+                               .format(self.offer_id), **params)

amadeus/shopping/_hotel_offers_search.py

@@ -0,0 +1,22 @@
+from amadeus.client.decorator import Decorator
+
+
+class HotelOffersSearch(Decorator, object):
+    def get(self, **params):
+        '''
+        Get all offers given hotels
+
+        .. code-block:: python
+
+           amadeus.shopping.hotel_offers_search.get(hotelIds=RTPAR001',
+            adults='2')
+
+        :param hotelId: Amadeus Property Code (8 chars), for
+            example ``RTPAR001``.
+
+        :param adults: the number of adult passengers with age 12 or older
+
+        :rtype: amadeus.Response
+        :raises amadeus.ResponseError: if the request could not be completed
+        '''
+        return self.client.get('/v3/shopping/hotel-offers', **params)

docs/index.rst

@@ -62,6 +62,12 @@ Shopping/Hotels
 .. autoclass:: amadeus.shopping.hotel.Offer
   :members: get
 
+.. autoclass:: amadeus.shopping.hotel.HotelOffersSearch
+  :members: get
+
+.. autoclass:: amadeus.shopping.hotel.HotelOfferSearch
+  :members: get
+
 Shopping/FlightOffers
 ===============
 

specs/namespaces/namespaces_spec.py

@@ -52,6 +52,9 @@
         expect(client.shopping.hotel_offer).not_to(be_none)
         expect(client.shopping.hotel_offers_by_hotel).not_to(be_none)
 
+        expect(client.shopping.hotel_offers_search).not_to(be_none)
+        expect(client.shopping.hotel_offer_search).not_to(be_none)
+
         expect(client.shopping.activities).not_to(be_none)
 
         expect(client.shopping.availability).not_to(be_none)
@@ -133,6 +136,9 @@
         expect(client.shopping.hotel_offers_by_hotel.get).not_to(be_none)
         expect(client.shopping.hotel_offer('123').get).not_to(be_none)
 
+        expect(client.shopping.hotel_offers_search.get).not_to(be_none)
+        expect(client.shopping.hotel_offer_search('123').get).not_to(be_none)
+
         expect(client.e_reputation.hotel_sentiments.get).not_to(be_none)
 
         expect(client.airport.predictions.on_time.get).not_to(be_none)
@@ -299,6 +305,20 @@
                 '/v2/shopping/hotel-offers/XXX', a='b'
             ))
 
+        with it('.shopping.hotel_offers_search.get'):
+            self.client.shopping.hotel_offers_search.get(
+                hotelIds='RTPAR001', adults=2)
+            expect(self.client.get).to(have_been_called_with(
+                '/v3/shopping/hotel-offers', hotelIds='RTPAR001',
+                adults=2
+            ))
+
+        with it('.shopping.hotel_offer_search().get'):
+            self.client.shopping.hotel_offer_search('XXX').get(a='b')
+            expect(self.client.get).to(have_been_called_with(
+                '/v3/shopping/hotel-offers/XXX', a='b'
+            ))
+
         with it('.shopping.seatmaps.get'):
             self.client.shopping.seatmaps.get(**{'a': 'b'})
             expect(self.client.get).to(have_been_called_with(