SNOW-2205633: Migrating to new okta (#2407)

Author: sfc-gh-pcyrek

.github/workflows/parameters/private/parameters_aws_auth_tests.json.gpg

@@ -23,5 +23,5 @@ docker run \
   -v $(cd $THIS_DIR/.. && pwd):/mnt/host \
   -v $WORKSPACE:/mnt/workspace \
   --rm \
-  nexus.int.snowflakecomputing.com:8086/docker/snowdrivers-test-external-browser-python:1 \
+    nexus.int.snowflakecomputing.com:8086/docker/snowdrivers-test-external-browser-python:2 \
   "/mnt/host/ci/container/test_authentication.sh"

ci/test_authentication.sh

@@ -74,28 +74,6 @@ def test_snowflake_authorization_code_timeout():
     )
 
 
-@pytest.mark.auth
-def test_snowflake_authorization_code_with_token_cache():
-    connection_parameters = (
-        AuthConnectionParameters().get_snowflake_authorization_code_connection_parameters()
-    )
-    connection_parameters["external_browser_timeout"] = 15
-    connection_parameters["client_store_temporary_credential"] = True
-    test_helper = AuthorizationTestHelper(connection_parameters)
-    browser_login, browser_password = get_soteria_okta_login_credentials().values()
-
-    test_helper.connect_and_provide_credentials(
-        Scenario.INTERNAL_OAUTH_SNOWFLAKE_SUCCESS, browser_login, browser_password
-    )
-
-    clean_browser_processes()
-
-    assert (
-        test_helper.connect_and_execute_simple_query() is True
-    ), "Connection should be established"
-    assert test_helper.get_error_msg() == "", "Error message should be empty"
-
-
 @pytest.mark.auth
 def test_snowflake_authorization_code_without_token_cache():
     connection_parameters = (

test/auth/test_snowflake_authorization_code.py

@@ -23,9 +23,6 @@ def setup_and_teardown():
     clean_browser_processes()
 
 
-@pytest.mark.skip(
-    "temporarily disabled, update redirect uri for the security integration will break other drivers tests"
-)
 @pytest.mark.auth
 def test_snowflake_authorization_code_wildcards_successful():
     connection_parameters = (
@@ -41,9 +38,6 @@ def test_snowflake_authorization_code_wildcards_successful():
     assert test_helper.error_msg == "", "Error message should be empty"
 
 
-@pytest.mark.skip(
-    "temporarily disabled, update redirect uri for the security integration will break other drivers tests"
-)
 @pytest.mark.auth
 def test_snowflake_authorization_code_wildcards_mismatched_user():
     connection_parameters = (