Skip to content

Commit f692c62

Browse files
NO-SNOW: Fix tests
1 parent 3bbebf9 commit f692c62

File tree

10 files changed

+10
-1
lines changed

10 files changed

+10
-1
lines changed

setup.cfg

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,7 @@ development =
9696
pytest-xdist
9797
pytzdata
9898
responses
99+
pytest-asyncio
99100
pandas =
100101
pandas>=2.1.2,<3.0.0
101102
pyarrow

src/snowflake/connector/aio/_connection.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1019,6 +1019,7 @@ async def connect(self, **kwargs) -> None:
10191019
else:
10201020
self.__config(**self._conn_parameters)
10211021

1022+
self._crl_config: CRLConfig = CRLConfig.from_connection(self)
10221023
self._http_config: AioHttpConfig = AioHttpConfig(
10231024
connector_factory=SnowflakeSSLConnectorFactory(),
10241025
use_pooling=not self.disable_request_pooling,

src/snowflake/connector/aio/_session_manager.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ async def connect(
8686
)
8787
if (
8888
FEATURE_CRL_CONFIG.cert_revocation_check_mode
89-
!= CertRevocationCheckMode.ENABLED
89+
!= CertRevocationCheckMode.DISABLED
9090
):
9191
self.validate_crl(protocol, req)
9292
logger.debug(

src/snowflake/connector/aio/auth/_auth.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ async def authenticate(
9999
self._rest._connection._internal_application_name,
100100
self._rest._connection._internal_application_version,
101101
self._rest._connection._ocsp_mode(),
102+
self._rest._connection.cert_revocation_check_mode,
102103
self._rest._connection._login_timeout,
103104
self._rest._connection._network_timeout,
104105
self._rest._connection._socket_timeout,

src/snowflake/connector/aio/auth/_okta.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,7 @@ async def _step1(
121121
conn._internal_application_name,
122122
conn._internal_application_version,
123123
conn._ocsp_mode(),
124+
conn.cert_revocation_check_mode,
124125
conn.login_timeout,
125126
conn.network_timeout,
126127
conn.socket_timeout,

src/snowflake/connector/aio/auth/_webbrowser.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -377,6 +377,7 @@ async def _get_sso_url(
377377
conn._internal_application_name,
378378
conn._internal_application_version,
379379
conn._ocsp_mode(),
380+
conn.cert_revocation_check_mode,
380381
conn.login_timeout,
381382
conn.network_timeout,
382383
conn.socket_timeout,

test/unit/aio/test_auth_async.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ def _init_rest(application, post_requset):
2424
connection = mock_connection()
2525
connection.errorhandler = Mock(return_value=None)
2626
connection._ocsp_mode = Mock(return_value=OCSPMode.FAIL_OPEN)
27+
connection.cert_revocation_check_mode = "TEST_CRL_MODE"
2728
type(connection).application = PropertyMock(return_value=application)
2829
type(connection)._internal_application_name = PropertyMock(return_value=CLIENT_NAME)
2930
type(connection)._internal_application_version = PropertyMock(

test/unit/aio/test_auth_keypair_async.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -146,6 +146,7 @@ def _init_rest(application, post_requset):
146146
connection = mock_connection()
147147
connection.errorhandler = Mock(return_value=None)
148148
connection._ocsp_mode = Mock(return_value=OCSPMode.FAIL_OPEN)
149+
connection.cert_revocation_check_mode = "TEST_CRL_MODE"
149150
type(connection).application = PropertyMock(return_value=application)
150151
type(connection)._internal_application_name = PropertyMock(return_value=CLIENT_NAME)
151152
type(connection)._internal_application_version = PropertyMock(

test/unit/aio/test_auth_okta_async.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -342,6 +342,7 @@ async def post_request(url, headers, body, **kwargs):
342342
connection = mock_connection(disable_saml_url_check=disable_saml_url_check)
343343
connection.errorhandler = Mock(return_value=None)
344344
connection._ocsp_mode = Mock(return_value=OCSPMode.FAIL_OPEN)
345+
connection.cert_revocation_check_mode = "TEST_CRL_MODE"
345346
type(connection).application = PropertyMock(return_value=CLIENT_NAME)
346347
type(connection)._internal_application_name = PropertyMock(return_value=CLIENT_NAME)
347348
type(connection)._internal_application_version = PropertyMock(

test/unit/aio/test_auth_webbrowser_async.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -413,6 +413,7 @@ async def post_request(url, headers, body, **kwargs):
413413
connection = mock_connection(socket_timeout=socket_timeout)
414414
connection.errorhandler = Mock(return_value=None)
415415
connection._ocsp_mode = Mock(return_value=OCSPMode.FAIL_OPEN)
416+
connection.cert_revocation_check_mode = "TEST_CRL_MODE"
416417
connection._disable_console_login = disable_console_login
417418
type(connection).application = PropertyMock(return_value=CLIENT_NAME)
418419
type(connection)._internal_application_name = PropertyMock(return_value=CLIENT_NAME)

0 commit comments

Comments
 (0)