Skip to content

Commit

Permalink
Fix linter
Browse files Browse the repository at this point in the history
  • Loading branch information
svdimchenko committed Feb 4, 2025
1 parent 04a6064 commit 8d3d181
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,7 @@ class DBTCommonConfig(
include_database_name: bool = Field(
default=True,
description="Whether to add database name to the table urn. "
"Set to False to skip it for engines like AWS Athena where it's not required.",
"Set to False to skip it for engines like AWS Athena where it's not required.",
)

@validator("target_platform")
Expand Down
5 changes: 3 additions & 2 deletions metadata-ingestion/tests/unit/dbt/test_dbt_source.py
Original file line number Diff line number Diff line change
Expand Up @@ -494,6 +494,7 @@ def test_get_column_type_redshift():
== "Got an unexpected column type. The column's parsed field type will not be populated."
)


def test_include_database_name_default():
config_dict = {
"manifest_path": "dummy_path",
Expand All @@ -503,9 +504,9 @@ def test_include_database_name_default():
config = DBTCoreConfig.parse_obj({**config_dict})
assert config.include_database_name is True


@pytest.mark.parametrize(
("include_database_name", "expected"),
[("false", False), ("true", True)]
("include_database_name", "expected"), [("false", False), ("true", True)]
)
def test_include_database_name(include_database_name: str, expected: bool):
config_dict = {
Expand Down

0 comments on commit 8d3d181

Please sign in to comment.