test

demisto_sdk/tests/integration_tests/logger_integration_test.py

@@ -6,6 +6,7 @@
 from click.testing import CliRunner
 
 from demisto_sdk.__main__ import main
+from demisto_sdk.commands.common.logger import LOG_FILE_PATH
 from TestSuite.test_tools import str_in_call_args_list
 
 TEMP_DIRS_PREFIX = "demisto-sdk-test-"
@@ -144,9 +145,8 @@ def test_logs_dir_not_a_dir(mocker):
     """
     default_logs_dir = Path(mkdtemp(prefix=TEMP_DIRS_PREFIX)).resolve()
     custom_logs_dir = (
-        Path(mkdtemp(prefix=TEMP_DIRS_PREFIX)).resolve() / "some_random_file"
+        Path(mkdtemp(prefix=TEMP_DIRS_PREFIX)).resolve() / "some_random_file.txt"
     )
-    custom_logs_dir_parent = custom_logs_dir.parent
     custom_logs_dir.touch()
     assert custom_logs_dir.is_file()  # Ensure file was created successfully
 
@@ -156,12 +156,16 @@ def test_logs_dir_not_a_dir(mocker):
     runner = CliRunner(mix_stderr=False)
     runner.invoke(main, ["validate", "-a", "--log-file-path", str(custom_logs_dir)])
 
+    custom_logs_dir_parent = custom_logs_dir.parent
+
     assert str_in_call_args_list(
         logger_warning.call_args_list,
         f"Log file path '{custom_logs_dir}' is a file and not a directory. Log file will be created in "
         f"parent directory '{custom_logs_dir_parent}'.",
     )
-    assert list(custom_logs_dir_parent.glob("*")) == [
+    files_in_parent_dir = list(custom_logs_dir_parent.glob("*"))
+    assert len(files_in_parent_dir) == 2
+    assert files_in_parent_dir == [
         custom_logs_dir,  # The random file we created
         custom_logs_dir_parent / "demisto_sdk_debug.log",
     ]