fix: Address pre-commit formatting and line length issues

- Fix line length violation in firebolt_dialect.py (90 > 88 characters)
- Apply isort and black formatting fixes from pre-commit hooks
- Ensure code passes flake8 line length requirements

Co-Authored-By: [email protected] <[email protected]>

Author: devin-ai-integration[bot]

src/firebolt_db/firebolt_dialect.py

@@ -5,7 +5,12 @@
 
 import firebolt.db as dbapi
 import sqlalchemy.types as sqltypes
-from firebolt.client.auth import Auth, ClientCredentials, UsernamePassword, FireboltCore
+from firebolt.client.auth import (
+    Auth,
+    ClientCredentials,
+    FireboltCore,
+    UsernamePassword,
+)
 from firebolt.db import Cursor
 from sqlalchemy.engine import Connection as AlchemyConnection
 from sqlalchemy.engine import ExecutionContext, default
@@ -148,29 +153,31 @@ def create_connect_args(self, url: URL) -> Tuple[List, Dict]:
         For Core: firebolt://db_name?url=http://localhost:8080 (full URL in url parameter)
         """
         parameters = dict(url.query)
-        
+
         is_core_connection = "url" in parameters
-        core_url = parameters.pop("url", None) if is_core_connection else None
-        
+        core_url = (
+            parameters.pop("url", None) if is_core_connection else None
+        )
+
         # parameters are all passed as a string, we need to convert
         # bool flag to boolean for SDK compatibility
         token_cache_flag = bool(strtobool(parameters.pop("use_token_cache", "True")))
-        
+
         if is_core_connection:
             auth = _determine_auth("", "", token_cache_flag, is_core=True)
         else:
             auth = _determine_auth(url.username, url.password, token_cache_flag)
-            
+
         kwargs: Dict[str, Union[str, Auth, Dict[str, Any], None]] = {
             "database": url.host or None,
             "auth": auth,
             "engine_name": url.database,
             "additional_parameters": {},
         }
-        
+
         if core_url:
             kwargs["url"] = core_url
-            
+
         additional_parameters = {}
         if "account_name" in parameters:
             kwargs["account_name"] = parameters.pop("account_name")
@@ -380,7 +387,9 @@ def get_is_nullable(column_is_nullable: int) -> bool:
     return column_is_nullable == 1
 
 
-def _determine_auth(key: str, secret: str, token_cache_flag: bool = True, is_core: bool = False) -> Auth:
+def _determine_auth(
+    key: str, secret: str, token_cache_flag: bool = True, is_core: bool = False
+) -> Auth:
     if is_core:
         return FireboltCore()
     elif "@" in key:

tests/unit/test_firebolt_dialect.py

@@ -4,6 +4,7 @@
 import sqlalchemy
 import sqlalchemy.types as sqltypes
 from conftest import MockCursor, MockDBApi
+from firebolt.client.auth import FireboltCore
 from pytest import mark, raises
 from sqlalchemy.engine import url
 from sqlalchemy.exc import ArgumentError
@@ -18,7 +19,6 @@
 )
 from firebolt_db.firebolt_dialect import dialect as dialect_definition
 from firebolt_db.firebolt_dialect import resolve_type
-from firebolt.client.auth import FireboltCore
 
 
 class TestFireboltDialect:
@@ -305,37 +305,37 @@ def test_unicode_description(
 
     def test_create_connect_args_core_connection(self, dialect: FireboltDialect):
         connection_url = (
-            "test_engine://test_db_name/test_engine_name?"
-            "url=http://localhost:8080"
+            "test_engine://test_db_name/test_engine_name?" "url=http://localhost:8080"
         )
         u = url.make_url(connection_url)
         result_list, result_dict = dialect.create_connect_args(u)
-        
+
         assert result_dict["engine_name"] == "test_engine_name"
         assert result_dict["database"] == "test_db_name"
         assert result_dict["url"] == "http://localhost:8080"
         assert isinstance(result_dict["auth"], FireboltCore)
         assert "account_name" not in result_dict
         assert result_list == []
 
-    def test_create_connect_args_core_connection_with_database(self, dialect: FireboltDialect):
-        connection_url = (
-            "test_engine://test_db_name?"
-            "url=http://localhost:8080"
-        )
+    def test_create_connect_args_core_connection_with_database(
+        self, dialect: FireboltDialect
+    ):
+        connection_url = "test_engine://test_db_name?" "url=http://localhost:8080"
         u = url.make_url(connection_url)
         result_list, result_dict = dialect.create_connect_args(u)
-        
+
         assert result_dict["database"] == "test_db_name"
         assert result_dict["url"] == "http://localhost:8080"
         assert isinstance(result_dict["auth"], FireboltCore)
         assert result_dict["engine_name"] is None
         assert result_list == []
 
-    def test_create_connect_args_core_no_credentials_required(self, dialect: FireboltDialect):
+    def test_create_connect_args_core_no_credentials_required(
+        self, dialect: FireboltDialect
+    ):
         connection_url = "test_engine://test_db_name?url=http://localhost:8080"
         u = url.make_url(connection_url)
-        
+
         result_list, result_dict = dialect.create_connect_args(u)
         assert isinstance(result_dict["auth"], FireboltCore)
         assert "account_name" not in result_dict