fix: Make account name optional (#92)

Author: ptiurin

src/firebolt_db/firebolt_dialect.py

@@ -153,7 +153,8 @@ def create_connect_args(self, url: URL) -> Tuple[List, Dict]:
         additional_parameters = {}
         if "account_name" in parameters:
             kwargs["account_name"] = parameters.pop("account_name")
-        else:
+        elif isinstance(auth, ClientCredentials):
+            # account_name is required for client credentials authentication
             raise ArgumentError(
                 "account_name parameter must be provided to authenticate"
             )

tests/unit/test_firebolt_dialect.py

@@ -36,7 +36,6 @@ def test_create_dialect(self, dialect: FireboltDialect):
     def test_create_connect_args_user_password(self, dialect: FireboltDialect):
         u = url.make_url(
             "test_engine://[email protected]:test_password@test_db_name/test_engine_name"
-            "?account_name=dummy"
         )
         with mock.patch.dict(os.environ, {"FIREBOLT_BASE_URL": "test_url"}):
             result_list, result_dict = dialect.create_connect_args(u)