recover to not bind port

Author: sfc-gh-pcyrek

test/conftest.py

@@ -2,7 +2,6 @@
 from __future__ import annotations
 
 import os
-import socket
 from contextlib import contextmanager
 from logging import getLogger
 from pathlib import Path
@@ -151,15 +150,6 @@ def pytest_runtest_setup(item) -> None:
             pytest.skip("Skipping auth test in current environment")
 
 
-def find_free_port() -> int:
-    """Find a free port to avoid conflicts in parallel test environments."""
-    with socket.socket(socket.AF_INET, socket.SOCK_STREAM) as s:
-        s.bind(("127.0.0.1", 0))
-        s.listen(1)
-        port = s.getsockname()[1]
-    return port
-
-
 def get_server_parameter_value(connection, parameter_name: str) -> str | None:
     """Get server parameter value, returns None if parameter doesn't exist."""
     try:

test/unit/test_oauth_token.py

@@ -367,7 +367,7 @@ def test_oauth_code_custom_urls(
 
 
 @pytest.mark.skipolddriver
-@patch("snowflake.connector.auth._http_server.AuthHttpServer.DEFAULT_TIMEOUT", 120)
+@patch("snowflake.connector.auth._http_server.AuthHttpServer.DEFAULT_TIMEOUT", 30)
 def test_oauth_code_local_application_custom_urls_successful_flow(
     wiremock_client: WiremockClient,
     wiremock_oauth_authorization_code_dir,
@@ -376,8 +376,6 @@ def test_oauth_code_local_application_custom_urls_successful_flow(
     monkeypatch,
     omit_oauth_urls_check,
 ) -> None:
-    from ..conftest import find_free_port
-
     monkeypatch.setenv("SNOWFLAKE_AUTH_SOCKET_REUSE_PORT", "true")
 
     wiremock_client.import_mapping(
@@ -391,10 +389,6 @@ def test_oauth_code_local_application_custom_urls_successful_flow(
         wiremock_generic_mappings_dir / "snowflake_disconnect_successful.json"
     )
 
-    # Use dynamic port to avoid conflicts and higher timeout for all environments
-    free_port = find_free_port()
-    oauth_redirect_uri = f"http://localhost:{free_port}/snowflake/oauth-redirect"
-
     with mock.patch("webbrowser.open", new=webbrowser_mock.open):
         with mock.patch("secrets.token_urlsafe", return_value="abc123"):
             cnx = snowflake.connector.connect(
@@ -407,10 +401,9 @@ def test_oauth_code_local_application_custom_urls_successful_flow(
                 role="ANALYST",
                 oauth_token_request_url=f"http://{wiremock_client.wiremock_host}:{wiremock_client.wiremock_http_port}/tokenrequest",
                 oauth_authorization_url=f"http://{wiremock_client.wiremock_host}:{wiremock_client.wiremock_http_port}/authorization",
-                oauth_redirect_uri=oauth_redirect_uri,
+                oauth_redirect_uri="http://localhost:8009/snowflake/oauth-redirect",
                 host=wiremock_client.wiremock_host,
                 port=wiremock_client.wiremock_http_port,
-                external_browser_timeout=120,  # Higher timeout for all environments
             )
 
             assert cnx, "invalid cnx"