Merge pull request #1412 from burnash/bugfix/client_init

Fixup gspread client init arguments
burnash · Feb 14, 2024 · 4267229 · 4267229
2 parents f0a8779 + 03a10a5
commit 4267229
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/gspread/auth.py b/gspread/auth.py
@@ -77,7 +77,7 @@ def authorize(
     :rtype: :class:`gspread.client.Client`
     """
 
-    return Client(auth=credentials, http_client=http_client, session=session)
+    return Client(auth=credentials, session=session, http_client=http_client)
 
 
 class FlowCallable(Protocol):

diff --git a/gspread/client.py b/gspread/client.py
@@ -38,8 +38,8 @@ class Client:
     def __init__(
         self,
         auth: Credentials,
-        http_client: HTTPClientType = HTTPClient,
         session: Optional[Session] = None,
+        http_client: HTTPClientType = HTTPClient,
     ) -> None:
         self.http_client = http_client(auth, session)
 

diff --git a/tests/conftest.py b/tests/conftest.py
@@ -87,7 +87,7 @@ def client():
     else:
         auth_credentials = DummyCredentials(DUMMY_ACCESS_TOKEN)
 
-    gc = Client(auth_credentials, BackOffHTTPClient)
+    gc = Client(auth=auth_credentials, http_client=BackOffHTTPClient)
     if not isinstance(gc, gspread.client.Client) is True:
         raise AssertionError