Skip to content

Commit 99bf9b2

Browse files
adjust MFA test to Windows - use keyring mocks too
1 parent dd2e6fb commit 99bf9b2

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

test/integ/sso/test_unit_mfa_cache.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,15 +8,16 @@
88
import pytest
99

1010
import snowflake.connector
11-
from snowflake.connector.compat import IS_LINUX
1211
from snowflake.connector.errors import DatabaseError
1312

1413
try:
15-
from snowflake.connector.compat import IS_MACOS
14+
from snowflake.connector.compat import IS_LINUX, IS_MACOS, IS_WINDOWS
1615
except ImportError:
1716
import platform
1817

1918
IS_MACOS = platform.system() == "Darwin"
19+
IS_LINUX = platform.system() == "Linux"
20+
IS_WINDOWS = platform.system() == "Windows"
2021

2122

2223
# Although this is an unit test, we put it under test/integ/sso, since it needs keyring package installed
@@ -162,7 +163,7 @@ def test_body(conn_cfg):
162163
if IS_LINUX:
163164
conn_cfg["client_request_mfa_token"] = True
164165

165-
if IS_MACOS:
166+
if IS_MACOS or IS_WINDOWS:
166167
with patch(
167168
"keyring.delete_password", Mock(side_effect=mock_del_password)
168169
), patch("keyring.set_password", Mock(side_effect=mock_set_password)), patch(

0 commit comments

Comments
 (0)