Skip to content

Commit

Permalink
Fix Linting
Browse files Browse the repository at this point in the history
  • Loading branch information
stevenayers committed Mar 4, 2025
1 parent e53e903 commit 69924e9
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 4 deletions.
2 changes: 1 addition & 1 deletion dlt/destinations/impl/snowflake/utils.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
from typing import Optional, Dict, Any
from typing import Optional
from urllib.parse import urlparse, urlunparse

from dlt.common.configuration.specs import (
Expand Down
14 changes: 11 additions & 3 deletions tests/load/snowflake/test_snowflake_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,15 @@ def test_file_format_parquet_vectorized(test_table, local_file_path, local_stage
(False, ";", "ASCII", True),
],
)
def test_file_format_csv(include_header, delimiter, encoding, on_error_continue, test_table, local_file_path, local_stage_path):
def test_file_format_csv(
include_header,
delimiter,
encoding,
on_error_continue,
test_table,
local_file_path,
local_stage_path,
):
"""Test CSV format handling in gen_copy_sql with various options."""
csv_config = CsvFormatConfiguration(
include_header=include_header,
Expand Down Expand Up @@ -412,7 +420,7 @@ def test_full_workflow_s3_with_aws_credentials(test_table, aws_credentials):
staging_credentials=aws_credentials,
)

# Verify the final SQL
# Verify the final SQL
assert_sql_contains(
sql,
f"COPY INTO {test_table}",
Expand Down Expand Up @@ -486,4 +494,4 @@ def test_snowflake_azure_converter() -> None:
assert (
azure_url
== "azure://my_account.blob.core.windows.net/dlt-ci-test-bucket/path/to/file.parquet"
)
)

0 comments on commit 69924e9

Please sign in to comment.