make commit the default for start and stop commands

Author: gilesknap

src/edge_containers_cli/cli.py

@@ -266,7 +266,7 @@ def start(
         autocompletion=all_svc,
         show_default=False,
     ),
-    commit: bool = typer.Option(False, help="Commits the values to the git repo"),
+    commit: bool = typer.Option(True, help="Commits the values to the git repo"),
 ):
     """Start a service"""
     try:
@@ -284,7 +284,7 @@ def stop(
         autocompletion=running_svc,
         show_default=False,
     ),
-    commit: bool = typer.Option(False, help="Commits the values to the git repo"),
+    commit: bool = typer.Option(True, help="Commits the values to the git repo"),
 ):
     """Stop a service"""
     try:

tests/test_argocd.py

@@ -43,19 +43,19 @@ def test_start_commit(mock_run, ARGOCD, data: Path):
 
 def test_start(mock_run, ARGOCD):
     mock_run.set_seq(ARGOCD.checks + ARGOCD.start)
-    mock_run.run_cli("start bl01t-ea-test-01")
+    mock_run.run_cli("start bl01t-ea-test-01 --no-commit")
 
 
 def test_stop_commit(mock_run, ARGOCD, data: Path):
     mock_run.set_seq(ARGOCD.checks + ARGOCD.stop_commit)
     TMPDIR.mkdir()
     shutil.copytree(data / "bl01t-deployment/apps", TMPDIR / "apps")
-    mock_run.run_cli("stop bl01t-ea-test-01 --commit")
+    mock_run.run_cli("stop bl01t-ea-test-01")
 
 
 def test_stop(mock_run, ARGOCD):
     mock_run.set_seq(ARGOCD.checks + ARGOCD.stop)
-    mock_run.run_cli("stop bl01t-ea-test-01")
+    mock_run.run_cli("stop bl01t-ea-test-01 --no-commit")
 
 
 def test_ps(mock_run, ARGOCD):