Skip to content

Commit 8d2136f

Browse files
millintomasfarias
authored andcommitted
fix: remove unnecessary os.sep
1 parent b129b3a commit 8d2136f

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

airflow_dbt_python/hooks/dbt.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
import json
66
import logging
7-
import os
87
import sys
98
from abc import ABC
109
from contextlib import contextmanager
@@ -347,8 +346,8 @@ def prepare_directory(
347346
profiles_dir_path = None
348347

349348
return (
350-
str(project_dir_path) + os.sep,
351-
str(profiles_dir_path) + os.sep if profiles_dir_path is not None else None,
349+
str(project_dir_path),
350+
str(profiles_dir_path) if profiles_dir_path is not None else None,
352351
)
353352

354353
def setup_dbt_logging(self, debug: Optional[bool]):

tests/hooks/dbt/test_dbt_hook_base.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -379,8 +379,8 @@ def test_dbt_base_dbt_directory_changed_to_s3(
379379
assert Path(tmp_dir).exists()
380380
assert Path(tmp_dir).is_dir()
381381

382-
assert config.project_dir == f"{tmp_dir}/"
383-
assert config.profiles_dir == f"{tmp_dir}/"
382+
assert config.project_dir == tmp_dir
383+
assert config.profiles_dir == tmp_dir
384384
assert config.state == f"{tmp_dir}/target"
385385

386386
assert Path(f"{tmp_dir}/profiles.yml").exists()

0 commit comments

Comments
 (0)