refactor: VALID_WORKSPACE_DIR_FILE_EXTENSIONS

Author: tony

src/tmuxp/cli/constants.py

@@ -1 +1 @@
-VALID_CONFIG_DIR_FILE_EXTENSIONS = [".yaml", ".yml", ".json"]
+VALID_WORKSPACE_DIR_FILE_EXTENSIONS = [".yaml", ".yml", ".json"]

src/tmuxp/cli/ls.py

@@ -2,7 +2,7 @@
 import os
 import typing as t
 
-from .constants import VALID_CONFIG_DIR_FILE_EXTENSIONS
+from .constants import VALID_WORKSPACE_DIR_FILE_EXTENSIONS
 from .utils import get_config_dir
 
 
@@ -19,6 +19,6 @@ def command_ls(
     if os.path.exists(tmuxp_dir) and os.path.isdir(tmuxp_dir):
         for f in sorted(os.listdir(tmuxp_dir)):
             stem, ext = os.path.splitext(f)
-            if os.path.isdir(f) or ext not in VALID_CONFIG_DIR_FILE_EXTENSIONS:
+            if os.path.isdir(f) or ext not in VALID_WORKSPACE_DIR_FILE_EXTENSIONS:
                 continue
             print(stem)

src/tmuxp/cli/utils.py

@@ -8,7 +8,7 @@
 from tmuxp.types import StrPath
 
 from .. import log
-from .constants import VALID_CONFIG_DIR_FILE_EXTENSIONS
+from .constants import VALID_WORKSPACE_DIR_FILE_EXTENSIONS
 
 logger = logging.getLogger(__name__)
 
@@ -121,7 +121,7 @@ def scan_config(
                 x
                 for x in [
                     f"{join(workspace_dir, workspace_file)}{ext}"
-                    for ext in VALID_CONFIG_DIR_FILE_EXTENSIONS
+                    for ext in VALID_WORKSPACE_DIR_FILE_EXTENSIONS
                 ]
                 if exists(x)
             ]

tests/test_cli.py

@@ -1362,7 +1362,7 @@ def test_ls_cli(
 
     # should ignore:
     # - directories should be ignored
-    # - extensions not covered in VALID_CONFIG_DIR_FILE_EXTENSIONS
+    # - extensions not covered in VALID_WORKSPACE_DIR_FILE_EXTENSIONS
     ignored_filenames = [".git/", ".gitignore/", "session_4.txt"]
     stems = [os.path.splitext(f)[0] for f in filenames if f not in ignored_filenames]