added legacy-escape command

Author: Andrei Neagu

packages/service-integration/src/service_integration/cli/__init__.py

@@ -7,7 +7,7 @@
 
 from .._meta import __version__
 from ..settings import AppSettings
-from . import _compose_spec, _metadata, _run_creator, _test
+from . import _compose_spec, _escaping, _metadata, _run_creator, _test
 from ._config import config_app
 
 app = typer.Typer()
@@ -76,3 +76,4 @@ def main(
 app.command("bump-version")(_metadata.bump_version)
 app.command("get-version")(_metadata.get_version)
 app.command("run-creator")(_run_creator.run_creator)
+app.command("legacy-escape")(_escaping.legacy_escape)

packages/service-integration/src/service_integration/cli/_escaping.py

@@ -0,0 +1,24 @@
+from pathlib import Path
+from typing import Annotated
+
+import typer
+
+from ..osparc_config import OSPARC_CONFIG_DIRNAME
+
+
+def legacy_escape(
+    osparc_config_dirname: Annotated[
+        Path,
+        typer.Option(
+            "--osparc-config-dirname",
+            help="Path to where the .osparc configuration directory is located",
+        ),
+    ] = Path(OSPARC_CONFIG_DIRNAME),
+):
+    """Escapes the `$${` with `$$$${` in all .y*ml files in the osparc config directory."""
+    for file in osparc_config_dirname.glob("*.y*ml"):
+        read_text = file.read_text()
+        replaced_text = read_text.replace("$${", "$$$${")
+        if read_text != replaced_text:
+            print(f"Escaped sequnce in {file}")
+        file.write_text(replaced_text)

packages/service-integration/tests/test_cli.py

@@ -1,18 +1,54 @@
+import os
+import shutil
+import traceback
 from collections.abc import Callable
+from pathlib import Path
 
+import pytest
+from click.testing import Result
 from service_integration import __version__
 
 
+def _format_cli_error(result: Result) -> str:
+    assert result.exception
+    tb_message = "\n".join(traceback.format_tb(result.exception.__traceback__))
+    return f"Below exception was raised by the cli:\n{tb_message}"
+
+
 def test_cli_help(run_program_with_args: Callable):
     result = run_program_with_args(
         "--help",
     )
-    assert result.exit_code == 0
+    assert result.exit_code == os.EX_OK, _format_cli_error(result)
 
 
 def test_cli_version(run_program_with_args: Callable):
     result = run_program_with_args(
         "--version",
     )
-    assert result.exit_code == 0
+    assert result.exit_code == os.EX_OK, _format_cli_error(result)
     assert __version__ == result.output.strip()
+
+
+@pytest.fixture
+def copy_tests_data_dir(tests_data_dir: Path, tmp_path: Path) -> Path:
+    new_dir_path = tmp_path / "copy_tests_data_dir"
+    new_dir_path.mkdir(exist_ok=True, parents=True)
+
+    for item in tests_data_dir.glob("*"):
+        print(f"Copying {item} to {new_dir_path / item.name}")
+        shutil.copy2(item, new_dir_path / item.name)
+
+    return new_dir_path
+
+
+def test_cli_legacy_escape(copy_tests_data_dir: Path, run_program_with_args: Callable):
+    result = run_program_with_args(
+        "legacy-escape", "--osparc-config-dirname", copy_tests_data_dir
+    )
+    assert result.exit_code == os.EX_OK, _format_cli_error(result)
+    # NOTE only 1 file will have a sequnce that will be escaped
+    assert (
+        result.output.strip()
+        == f"Escaped sequnce in {copy_tests_data_dir}/docker-compose-meta.yml"
+    )