testing refresh_authentication method is able to update api_params

docs/source/adapter_class.rst

@@ -104,4 +104,4 @@ Given an exception, should returne if the authentication is expired. If so, tapi
 
 .. method:: refresh_authentication(self, api_params, *args, **kwargs): 
 
-Should run refresh authentication logic.
+Should run refresh authentication logic. Make sure you update `api_params` dictionary with the new token.

tests/client.py

@@ -61,7 +61,7 @@ def is_authentication_expired(self, exception, *args, **kwargs):
         return exception.status_code == 401
 
     def refresh_authentication(self, api_params, *args, **kwargs):
-        pass
+        api_params['token'] = 'new_token'
 
 
 TokenRefreshClient = generate_wrapper_from_adapter(TokenRefreshClientAdapter)

tests/test_tapioca.py

@@ -450,7 +450,7 @@ def test_simple_pages_max_item_zero_iterator(self):
 class TestTokenRefreshing(unittest.TestCase):
 
     def setUp(self):
-        self.wrapper = TokenRefreshClient()
+        self.wrapper = TokenRefreshClient(token='token')
 
     @responses.activate
     def test_not_token_refresh_ready_client_call_raises_not_implemented(self):
@@ -474,21 +474,25 @@ def test_token_expired_and_no_refresh_flag(self):
         with self.assertRaises(ClientError) as context:
             response = self.wrapper.test().post()
 
-    def request_callback(self, request):
-      if self.first_call:
-        self.first_call = False
-        return (401, {'content_type':'application/json'}, json.dumps('{"error": "Token expired"}'))
-      else:
-        self.first_call = None
-        return (201, {'content_type':'application/json'}, json.dumps('{"error": "Token expired"}'))
-
     @responses.activate
     def test_token_expired_with_active_refresh_flag(self):
         self.first_call = True
+
+        def request_callback(request):
+            if self.first_call:
+                self.first_call = False
+                return (401, {'content_type':'application/json'}, json.dumps('{"error": "Token expired"}'))
+            else:
+                self.first_call = None
+                return (201, {'content_type':'application/json'}, '')
+
         responses.add_callback(
             responses.POST, self.wrapper.test().data,
-            callback=self.request_callback,
+            callback=request_callback,
             content_type='application/json',
         )
 
         response = self.wrapper.test().post(refresh_auth=True)
+
+        # refresh_authentication method should be able to update api_params
+        self.assertEqual(response._api_params['token'], 'new_token')