Merge pull request #182 from amadeus4dev/hotelv2-remove

Remove Hotel Search API v2

Author: minjikarin

README.rst

@@ -257,14 +257,6 @@ List of supported endpoints
 
     # Flight Busiest Travel Period
     amadeus.travel.analytics.air_traffic.busiest_period.get(cityCode='MAD', period='2017', direction='ARRIVING')
-    
-    # Hotel Search
-    # Get list of Hotels by city code
-    amadeus.shopping.hotel_offers.get(cityCode='LON')
-    # Get list of offers for a specific hotel
-    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/namespaces/_shopping.py

@@ -3,9 +3,6 @@
 from amadeus.shopping._flight_destinations import FlightDestinations
 from amadeus.shopping._flight_offers import FlightOffers
 from amadeus.shopping._flight_offers_search import FlightOffersSearch
-from amadeus.shopping._hotel_offers import HotelOffers
-from amadeus.shopping._hotel_offers_by_hotel import HotelOffersByHotel
-from amadeus.shopping._hotel_offer import HotelOffer
 from amadeus.shopping._seatmaps import Seatmaps
 from amadeus.shopping._activities import Activities
 from amadeus.shopping._activity import Activity
@@ -20,17 +17,12 @@ def __init__(self, client):
         self.flight_dates = FlightDates(client)
         self.flight_destinations = FlightDestinations(client)
         self.flight_offers = FlightOffers(client)
-        self.hotel_offers = HotelOffers(client)
-        self.hotel_offers_by_hotel = HotelOffersByHotel(client)
         self.flight_offers_search = FlightOffersSearch(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)
 

amadeus/shopping/__init__.py

@@ -1,13 +1,10 @@
-from ._flight_offers import FlightOffers
 from ._flight_dates import FlightDates
 from ._flight_destinations import FlightDestinations
 from ._flight_offers_search import FlightOffersSearch
-from ._hotel_offers import HotelOffers
-from ._hotel_offers_by_hotel import HotelOffersByHotel
-from ._hotel_offer import HotelOffer
+from ._hotel_offer_search import HotelOfferSearch
+from ._hotel_offers_search import HotelOffersSearch
 from ._activities import Activities
 
-__all__ = ['FlightOffers', 'FlightDates',
-           'FlightDestinations', 'FlightOffersSearch',
-           'HotelOffers', 'HotelOffersByHotel',
-           'HotelOffer', 'Activities']
+__all__ = ['FlightDates', 'FlightDestinations',
+           'HotelOffersSearch', 'HotelOfferSearch',
+           'FlightOffersSearch', 'Activities']

amadeus/shopping/_hotel_offer.py

@@ -53,15 +53,6 @@ Shopping/Flights
 Shopping/Hotels
 ===============
 
-.. autoclass:: amadeus.shopping.HotelOffers
-  :members: get
-
-.. autoclass:: amadeus.shopping.hotel.HotelOffers
-  :members: get
-
-.. autoclass:: amadeus.shopping.hotel.Offer
-  :members: get
-
 .. autoclass:: amadeus.shopping.hotel.HotelOffersSearch
   :members: get
 

amadeus/shopping/_hotel_offers.py

@@ -40,9 +40,6 @@ def test_expected_paths(client):
     assert client.shopping.flight_offers.pricing is not None
     assert client.shopping.flight_offers.upselling is not None
     assert client.shopping.seatmaps is not None
-    assert client.shopping.hotel_offers is not None
-    assert client.shopping.hotel_offer is not None
-    assert client.shopping.hotel_offers_by_hotel is not None
     assert client.shopping.hotel_offers_search is not None
     assert client.shopping.hotel_offer_search is not None
     assert client.shopping.activities is not None
@@ -97,9 +94,6 @@ def test_expected_get_methods(client):
     assert client.shopping.flight_destinations.get is not None
     assert client.shopping.flight_offers_search.get is not None
     assert client.shopping.seatmaps.get is not None
-    assert client.shopping.hotel_offers.get is not None
-    assert client.shopping.hotel_offers_by_hotel.get is not None
-    assert client.shopping.hotel_offer('123').get is not None
     assert client.shopping.hotel_offers_search.get is not None
     assert client.shopping.hotel_offer_search('123').get is not None
     assert client.e_reputation.hotel_sentiments.get is not None
@@ -259,27 +253,6 @@ def test_shopping_flight_offers_search_get(client_setup):
     )
 
 
-def test_shopping_hotel_offers_get(client_setup):
-    client_setup.shopping.hotel_offers.get(cityCode='MAD')
-    client_setup.get.assert_called_with(
-        '/v2/shopping/hotel-offers', cityCode='MAD'
-    )
-
-
-def test_shopping_hotel_offers_by_hotel_get(client_setup):
-    client_setup.shopping.hotel_offers_by_hotel.get(hotelId='XKPARC12')
-    client_setup.get.assert_called_with(
-        '/v2/shopping/hotel-offers/by-hotel', hotelId='XKPARC12'
-    )
-
-
-def test_shopping_hotel_offer_get(client_setup):
-    client_setup.shopping.hotel_offer('XXX').get(a='b')
-    client_setup.get.assert_called_with(
-        '/v2/shopping/hotel-offers/XXX', a='b'
-    )
-
-
 def test_shopping_hotel_offers_search_get(client_setup):
     client_setup.shopping.hotel_offers_search.get(
         hotelIds='RTPAR001', adults=2)