@almet review

Kinto · Feb 5, 2016 · 8c402c1 · 8c402c1
1 parent ef41e40
commit 8c402c1
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 9 deletions.
diff --git a/kinto_client/batch.py b/kinto_client/batch.py
@@ -51,14 +51,15 @@ def send(self):
         result = []
         requests = self._build_requests()
         for chunk in utils.chunks(requests, self.batch_max_requests):
-            args = ('POST', self.endpoints.get('batch'))
-            kwargs = dict(payload={'requests': chunk})
+            kwargs = dict(method='POST',
+                          endpoint=self.endpoints.get('batch'),
+                          payload={'requests': chunk})
 
             retry = self.nb_retry
             while retry >= 0:
                 headers = {}
                 try:
-                    resp, headers = self.session.request(*args, **kwargs)
+                    resp, headers = self.session.request(**kwargs)
                 except KintoException as e:
                     if retry <= 0:
                         raise e

diff --git a/kinto_client/tests/test_batch.py b/kinto_client/tests/test_batch.py
@@ -24,8 +24,8 @@ def test_send_adds_data_attribute(self):
         batch.send()
 
         self.client.session.request.assert_called_with(
-            'POST',
-            self.client.endpoints.get('batch'),
+            method='POST',
+            endpoint=self.client.endpoints.get('batch'),
             payload={'requests': [{
                 'method': 'GET',
                 'path': '/foobar/baz',
@@ -40,8 +40,8 @@ def test_send_adds_permissions_attribute(self):
         batch.send()
 
         self.client.session.request.assert_called_with(
-            'POST',
-            self.client.endpoints.get('batch'),
+            method='POST',
+            endpoint=self.client.endpoints.get('batch'),
             payload={'requests': [{
                 'method': 'GET',
                 'path': '/foobar/baz',
@@ -55,8 +55,8 @@ def test_send_adds_headers_if_specified(self):
         batch.send()
 
         self.client.session.request.assert_called_with(
-            'POST',
-            self.client.endpoints.get('batch'),
+            method='POST',
+            endpoint=self.client.endpoints.get('batch'),
             payload={'requests': [{
                 'method': 'GET',
                 'path': '/foobar/baz',