fix: remove unnecessary os.sep

Author: millin

airflow_dbt_python/hooks/dbt.py

@@ -4,7 +4,6 @@
 
 import json
 import logging
-import os
 import sys
 from abc import ABC
 from contextlib import contextmanager
@@ -258,7 +257,7 @@ def get_dbt_task_config(self, command: str, **config_kwargs) -> BaseConfig:
     @contextmanager
     def dbt_directory(
         self,
-        config,
+        config: BaseConfig,
         upload_dbt_project: bool = False,
         delete_before_upload: bool = False,
         replace_on_upload: bool = False,
@@ -347,8 +346,8 @@ def prepare_directory(
             profiles_dir_path = None
 
         return (
-            str(project_dir_path) + os.sep,
-            str(profiles_dir_path) + os.sep if profiles_dir_path is not None else None,
+            str(project_dir_path),
+            str(profiles_dir_path) if profiles_dir_path is not None else None,
         )
 
     def setup_dbt_logging(self, debug: Optional[bool]):

tests/hooks/dbt/test_dbt_hook_base.py

@@ -379,8 +379,8 @@ def test_dbt_base_dbt_directory_changed_to_s3(
         assert Path(tmp_dir).exists()
         assert Path(tmp_dir).is_dir()
 
-        assert config.project_dir == f"{tmp_dir}/"
-        assert config.profiles_dir == f"{tmp_dir}/"
+        assert config.project_dir == tmp_dir
+        assert config.profiles_dir == tmp_dir
         assert config.state == f"{tmp_dir}/target"
 
         assert Path(f"{tmp_dir}/profiles.yml").exists()