NO-SNOW: Simplify callable instance into function

Author: sfc-gh-fpawlowski

src/snowflake/connector/aio/__init__.py

@@ -1,14 +1,7 @@
 from __future__ import annotations
 
 from functools import wraps
-from typing import (
-    Any,
-    Coroutine,
-    Generator,
-    Protocol,
-    TypeVar,
-    runtime_checkable,
-)
+from typing import Any, Coroutine, Generator, Protocol, TypeVar, runtime_checkable
 
 from ._connection import SnowflakeConnection
 from ._cursor import DictCursor, SnowflakeCursor
@@ -98,8 +91,6 @@ async def __aexit__(
         ...
 
 
-
-
 class _AsyncConnectContextManager(HybridCoroutineContextManager[SnowflakeConnection]):
     """Hybrid wrapper that enables both awaiting and async context manager usage.
 
@@ -160,6 +151,7 @@ def Connect(**kwargs: Any) -> HybridCoroutineContextManager[SnowflakeConnection]
     - conn = await connect(...)
     - async with connect(...) as conn:
     """
+
     async def _connect_coro() -> SnowflakeConnection:
         conn = SnowflakeConnection(**kwargs)
         await conn.connect()
@@ -168,4 +160,4 @@ async def _connect_coro() -> SnowflakeConnection:
     return _AsyncConnectContextManager(_connect_coro())
 
 
-connect = Connect
+connect = Connect

test/integ/test_connection.py

@@ -1629,15 +1629,15 @@ def test_connect_metadata_preservation():
     assert callable(connect), "connect should be callable"
 
     # Test 9: Check type() and __class__ values (important for user introspection)
-    assert type(connect).__name__ == "function", (
-        f"type(connect).__name__ should be 'function', but got '{type(connect).__name__}'"
-    )
-    assert connect.__class__.__name__ == "function", (
-        f"connect.__class__.__name__ should be 'function', but got '{connect.__class__.__name__}'"
-    )
-    assert inspect.isfunction(connect), (
-        "connect should be recognized as a function by inspect.isfunction()"
-    )
+    assert (
+        type(connect).__name__ == "function"
+    ), f"type(connect).__name__ should be 'function', but got '{type(connect).__name__}'"
+    assert (
+        connect.__class__.__name__ == "function"
+    ), f"connect.__class__.__name__ should be 'function', but got '{connect.__class__.__name__}'"
+    assert inspect.isfunction(
+        connect
+    ), "connect should be recognized as a function by inspect.isfunction()"
 
     # Test 10: Verify the function has proper introspection capabilities
     # IDEs and type checkers should be able to resolve parameters

test/unit/aio/test_connection_async_unit.py

@@ -877,9 +877,9 @@ async def test_connect_metadata_preservation():
     from snowflake.connector.aio import SnowflakeConnection, connect
 
     # Test 1: Check __name__ is correct
-    assert connect.__name__ == "__init__", (
-        f"connect.__name__ should be '__init__', but got '{connect.__name__}'"
-    )
+    assert (
+        connect.__name__ == "__init__"
+    ), f"connect.__name__ should be '__init__', but got '{connect.__name__}'"
     assert (
         connect.__qualname__ == "SnowflakeConnection.__init__"
     ), f"connect.__qualname__ should be 'connect', but got '{connect.__qualname__}'"
@@ -936,15 +936,15 @@ async def test_connect_metadata_preservation():
     assert callable(connect), "connect should be callable"
 
     # Test 9: Check type() and __class__ values (important for user introspection)
-    assert type(connect).__name__ == "function", (
-        f"type(connect).__name__ should be 'function', but got '{type(connect).__name__}'"
-    )
-    assert connect.__class__.__name__ == "function", (
-        f"connect.__class__.__name__ should be 'function', but got '{connect.__class__.__name__}'"
-    )
-    assert inspect.isfunction(connect), (
-        "connect should be recognized as a function by inspect.isfunction()"
-    )
+    assert (
+        type(connect).__name__ == "function"
+    ), f"type(connect).__name__ should be 'function', but got '{type(connect).__name__}'"
+    assert (
+        connect.__class__.__name__ == "function"
+    ), f"connect.__class__.__name__ should be 'function', but got '{connect.__class__.__name__}'"
+    assert inspect.isfunction(
+        connect
+    ), "connect should be recognized as a function by inspect.isfunction()"
 
     # Test 10: Verify the function has proper introspection capabilities
     # IDEs and type checkers should be able to resolve parameters