Make seed option implicit and add --no-seed (#280)

ansible · Dec 23, 2024 · 1a346e3 · 1a346e3
1 parent ce96fdb
commit 1a346e3
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/src/ansible_dev_environment/arg_parser.py b/src/ansible_dev_environment/arg_parser.py
@@ -197,8 +197,9 @@ def parse() -> argparse.Namespace:
     install.add_argument(
         # "-adt",
         "--seed",
-        action="store_true",
-        dest="adt",
+        action=argparse.BooleanOptionalAction,
+        default=True,
+        dest="seed",
         help="Install seed packages inside the virtual environment (ansible-dev-tools).",
     )
 

diff --git a/src/ansible_dev_environment/subcommands/installer.py b/src/ansible_dev_environment/subcommands/installer.py
@@ -56,7 +56,7 @@ def run(self) -> None:
             err = "Multiple optional dependencies are not supported at this time."
             self._output.critical(err)
 
-        if self._config.args.adt:
+        if self._config.args.seed:
             self._install_dev_tools()
         else:
             self._install_core()

diff --git a/tests/conftest.py b/tests/conftest.py
@@ -195,6 +195,7 @@ def session_venv(session_dir: Path, monkey_session: pytest.MonkeyPatch) -> Confi
         [
             "ade",
             "install",
+            "--no-seed",
             "-r",
             str(GALAXY_CACHE / REQS_FILE_NAME),
             "--venv",

diff --git a/tests/unit/test_installer.py b/tests/unit/test_installer.py
@@ -271,7 +271,7 @@ def test_no_adt_install(
     args = Namespace(
         venv=venv,
         verbose=0,
-        adt=False,
+        seed=False,
         system_site_packages=False,
         collection_specifier=None,
         requirement=None,
@@ -302,7 +302,7 @@ def test_adt_install(
     args = Namespace(
         venv=venv,
         verbose=0,
-        adt=True,
+        seed=True,
         system_site_packages=False,
         collection_specifier=None,
         requirement=None,