Skip to content

Commit 4f590bf

Browse files
committed
Fix a bug
Signed-off-by: Jacky Hu <[email protected]>
1 parent 9707027 commit 4f590bf

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/databricks/sql/auth/auth.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@ def get_auth_provider(cfg: ClientContext):
7575
cfg.oauth_redirect_port_range,
7676
cfg.oauth_client_id,
7777
cfg.oauth_scopes,
78-
cfg.auth_type,
7978
)
8079
else:
8180
raise RuntimeError("No valid authentication settings!")

tests/unit/test_auth.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
ExternalAuthProvider,
1010
AuthType,
1111
)
12-
from databricks.sql.auth.auth import get_python_sql_connector_auth_provider
12+
from databricks.sql.auth.auth import get_python_sql_connector_auth_provider, PYSQL_OAUTH_CLIENT_ID
1313
from databricks.sql.auth.oauth import OAuthManager
1414
from databricks.sql.auth.authenticators import DatabricksOAuthProvider
1515
from databricks.sql.auth.endpoint import (
@@ -184,4 +184,5 @@ def test_get_python_sql_connector_default_auth(self, mock__initial_get_token):
184184
hostname = "foo.cloud.databricks.com"
185185
auth_provider = get_python_sql_connector_auth_provider(hostname)
186186
self.assertTrue(type(auth_provider).__name__, "DatabricksOAuthProvider")
187+
self.assertTrue(auth_provider._client_id,PYSQL_OAUTH_CLIENT_ID)
187188

0 commit comments

Comments
 (0)