From 842e15c4f048ab3a2a5dba14ae95e36dd440a2fd Mon Sep 17 00:00:00 2001 From: Jesse Bannon Date: Thu, 9 Mar 2023 23:37:38 -0800 Subject: [PATCH] test fix --- src/ytdl_sub/validators/file_path_validators.py | 4 ++-- tests/unit/validators/test_file_path_validators.py | 1 - 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/ytdl_sub/validators/file_path_validators.py b/src/ytdl_sub/validators/file_path_validators.py index f37d39a3..fb070d5e 100644 --- a/src/ytdl_sub/validators/file_path_validators.py +++ b/src/ytdl_sub/validators/file_path_validators.py @@ -83,13 +83,13 @@ class StringFormatterFilePathValidator(StringFormatterValidator): def apply_formatter(self, variable_dict: Dict[str, str]) -> str: """Turn into a Path, then a string, to get correct directory separators""" - file_path = str(Path(super().apply_formatter(variable_dict))) + file_path = Path(super().apply_formatter(variable_dict)) file_directory, file_name = os.path.split(Path(file_path)) if self._is_file_name_too_long(file_name): return str(Path(file_directory) / self._truncate_file_name(file_name)) - return file_path + return str(file_path) class OverridesStringFormatterValidatorFilePathValidator(OverridesStringFormatterValidator): diff --git a/tests/unit/validators/test_file_path_validators.py b/tests/unit/validators/test_file_path_validators.py index 7fe057de..aa0ab181 100644 --- a/tests/unit/validators/test_file_path_validators.py +++ b/tests/unit/validators/test_file_path_validators.py @@ -17,7 +17,6 @@ class TestStringFormatterFilePathValidator: + [f".en.{ext}" for ext in SUBTITLE_EXTENSIONS], ) def test_truncates_file_name_successfully(self, ext: str): - ext = ".info.json" with tempfile.TemporaryDirectory() as temp_dir: file_name = ("a" * 10000) + ext file_path = str(Path(temp_dir) / file_name)