Merge pull request #191 from amadeus4dev/updates-transfer-apis

Rename transfer offers method

README.rst

@@ -350,14 +350,14 @@ List of supported endpoints
     # Airline Routes
     amadeus.airline.destinations.get(airlineCode='BA')
 
-    # Transfer Offers Search
-    amadeus.shopping.transfer_offers_search.post(body)
+    # Transfer Search
+    amadeus.shopping.transfer_offers.post(body)
 
-    # Transfer Create Order
+    # Transfer Booking
     amadeus.ordering.transfer_orders.post(body, offerId='1000000000')
 
-    # Transfer Order Management
-    amadeus.ordering.transfer_order('ABC').transfers.cancellation.post(body, confirmNbr=123)
+    # Transfer Management
+    amadeus.ordering.transfer_orders('ABC').transfers.cancellation.post(body, confirmNbr=123)
 
 Development & Contributing
 --------------------------

amadeus/namespaces/_shopping.py

@@ -9,7 +9,7 @@
 from amadeus.shopping._availability import Availability
 from amadeus.shopping._hotel_offer_search import HotelOfferSearch
 from amadeus.shopping._hotel_offers_search import HotelOffersSearch
-from amadeus.shopping._transfer_offers_search import TransferOffersSearch
+from amadeus.shopping._transfer_offers import TransferOffers
 
 
 class Shopping(Decorator, object):
@@ -23,7 +23,7 @@ def __init__(self, client):
         self.activities = Activities(client)
         self.availability = Availability(client)
         self.hotel_offers_search = HotelOffersSearch(client)
-        self.transfer_offers_search = TransferOffersSearch(client)
+        self.transfer_offers = TransferOffers(client)
 
     def hotel_offer_search(self, offer_id):
         return HotelOfferSearch(self.client, offer_id)

amadeus/shopping/__init__.py

@@ -4,9 +4,9 @@
 from ._hotel_offer_search import HotelOfferSearch
 from ._hotel_offers_search import HotelOffersSearch
 from ._activities import Activities
-from ._transfer_offers_search import TransferOffersSearch
+from ._transfer_offers import TransferOffers
 
 __all__ = ['FlightDates', 'FlightDestinations',
            'HotelOffersSearch', 'HotelOfferSearch',
            'FlightOffersSearch', 'Activities',
-           'TransferOffersSearch']
+           'TransferOffers']

amadeus/shopping/_transfer_offers_search.py → amadeus/shopping/_transfer_offers.py

@@ -1,14 +1,14 @@
 from amadeus.client.decorator import Decorator
 
 
-class TransferOffersSearch(Decorator, object):
+class TransferOffers(Decorator, object):
     def post(self, body):
         '''
         Get transfer offers
 
         .. code-block:: python
 
-            amadeus.shopping.transfer_offers_search.post(body)
+            amadeus.shopping.transfer_offers.post(body)
 
         :param body: the parameters to send to the API
 

docs/index.rst

@@ -92,7 +92,7 @@ Shopping/Availability
 Shopping/Transfers
 ================
 
-.. autoclass:: amadeus.shopping.TransferOffersSearch
+.. autoclass:: amadeus.shopping.TransferOffers
   :members: post
 
 Travel/Analytics

specs/namespaces/test_namespaces.py

@@ -67,7 +67,7 @@ def test_expected_paths(client):
     assert client.duty_of_care is not None
     assert client.duty_of_care.diseases.covid19_report is not None
     assert client.airline.destinations is not None
-    assert client.shopping.transfer_offers_search is not None
+    assert client.shopping.transfer_offers is not None
     assert client.ordering.transfer_orders is not None
     assert client.ordering.transfer_order is not None
 
@@ -548,8 +548,8 @@ def test_airline_destinations_get(client_setup):
     )
 
 
-def test_shopping_transfer_offers_search_post(client_setup):
-    client_setup.shopping.transfer_offers_search.post({'foo': 'bar'})
+def test_shopping_transfer_offers_post(client_setup):
+    client_setup.shopping.transfer_offers.post({'foo': 'bar'})
     client_setup.post.assert_called_with(
         '/v1/shopping/transfer-offers', {'foo': 'bar'}
     )