git restore --source=origin/master tests src

Author: tony

src/vcspull/cli/__init__.py

@@ -97,29 +97,27 @@ def create_parser(
 
     add_parser = subparsers.add_parser(
         "add",
-        help="add a new repository to the configuration",
+        help="add a repository to the configuration",
         formatter_class=argparse.RawDescriptionHelpFormatter,
-        description="Adds a new repository to the vcspull configuration file.",
+        description="Add a repository to the vcspull configuration file.",
     )
     create_add_subparser(add_parser)
 
     add_from_fs_parser = subparsers.add_parser(
         "add-from-fs",
-        help="scan a directory for git repositories and add them to the configuration",
+        help="scan filesystem for git repositories and add them to the configuration",
         formatter_class=argparse.RawDescriptionHelpFormatter,
-        description="Scans a directory for git repositories and adds them to the vcspull configuration file.",
+        description="Scan a directory for git repositories and add them to the vcspull configuration file.",
     )
     create_add_from_fs_subparser(add_from_fs_parser)
 
-    all_subparsers_dict = {
-        "sync": sync_parser,
-        "add": add_parser,
-        "add_from_fs": add_from_fs_parser,
-    }
-
-    if get_all_subparsers:
-        return parser, subparsers, all_subparsers_dict
-
+    if get_all_subparsers and return_subparsers:
+        all_parsers = {
+            "sync": sync_parser,
+            "add": add_parser,
+            "add_from_fs": add_from_fs_parser,
+        }
+        return parser, subparsers, all_parsers
     if return_subparsers:
         return parser, sync_parser
     return parser
@@ -169,7 +167,7 @@ def cli(_args: list[str] | None = None) -> None:
             else None,
         }
         add_repo(**add_repo_kwargs)
-    elif args.subparser_name == "add_from_fs":
+    elif args.subparser_name == "add-from-fs":
         all_parsers["add_from_fs"]
         add_from_fs_kwargs = {
             "scan_dir_str": args.scan_dir,

tests/helpers.py

@@ -5,7 +5,6 @@
 import os
 import typing as t
 
-import yaml
 from typing_extensions import Self
 
 from vcspull._internal.config_reader import ConfigReader
@@ -62,14 +61,3 @@ def write_config(config_path: pathlib.Path, content: str) -> pathlib.Path:
 def load_raw(data: str, fmt: t.Literal["yaml", "json"]) -> dict[str, t.Any]:
     """Load configuration data via string value. Accepts yaml or json."""
     return ConfigReader._load(fmt=fmt, content=data)
-
-
-def save_config_yaml(config_file_path: pathlib.Path, data: dict[t.Any, t.Any]) -> None:
-    """Save a dictionary to a YAML file by utilizing write_config.
-
-    The primary difference from `write_config` is that this function
-    accepts a dictionary and handles YAML serialization before calling
-    `write_config`.
-    """
-    yaml_content: str = yaml.dump(data)
-    write_config(config_file_path, yaml_content)