Add Support for Hotel Booking (#67)

* initial commit

* update impl and tests

* remove extra line from readme

* update readme and code samples

* update examples

* fix offer parameter

* remove pragma for coverage

* update readme

Author: tsolakoua

README.rst

@@ -266,7 +266,11 @@ List of supported endpoints
     # Get list of offers for a specific hotel
     amadeus.shopping.hotel_offers_by_hotel.get(hotelId = 'BGLONBGB')
     # Confirm the availability of a specific offer
-    amadeus.shopping.hotel_offer('8123DD9DE5102DADF5DA3B55C8C575F54114336EE718578753888747FE0652FC').get()
+    offerId = amadeus.shopping.hotel_offer('8123DD9DE5102DADF5DA3B55C8C575F54114336EE718578753888747FE0652FC').get()
+
+    # Hotel Booking
+    # The offerId comes from the hotel_offer above
+    amadeus.booking.hotel_bookings.post(offerId, guests, payments)
 
     # Hotel Ratings
     # What travelers think about this hotel?

amadeus/booking/__init__.py

@@ -1,4 +1,5 @@
 from ._flight_orders import FlightOrders
 from ._flight_order import FlightOrder
+from ._hotel_bookings import HotelBookings
 
-__all__ = ['FlightOrders', 'FlightOrder']
+__all__ = ['FlightOrders', 'FlightOrder', 'HotelBookings']

amadeus/booking/_hotel_bookings.py

@@ -0,0 +1,33 @@
+from amadeus.client.decorator import Decorator
+
+
+class HotelBookings(Decorator, object):
+    def post(self, hotel_offer_id, guests, payments):
+        '''
+        Books a hotel
+
+        .. code-block:: python
+
+            amadeus.booking.hotel_bookings.post(hotel_offer_id, guests, payments)
+
+        The parameters guests and payments can be passed as dictionary
+        or list of dictionaries. If they are dictionary in this method they are
+        converted to a list of dictionaries.
+
+        :rtype: amadeus.Response
+        :raises amadeus.ResponseError: if the request could not be completed
+        '''
+        guests_info = []
+        payment_info = []
+        if type(guests) is not list:
+            guests_info.append(guests)
+        else:
+            guests_info.extend(guests)
+        if type(payments) is not list:
+            payment_info.append(payments)
+        else:
+            payment_info.extend(payments)
+        body = {'data': {'offerId': hotel_offer_id,
+                         'guests': guests_info,
+                         'payments': payment_info}}
+        return self.client.post('/v1/booking/hotel-bookings', body)

amadeus/namespaces/_booking.py

@@ -1,15 +1,17 @@
 from amadeus.booking._flight_orders import FlightOrders
 from amadeus.booking._flight_order import FlightOrder
+from amadeus.booking._hotel_bookings import HotelBookings
 from amadeus.client.decorator import Decorator
 
 
 class Booking(Decorator, object):
     def __init__(self, client):
         Decorator.__init__(self, client)
         self.flight_orders = FlightOrders(client)
+        self.hotel_bookings = HotelBookings(client)
 
     def flight_order(self, flight_order_id):
         return FlightOrder(self.client, flight_order_id)
 
 
-__all__ = ['FlightOrders', 'FlightOrder']
+__all__ = ['FlightOrders', 'FlightOrder', 'HotelBookings']

docs/index.rst

@@ -146,4 +146,7 @@ Booking
   :members: post
 
 .. autoclass:: amadeus.booking.FlightOrder
-  :members: get
+  :members: get
+
+.. autoclass:: amadeus.booking.HotelBookings
+  :members: post

examples/hotel_search/hotel_search.py

@@ -7,12 +7,15 @@
     hotels_by_city = amadeus.shopping.hotel_offers.get(cityCode='LON')
     # print(hotels_by_city.data)
     # Get list of offers for a specific hotel
-    hotel_offers = amadeus.shopping.hotel_offers_by_hotel.get(hotelId = 'BGLONBGB')
+    hotel_offers = amadeus.shopping.hotel_offers_by_hotel.get(hotelId = 'HSMADAMI')
     # print(hotel_offers.data)
     # Confirm the availability of a specific offer
-    # print(hotel_offers.data)
-    offer_availability = amadeus.shopping.hotel_offer(hotel_offers.data['offers'][0]['id']).get()
+    offer = hotel_offers.data['offers'][0]['id']
+    offer_availability = amadeus.shopping.hotel_offer(offer).get()
     # print(offer_availability.data)
-
-except ResponseError as error:
-    raise error
+    guests = [{'id': 1, 'name': { 'title': 'MR', 'firstName': 'BOB', 'lastName': 'SMITH' }, 'contact': { 'phone': '+33679278416', 'email': '[email protected]'}}]
+    payments = { 'id': 1, 'method': 'creditCard', 'card': { 'vendorCode': 'VI', 'cardNumber': '4151289722471370', 'expiryDate': '2021-08' } }
+    hotel_booking = amadeus.booking.hotel_bookings.post(offer, guests, payments)
+    # print(hotel_booking.data)
+except ResponseError:
+    pass

specs/namespaces/namespaces_spec.py

@@ -327,3 +327,15 @@
             expect(self.client.get).to(have_been_called_with(
                 '/v1/booking/flight-orders/123', a='b'
             ))
+
+        with it('.shopping.booking.hotel_bookings.post'):
+            self.client.booking.hotel_bookings.post('123',
+                                                    {'foo': 'bar'},
+                                                    {'bar': 'foo'})
+            expect(self.client.post).to(have_been_called_with(
+                '/v1/booking/hotel-bookings',
+                {'data': {'offerId': '123',
+                          'guests': [{'foo': 'bar'}],
+                          'payments': [{'bar': 'foo'}]
+                          }}
+            ))