Merge pull request #79 from amadeus4dev/update-tests-createorders-pricing-hotelbooking

Update tests and remove unused code from Flight Offers Pricing

Author: tsolakoua

amadeus/shopping/flight_offers/_pricing.py

@@ -17,16 +17,12 @@ def post(self, body, **params):
         :rtype: amadeus.Response
         :raises amadeus.ResponseError: if the request could not be completed
         '''
-        url = '/v1/shopping/flight-offers/pricing'
+        url = '/v1/shopping/flight-offers/pricing?'
         flight_offers = []
-
-        if params is not None:
-            url = '/v1/shopping/flight-offers/pricing?'
         if type(body) is not list:
             flight_offers.append(body)
         else:
             flight_offers.extend(body)
-
         return self.client.post(url + urlencode(params),
                                 {'data':
                                 {'type': 'flight-offers-pricing',

specs/namespaces/namespaces_spec.py

@@ -284,6 +284,23 @@
                 '/v2/travel/trip-parser-jobs', {'foo': 'bar'}
             ))
 
+        with it('.travel.trip_parser_jobs.post_from_base64'):
+            self.client.travel.trip_parser_jobs.post(
+                self.client.travel.from_base64('dGVzdA=='))
+            expect(self.client.post).to(have_been_called_with(
+                '/v2/travel/trip-parser-jobs',
+                {'data': {'type': 'trip-parser-job', 'content': 'dGVzdA=='}}
+            ))
+
+        with it('.travel.trip_parser_jobs.post_from_file'):
+            file = 'specs/namespaces/trip_parser_test.eml'
+            self.client.travel.trip_parser_jobs.post(
+                self.client.travel.from_file(file))
+            expect(self.client.post).to(have_been_called_with(
+                '/v2/travel/trip-parser-jobs',
+                {'data': {'type': 'trip-parser-job', 'content': 'Qm9va2luZwo='}}
+            ))
+
         with it('.shopping.flight_offers_search.post'):
             self.client.shopping.flight_offers_search.post({'foo': 'bar'})
             expect(self.client.post).to(have_been_called_with(
@@ -297,7 +314,16 @@
             ))
 
         with it('.shopping.flight_offers.pricing.post'):
-            self.client.shopping.flight_offers.pricing.post({'foo': 'bar'})
+            self.client.shopping.flight_offers.pricing.post(
+                {'foo': 'bar'}, include='other-services')
+            expect(self.client.post).to(have_been_called_with(
+                '/v1/shopping/flight-offers/pricing?'+'include=other-services',
+                {'data': {'type': 'flight-offers-pricing',
+                          'flightOffers': [{'foo': 'bar'}]}}
+            ))
+
+        with it('.shopping.flight_offers.pricing.post_list'):
+            self.client.shopping.flight_offers.pricing.post([{'foo': 'bar'}])
             expect(self.client.post).to(have_been_called_with(
                 '/v1/shopping/flight-offers/pricing?',
                 {'data': {'type': 'flight-offers-pricing',
@@ -314,6 +340,17 @@
                           }}
             ))
 
+        with it('.shopping.booking.flight_orders.post_list'):
+            self.client.booking.flight_orders.post(
+                [{'foo': 'bar'}], [{'bar': 'foo'}])
+            expect(self.client.post).to(have_been_called_with(
+                '/v1/booking/flight-orders',
+                {'data': {'type': 'flight-order',
+                          'flightOffers': [{'foo': 'bar'}],
+                          'travelers': [{'bar': 'foo'}]
+                          }}
+            ))
+
         with it('.booking.flight_order().get'):
             self.client.booking.flight_order('123').get(a='b')
             expect(self.client.get).to(have_been_called_with(
@@ -331,3 +368,14 @@
                           'payments': [{'bar': 'foo'}]
                           }}
             ))
+        with it('.shopping.booking.hotel_bookings.post_list'):
+            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'}]
+                          }}
+            ))

specs/namespaces/trip_parser_test.eml

@@ -0,0 +1 @@
+Booking