elemctl 0.2.0__tar.gz → 0.3.0__tar.gz

{elemctl-0.2.0 → elemctl-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: elemctl
-Version: 0.2.0
+Version: 0.3.0
 Summary: CLI, MCP-сервер и библиотека для Console API v2 платформы 1С:Предприятие.Элемент (1cmycloud)
 Author: KeyFire
 License-Expression: MIT
@@ -112,6 +112,9 @@ elemctl apps list
 # карточка приложения (статус, uri, фактическая версия проекта)
 elemctl apps get <app-id>
 
+# создать приложение, только если его ещё нет: {"id": ..., "created": true|false}
+elemctl apps ensure acme-crm-dev --project-id <project-id> --latest-build --wait
+
 # полный цикл деплоя из исходников с проверкой применения
 elemctl deploy --app-id <app-id> --project-id <project-id> --project-dir acme/crm
 
@@ -184,6 +187,10 @@ Assembly.yaml            # манифест: ProjectKind, Vendor, Name, Version,
   `elemctl apps create <имя> --project-id <id> --latest-build` (MCP-инструмент
   `create_app` подставляет последнюю сборку автоматически), а после создания -
   `elemctl deploy`.
+- Удалённые приложения остаются в списке платформы со статусом `Deleted` и
+  прежним `id`, на котором `apps get` и `deploy` отвечают 404. `apps find` и
+  `apps ensure` их пропускают; вернуть прежний поиск - `apps find
+  --include-deleted`.
 - Приложение с неопубликованными правками в среде разработки платформа
   удалить не даёт (HTTP 400 `FAILED_PRECONDITION`), принудительного удаления
   в Console API нет - только через панель управления; elemctl подскажет это

{elemctl-0.2.0 → elemctl-0.3.0}/README.md

@@ -84,6 +84,9 @@ elemctl apps list
 # карточка приложения (статус, uri, фактическая версия проекта)
 elemctl apps get <app-id>
 
+# создать приложение, только если его ещё нет: {"id": ..., "created": true|false}
+elemctl apps ensure acme-crm-dev --project-id <project-id> --latest-build --wait
+
 # полный цикл деплоя из исходников с проверкой применения
 elemctl deploy --app-id <app-id> --project-id <project-id> --project-dir acme/crm
 
@@ -156,6 +159,10 @@ Assembly.yaml            # манифест: ProjectKind, Vendor, Name, Version,
   `elemctl apps create <имя> --project-id <id> --latest-build` (MCP-инструмент
   `create_app` подставляет последнюю сборку автоматически), а после создания -
   `elemctl deploy`.
+- Удалённые приложения остаются в списке платформы со статусом `Deleted` и
+  прежним `id`, на котором `apps get` и `deploy` отвечают 404. `apps find` и
+  `apps ensure` их пропускают; вернуть прежний поиск - `apps find
+  --include-deleted`.
 - Приложение с неопубликованными правками в среде разработки платформа
   удалить не даёт (HTTP 400 `FAILED_PRECONDITION`), принудительного удаления
   в Console API нет - только через панель управления; elemctl подскажет это

{elemctl-0.2.0 → elemctl-0.3.0}/docs/SPEC.md

@@ -255,13 +255,30 @@ stderr и код возврата 1.
 Команды (в скобках - существенные флаги):
 
 - `token` - получить и напечатать токен.
-- `apps list [--name]`, `apps get [APP_ID]`, `apps find NAME`,
+- `apps list [--name]`, `apps get [APP_ID]`, `apps find NAME [--include-deleted]`,
   `apps create NAME [--project-id --version-id --latest-build --space-id
+  --tech-version --no-dev-mode --wait]`,
+  `apps ensure NAME [--project-id --version-id --latest-build --space-id
   --tech-version --no-dev-mode --wait]`, `apps delete APP_ID`,
   `apps start [APP_ID]`, `apps stop [APP_ID]`.
   - `apps find` ищет по точному совпадению (без учёта регистра) имени среди
     полей `name`, `display-name`, `publication-context`; вывод
-    `{"id": ..., "found": true|false}`, код возврата 1 если не найдено.
+    `{"id": ..., "found": true|false}`, код возврата 0 в обоих случаях –
+    отсутствие приложения это ответ, а не ошибка. Ненулевой код возврата
+    означает сбой запроса и сопровождается JSON'ом с полем `error` в stderr.
+    В скриптах проверять поле `found`, а не код возврата.
+  - Удалённые приложения остаются в списке платформы со статусом `Deleted` и
+    прежним `id`. `apps find` их ПРОПУСКАЕТ: найденный id обязан быть пригоден
+    для работы, иначе вызывающий получает id, на котором `apps get` и `deploy`
+    отвечают 404. Флаг `--include-deleted` возвращает прежнее поведение – поиск
+    среди всех приложений, включая удалённые.
+  - `apps ensure` идемпотентно приводит приложение с данным именем в
+    существование: ищет по правилам `apps find` (удалённые не в счёт) и создаёт
+    только при отсутствии. Вывод `{"id": ..., "created": true|false}`;
+    `created: false` означает, что приложение уже было и НЕ трогалось. Флаги
+    создания те же, что у `apps create`, и действуют, только когда создание
+    происходит. Существующее приложение никогда не пересоздаётся: `delete` +
+    `create` дают новый URL и рвут внешние привязки к прежнему.
   - `--latest-build` - взять источником последнюю сборку проекта (защита от
     пустого каркаса, п. 6.2); `--wait` - дождаться готовности (п. 6.4) и
     вывести финальную карточку.

{elemctl-0.2.0 → elemctl-0.3.0}/pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "elemctl"
-version = "0.2.0"
+version = "0.3.0"
 description = "CLI, MCP-сервер и библиотека для Console API v2 платформы 1С:Предприятие.Элемент (1cmycloud)"
 readme = "README.md"
 requires-python = ">=3.10"

{elemctl-0.2.0 → elemctl-0.3.0}/src/elemctl/__init__.py

@@ -11,7 +11,7 @@ from .client import ElementClient
 from .config import Config
 from .errors import ApiError, BuildError, ConfigError, ElemctlError, TransportError
 
-__version__ = "0.2.0"
+__version__ = "0.3.0"
 
 __all__ = [
     "ApiError",

elemctl-0.3.0/src/elemctl/__main__.py

@@ -0,0 +1,14 @@
+"""Запуск пакета как модуля: python -m elemctl <аргументы>.
+
+Нужен тем, кто вызывает elemctl текущим интерпретатором, не полагаясь на
+консольную точку входа в PATH (например, обёртки в чужих репозиториях).
+"""
+
+from __future__ import annotations
+
+import sys
+
+from .cli import main
+
+if __name__ == "__main__":
+    sys.exit(main())

{elemctl-0.2.0 → elemctl-0.3.0}/src/elemctl/cli.py

@@ -97,18 +97,32 @@ def cmd_apps_get(args):
 
 
 def cmd_apps_find(args):
+    """Поиск приложения по имени. Отсутствие приложения – это ответ, а не ошибка.
+
+    Код возврата 0 в обоих случаях: признак несёт поле found. Ненулевой код возврата
+    означает сбой запроса (нет доступа, сеть, конфигурация) и сопровождается JSON'ом
+    с полем error в stderr – иначе вызывающий не отличит "стенда нет" от "не смогли спросить".
+
+    Удалённые приложения (статус Deleted) по умолчанию пропускаются, чтобы найденный id
+    был пригоден для работы; флаг --include-deleted возвращает поиск среди всех приложений.
+    """
     client = make_client(_config(args))
-    app = client.find_app(args.name)
+    app = client.find_app(args.name, include_deleted=args.include_deleted)
     if app is None:
         _emit({"id": None, "found": False})
-        return 1
+        return 0
     _emit({"id": app.get("id"), "found": True})
     return 0
 
 
-def cmd_apps_create(args):
-    config = _config(args)
-    client = make_client(config)
+def _create_app_from_args(client, config, args):
+    """Создать приложение по флагам создания; вернуть карточку.
+
+    Общая логика apps create и apps ensure. Источник – указанная сборка
+    (--version-id), последняя сборка проекта (--latest-build) либо проект
+    целиком (--project-id, чревато пустым каркасом). При --wait дожидается
+    готовности приложения.
+    """
     project_id = args.project_id or config.project_id
     version_id = args.version_id
 
@@ -143,7 +157,34 @@ def cmd_apps_create(args):
         app_id = (card or {}).get("id")
         if app_id:
             card = client.wait_app_ready(app_id, log=_progress)
-    _emit(card)
+    return card
+
+
+def cmd_apps_create(args):
+    config = _config(args)
+    client = make_client(config)
+    _emit(_create_app_from_args(client, config, args))
+    return 0
+
+
+def cmd_apps_ensure(args):
+    """Идемпотентно привести приложение с данным именем в существование.
+
+    Ищет приложение по правилам apps find (удалённые в статусе Deleted не в
+    счёт): если оно уже есть – ничего не делает и возвращает created: false;
+    если нет – создаёт по флагам создания и возвращает created: true.
+    Существующее приложение никогда не пересоздаётся: delete + create дают
+    новый URL и рвут внешние привязки к прежнему. Код возврата 0 в обоих
+    случаях; сбой запроса – JSON с полем error в stderr и код возврата 1.
+    """
+    config = _config(args)
+    client = make_client(config)
+    existing = client.find_app(args.name)
+    if existing is not None:
+        _emit({"id": existing.get("id"), "created": False})
+        return 0
+    card = _create_app_from_args(client, config, args)
+    _emit({"id": (card or {}).get("id"), "created": True})
     return 0
 
 
@@ -394,6 +435,18 @@ def cmd_mcp(args):
 # -- парсер --------------------------------------------------------------------
 
 
+def _add_create_flags(p):
+    """Добавить флаги источника и создания приложения (общие для apps create и apps ensure)."""
+    p.add_argument("name", metavar="NAME")
+    p.add_argument("--project-id", help="проект-источник")
+    p.add_argument("--version-id", help="id сборки-источника")
+    p.add_argument("--latest-build", action="store_true", help="источник – последняя сборка проекта")
+    p.add_argument("--space-id", help="пространство")
+    p.add_argument("--tech-version", help="версия технологии")
+    p.add_argument("--no-dev-mode", action="store_true", help="не создавать среду разработки")
+    p.add_argument("--wait", action="store_true", help="дождаться готовности приложения")
+
+
 def build_parser():
     parser = argparse.ArgumentParser(
         prog="elemctl",
@@ -425,19 +478,21 @@ def build_parser():
 
     p = apps_sub.add_parser("find", help="найти приложение по имени (точное совпадение без учёта регистра)")
     p.add_argument("name", metavar="NAME")
+    p.add_argument(
+        "--include-deleted",
+        action="store_true",
+        help="искать и среди удалённых приложений (по умолчанию пропускаются)",
+    )
     p.set_defaults(handler=cmd_apps_find)
 
     p = apps_sub.add_parser("create", help="создать приложение")
-    p.add_argument("name", metavar="NAME")
-    p.add_argument("--project-id", help="проект-источник")
-    p.add_argument("--version-id", help="id сборки-источника")
-    p.add_argument("--latest-build", action="store_true", help="источник – последняя сборка проекта")
-    p.add_argument("--space-id", help="пространство")
-    p.add_argument("--tech-version", help="версия технологии")
-    p.add_argument("--no-dev-mode", action="store_true", help="не создавать среду разработки")
-    p.add_argument("--wait", action="store_true", help="дождаться готовности и вывести карточку")
+    _add_create_flags(p)
     p.set_defaults(handler=cmd_apps_create)
 
+    p = apps_sub.add_parser("ensure", help="создать приложение, если его ещё нет (идемпотентно)")
+    _add_create_flags(p)
+    p.set_defaults(handler=cmd_apps_ensure)
+
     p = apps_sub.add_parser("delete", help="удалить приложение (необратимо, URL меняется при пересоздании)")
     p.add_argument("app_id", metavar="APP_ID")
     p.set_defaults(handler=cmd_apps_delete)

{elemctl-0.2.0 → elemctl-0.3.0}/src/elemctl/client.py

@@ -70,6 +70,21 @@ def _collapse_reference(value):
     return None
 
 
+# Статус удалённого приложения (сравнение без учёта регистра).
+DELETED_STATUS = "Deleted"
+
+
+def _is_deleted(app):
+    """Признак удалённого приложения.
+
+    Платформа не убирает удалённые приложения из списка, а помечает их
+    статусом Deleted, сохраняя прежний id. На таком id последующие get и
+    deploy отвечают 404, поэтому поиск по умолчанию их пропускает.
+    """
+    status = app.get("status")
+    return isinstance(status, str) and status.strip().lower() == DELETED_STATUS.lower()
+
+
 class ElementClient:
     """Программный клиент Console API v2."""
 
@@ -167,11 +182,14 @@ class ElementClient:
         """Карточка приложения (статус, uri, source.project-version и др.)."""
         return self._api("GET", f"/applications/{app_id}")
 
-    def find_app(self, name):
+    def find_app(self, name, *, include_deleted=False):
         """Найти приложение по точному совпадению имени без учёта регистра.
 
         Имя сверяется с полями name, display-name и publication-context.
-        Возвращается карточка из списка либо None.
+        Удалённые приложения (статус Deleted) по умолчанию пропускаются: на их
+        прежнем id последующие get и deploy отвечают 404. include_deleted=True
+        возвращает прежнее поведение – поиск среди всех приложений, включая
+        удалённые. Возвращается карточка из списка либо None.
         """
         target = (name or "").strip().lower()
         if not target:
@@ -179,6 +197,8 @@ class ElementClient:
         for app in self.list_apps():
             if not isinstance(app, dict):
                 continue
+            if not include_deleted and _is_deleted(app):
+                continue
             for key in ("name", "display-name", "publication-context"):
                 value = app.get(key)
                 if isinstance(value, str) and value.strip().lower() == target:

{elemctl-0.2.0 → elemctl-0.3.0}/src/elemctl/mcp_server.py

@@ -60,22 +60,24 @@ def create_server(config=None):
         return client().get_app(app_id)
 
     @server.tool()
-    def find_app(name: str) -> dict:
-        """Найти приложение по точному имени без учёта регистра; вернуть id и признак found."""
-        app = client().find_app(name)
+    def find_app(name: str, include_deleted: bool = False) -> dict:
+        """Найти приложение по точному имени без учёта регистра; вернуть id и признак found.
+
+        Удалённые приложения (статус Deleted) по умолчанию пропускаются: на их
+        прежнем id get и deploy отвечают 404. include_deleted=True ищет среди
+        всех приложений, включая удалённые.
+        """
+        app = client().find_app(name, include_deleted=include_deleted)
         if app is None:
             return {"id": None, "found": False}
         return {"id": app.get("id"), "found": True, "application": app}
 
-    @server.tool()
-    def create_app(
-        name: str,
-        project_id: str = "",
-        version_id: str = "",
-        space_id: str = "",
-        development_mode: bool = True,
-    ) -> dict:
-        """Создать приложение. При задании только project_id источником берётся последняя сборка проекта (создание из проекта целиком может дать пустой каркас)."""
+    def _create_app(name, project_id, version_id, space_id, development_mode):
+        """Создать приложение (общая логика create_app и ensure_app).
+
+        Источник – version_id (id сборки) либо последняя сборка project_id
+        (создание из проекта целиком может дать пустой каркас).
+        """
         source_version_id = version_id
         if not source_version_id:
             if not project_id:
@@ -93,6 +95,39 @@ def create_server(config=None):
             space_id=space_id or None,
         )
 
+    @server.tool()
+    def create_app(
+        name: str,
+        project_id: str = "",
+        version_id: str = "",
+        space_id: str = "",
+        development_mode: bool = True,
+    ) -> dict:
+        """Создать приложение. При задании только project_id источником берётся последняя сборка проекта (создание из проекта целиком может дать пустой каркас)."""
+        return _create_app(name, project_id, version_id, space_id, development_mode)
+
+    @server.tool()
+    def ensure_app(
+        name: str,
+        project_id: str = "",
+        version_id: str = "",
+        space_id: str = "",
+        development_mode: bool = True,
+    ) -> dict:
+        """Идемпотентно создать приложение по имени, если его ещё нет.
+
+        Существующее приложение НЕ пересоздаётся: при наличии возвращается
+        {"id": ..., "created": false} без изменений (delete + create дали бы
+        новый URL и порвали внешние привязки – OIDC redirect и т.п.). Удалённые
+        приложения (статус Deleted) не в счёт. Параметры создания – как у
+        create_app; они действуют, только когда создание происходит.
+        """
+        existing = client().find_app(name)
+        if existing is not None:
+            return {"id": existing.get("id"), "created": False}
+        card = _create_app(name, project_id, version_id, space_id, development_mode)
+        return {"id": (card or {}).get("id"), "created": True}
+
     @server.tool()
     def start_app(app_id: str) -> dict:
         """Запустить приложение."""

{elemctl-0.2.0 → elemctl-0.3.0}/src/elemctl.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: elemctl
-Version: 0.2.0
+Version: 0.3.0
 Summary: CLI, MCP-сервер и библиотека для Console API v2 платформы 1С:Предприятие.Элемент (1cmycloud)
 Author: KeyFire
 License-Expression: MIT
@@ -112,6 +112,9 @@ elemctl apps list
 # карточка приложения (статус, uri, фактическая версия проекта)
 elemctl apps get <app-id>
 
+# создать приложение, только если его ещё нет: {"id": ..., "created": true|false}
+elemctl apps ensure acme-crm-dev --project-id <project-id> --latest-build --wait
+
 # полный цикл деплоя из исходников с проверкой применения
 elemctl deploy --app-id <app-id> --project-id <project-id> --project-dir acme/crm
 
@@ -184,6 +187,10 @@ Assembly.yaml            # манифест: ProjectKind, Vendor, Name, Version,
   `elemctl apps create <имя> --project-id <id> --latest-build` (MCP-инструмент
   `create_app` подставляет последнюю сборку автоматически), а после создания -
   `elemctl deploy`.
+- Удалённые приложения остаются в списке платформы со статусом `Deleted` и
+  прежним `id`, на котором `apps get` и `deploy` отвечают 404. `apps find` и
+  `apps ensure` их пропускают; вернуть прежний поиск - `apps find
+  --include-deleted`.
 - Приложение с неопубликованными правками в среде разработки платформа
   удалить не даёт (HTTP 400 `FAILED_PRECONDITION`), принудительного удаления
   в Console API нет - только через панель управления; elemctl подскажет это

{elemctl-0.2.0 → elemctl-0.3.0}/src/elemctl.egg-info/SOURCES.txt

@@ -7,6 +7,7 @@ README.md
 pyproject.toml
 docs/SPEC.md
 src/elemctl/__init__.py
+src/elemctl/__main__.py
 src/elemctl/auth.py
 src/elemctl/build.py
 src/elemctl/cli.py

elemctl-0.3.0/tests/test_cli.py

@@ -0,0 +1,259 @@
+"""Тесты CLI: локальные команды и коды возврата (без сети)."""
+
+from __future__ import annotations
+
+import json
+import os
+import subprocess
+import sys
+from pathlib import Path
+
+import pytest
+
+import elemctl
+from elemctl import cli
+from elemctl.errors import ApiError
+
+
+@pytest.fixture(autouse=True)
+def clean_environment(monkeypatch, tmp_path):
+    """Изолировать тесты от переменных окружения и .env разработчика."""
+    for key in (
+        "ELEMENT_BASE_URL",
+        "ELEMENT_CLIENT_ID",
+        "ELEMENT_CLIENT_SECRET",
+        "ELEMENT_APP_ID",
+        "ELEMENT_PROJECT_ID",
+        "ELEMENT_SPACE_ID",
+    ):
+        monkeypatch.delenv(key, raising=False)
+    monkeypatch.chdir(tmp_path)
+
+
+def test_version_flag(capsys):
+    with pytest.raises(SystemExit) as excinfo:
+        cli.main(["--version"])
+    assert excinfo.value.code == 0
+    assert f"elemctl {elemctl.__version__}" in capsys.readouterr().out
+
+
+def test_module_entry_point():
+    """python -m elemctl – запасной путь для вызывающих без консольной точки входа в PATH."""
+    import_root = Path(elemctl.__file__).resolve().parent.parent
+    env = {**os.environ, "PYTHONPATH": str(import_root)}
+    result = subprocess.run([sys.executable, "-m", "elemctl", "--version"],
+                            capture_output=True, text=True, env=env)
+    assert result.returncode == 0, result.stderr
+    assert f"elemctl {elemctl.__version__}" in result.stdout
+
+
+def test_build_command(project_factory, tmp_path, capsys):
+    project_dir = project_factory()
+    out_dir = tmp_path / "dist"
+
+    rc = cli.main(
+        ["build", "--project-dir", str(project_dir), "--output", str(out_dir), "--branch", "", "--commit", ""]
+    )
+
+    assert rc == 0
+    payload = json.loads(capsys.readouterr().out)
+    archive = Path(payload["file"])
+    assert archive.exists()
+    assert archive.name == "crm 1.0-1.xasm"
+
+
+def test_deploy_dry_run_builds_only(project_factory, tmp_path, capsys):
+    project_dir = project_factory()
+    rc = cli.main(
+        [
+            "deploy",
+            "--dry-run",
+            "--project-dir",
+            str(project_dir),
+            "--output",
+            str(tmp_path / "dist"),
+        ]
+    )
+    assert rc == 0
+    payload = json.loads(capsys.readouterr().out)
+    assert Path(payload["file"]).exists()
+
+
+def test_apps_find_found_and_not_found(monkeypatch, capsys):
+    class FakeClient:
+        def find_app(self, name, *, include_deleted=False):
+            if name == "site-dev":
+                return {"id": "app-42", "display-name": "site-dev"}
+            return None
+
+    monkeypatch.setattr(cli, "make_client", lambda config: FakeClient())
+
+    rc = cli.main(["apps", "find", "site-dev"])
+    assert rc == 0
+    assert json.loads(capsys.readouterr().out) == {"id": "app-42", "found": True}
+
+    # Отсутствие приложения – штатный ответ: код возврата 0, признак несёт поле found.
+    rc = cli.main(["apps", "find", "нет-такого"])
+    assert rc == 0
+    assert json.loads(capsys.readouterr().out) == {"id": None, "found": False}
+
+
+def test_apps_find_request_failure_is_an_error(monkeypatch, capsys):
+    """Сбой запроса отличим от "не найдено": ненулевой код и error в stderr."""
+
+    class FailingClient:
+        def find_app(self, name, *, include_deleted=False):
+            raise ApiError("нет доступа", status=403)
+
+    monkeypatch.setattr(cli, "make_client", lambda config: FailingClient())
+
+    rc = cli.main(["apps", "find", "site-dev"])
+    assert rc == 1
+    captured = capsys.readouterr()
+    assert captured.out == ""
+    assert "error" in json.loads(captured.err)
+
+
+def test_apps_find_skips_deleted_unless_flag(monkeypatch, capsys):
+    """По умолчанию удалённое приложение не находится; --include-deleted его возвращает."""
+
+    class FakeClient:
+        def find_app(self, name, *, include_deleted=False):
+            # Приложение есть в списке платформы, но удалено: без флага его нет.
+            if name == "site-old" and include_deleted:
+                return {"id": "app-del", "display-name": "site-old", "status": "Deleted"}
+            return None
+
+    monkeypatch.setattr(cli, "make_client", lambda config: FakeClient())
+
+    rc = cli.main(["apps", "find", "site-old"])
+    assert rc == 0
+    assert json.loads(capsys.readouterr().out) == {"id": None, "found": False}
+
+    rc = cli.main(["apps", "find", "site-old", "--include-deleted"])
+    assert rc == 0
+    assert json.loads(capsys.readouterr().out) == {"id": "app-del", "found": True}
+
+
+def test_apps_ensure_existing_returns_created_false_without_creating(monkeypatch, capsys):
+    """Существующее приложение не пересоздаётся: created=false, create_app не вызывается."""
+
+    class FakeClient:
+        def find_app(self, name, *, include_deleted=False):
+            return {"id": "app-7", "display-name": name}
+
+        def create_app(self, *args, **kwargs):
+            raise AssertionError("создание не должно вызываться для существующего приложения")
+
+    monkeypatch.setattr(cli, "make_client", lambda config: FakeClient())
+
+    rc = cli.main(["apps", "ensure", "site-dev", "--version-id", "asm-1"])
+    assert rc == 0
+    assert json.loads(capsys.readouterr().out) == {"id": "app-7", "created": False}
+
+
+def test_apps_ensure_missing_creates_and_returns_created_true(monkeypatch, capsys):
+    """Отсутствующее приложение создаётся: created=true, источником идёт указанная сборка."""
+
+    class FakeClient:
+        def find_app(self, name, *, include_deleted=False):
+            return None
+
+        def create_app(
+            self,
+            display_name,
+            *,
+            project_version_id=None,
+            image_id=None,
+            development_mode=True,
+            space_id=None,
+            technology_version=None,
+        ):
+            assert display_name == "site-new"
+            assert project_version_id == "asm-9"
+            assert image_id is None
+            return {"id": "app-new", "display-name": display_name}
+
+    monkeypatch.setattr(cli, "make_client", lambda config: FakeClient())
+
+    rc = cli.main(["apps", "ensure", "site-new", "--version-id", "asm-9"])
+    assert rc == 0
+    assert json.loads(capsys.readouterr().out) == {"id": "app-new", "created": True}
+
+
+def test_apps_ensure_request_failure_is_an_error(monkeypatch, capsys):
+    """Сбой запроса в ensure: код возврата 1, пустой stdout, error в stderr."""
+
+    class FailingClient:
+        def find_app(self, name, *, include_deleted=False):
+            raise ApiError("нет доступа", status=403)
+
+    monkeypatch.setattr(cli, "make_client", lambda config: FailingClient())
+
+    rc = cli.main(["apps", "ensure", "site-dev"])
+    assert rc == 1
+    captured = capsys.readouterr()
+    assert captured.out == ""
+    assert "error" in json.loads(captured.err)
+
+
+def test_error_is_json_on_stderr(capsys):
+    # app-id не задан ни аргументом, ни конфигурацией.
+    rc = cli.main(["apps", "get"])
+    assert rc == 1
+    captured = capsys.readouterr()
+    payload = json.loads(captured.err)
+    assert "error" in payload
+    assert "app-id" in payload["error"]
+
+
+def test_deploy_exit_code_reflects_ok(monkeypatch, capsys, project_factory, tmp_path):
+    from tests.test_deploy import FakeDeployClient
+
+    # Успех: применённая версия совпала.
+    monkeypatch.setattr(
+        cli, "make_client", lambda config: FakeDeployClient(applied_version="1.0-1")
+    )
+    rc = cli.main(
+        [
+            "deploy",
+            "--app-id",
+            "app-1",
+            "--project-id",
+            "proj-1",
+            "--project-dir",
+            str(project_factory(repo_name="repo-ok")),
+            "--output",
+            str(tmp_path / "d1"),
+            "--build-version",
+            "1.0-1",
+        ]
+    )
+    captured = capsys.readouterr()
+    assert rc == 0
+    assert json.loads(captured.out)["ok"] is True
+
+    # Откат: версия не совпала – код возврата 1.
+    monkeypatch.setattr(
+        cli, "make_client", lambda config: FakeDeployClient(applied_version="1.0-0")
+    )
+    rc = cli.main(
+        [
+            "deploy",
+            "--app-id",
+            "app-1",
+            "--project-id",
+            "proj-1",
+            "--project-dir",
+            str(project_factory(repo_name="repo-fail")),
+            "--output",
+            str(tmp_path / "d2"),
+            "--build-version",
+            "1.0-1",
+        ]
+    )
+    captured = capsys.readouterr()
+    assert rc == 1
+    report = json.loads(captured.out)
+    assert report["ok"] is False
+    assert report["applied"] is False

{elemctl-0.2.0 → elemctl-0.3.0}/tests/test_client.py

@@ -63,6 +63,34 @@ def test_find_app_exact_case_insensitive(api):
     assert client.find_app("nope") is None
 
 
+def test_find_app_skips_deleted(api):
+    client, transport = api
+    transport.add(
+        "GET",
+        f"{API}/applications",
+        [
+            {"id": "old", "display-name": "site", "status": "Deleted"},
+            {"id": "live", "display-name": "site", "status": "Running"},
+        ],
+    )
+    # Удалённая карточка пропускается – находится живое приложение с тем же именем.
+    assert client.find_app("site")["id"] == "live"
+
+
+def test_find_app_include_deleted_returns_deleted(api):
+    client, transport = api
+    # Единственный ответ повторяется на оба вызова find_app.
+    transport.add(
+        "GET",
+        f"{API}/applications",
+        [{"id": "old", "display-name": "site-old", "status": "DELETED"}],
+    )
+    # По умолчанию удалённое не находится...
+    assert client.find_app("site-old") is None
+    # ...а с include_deleted=True возвращается прежнее поведение (сравнение статуса без учёта регистра).
+    assert client.find_app("site-old", include_deleted=True)["id"] == "old"
+
+
 def test_delete_failed_precondition_hint(api):
     client, transport = api
     transport.add(

{elemctl-0.2.0 → elemctl-0.3.0}/tests/test_mcp.py

@@ -15,6 +15,7 @@ EXPECTED_TOOLS = {
     "get_app",
     "find_app",
     "create_app",
+    "ensure_app",
     "start_app",
     "stop_app",
     "delete_app",
@@ -49,3 +50,20 @@ def test_delete_app_docstring_warns():
     tools = asyncio.run(server.list_tools())
     delete_tool = next(tool for tool in tools if tool.name == "delete_app")
     assert "URL" in (delete_tool.description or "")
+
+
+def test_ensure_app_docstring_states_no_recreate():
+    server = create_server()
+    tools = asyncio.run(server.list_tools())
+    ensure_tool = next(tool for tool in tools if tool.name == "ensure_app")
+    description = ensure_tool.description or ""
+    assert "пересозда" in description  # существующее приложение НЕ пересоздаётся
+    assert "created" in description
+
+
+def test_find_app_exposes_include_deleted():
+    server = create_server()
+    tools = asyncio.run(server.list_tools())
+    find_tool = next(tool for tool in tools if tool.name == "find_app")
+    properties = (find_tool.inputSchema or {}).get("properties") or {}
+    assert "include_deleted" in properties

elemctl-0.2.0/tests/test_cli.py

@@ -1,144 +0,0 @@
-"""Тесты CLI: локальные команды и коды возврата (без сети)."""
-
-from __future__ import annotations
-
-import json
-from pathlib import Path
-
-import pytest
-
-from elemctl import cli
-
-
-@pytest.fixture(autouse=True)
-def clean_environment(monkeypatch, tmp_path):
-    """Изолировать тесты от переменных окружения и .env разработчика."""
-    for key in (
-        "ELEMENT_BASE_URL",
-        "ELEMENT_CLIENT_ID",
-        "ELEMENT_CLIENT_SECRET",
-        "ELEMENT_APP_ID",
-        "ELEMENT_PROJECT_ID",
-        "ELEMENT_SPACE_ID",
-    ):
-        monkeypatch.delenv(key, raising=False)
-    monkeypatch.chdir(tmp_path)
-
-
-def test_version_flag(capsys):
-    with pytest.raises(SystemExit) as excinfo:
-        cli.main(["--version"])
-    assert excinfo.value.code == 0
-    assert "elemctl 0.2.0" in capsys.readouterr().out
-
-
-def test_build_command(project_factory, tmp_path, capsys):
-    project_dir = project_factory()
-    out_dir = tmp_path / "dist"
-
-    rc = cli.main(
-        ["build", "--project-dir", str(project_dir), "--output", str(out_dir), "--branch", "", "--commit", ""]
-    )
-
-    assert rc == 0
-    payload = json.loads(capsys.readouterr().out)
-    archive = Path(payload["file"])
-    assert archive.exists()
-    assert archive.name == "crm 1.0-1.xasm"
-
-
-def test_deploy_dry_run_builds_only(project_factory, tmp_path, capsys):
-    project_dir = project_factory()
-    rc = cli.main(
-        [
-            "deploy",
-            "--dry-run",
-            "--project-dir",
-            str(project_dir),
-            "--output",
-            str(tmp_path / "dist"),
-        ]
-    )
-    assert rc == 0
-    payload = json.loads(capsys.readouterr().out)
-    assert Path(payload["file"]).exists()
-
-
-def test_apps_find_found_and_not_found(monkeypatch, capsys):
-    class FakeClient:
-        def find_app(self, name):
-            if name == "site-dev":
-                return {"id": "app-42", "display-name": "site-dev"}
-            return None
-
-    monkeypatch.setattr(cli, "make_client", lambda config: FakeClient())
-
-    rc = cli.main(["apps", "find", "site-dev"])
-    assert rc == 0
-    assert json.loads(capsys.readouterr().out) == {"id": "app-42", "found": True}
-
-    rc = cli.main(["apps", "find", "нет-такого"])
-    assert rc == 1
-    assert json.loads(capsys.readouterr().out) == {"id": None, "found": False}
-
-
-def test_error_is_json_on_stderr(capsys):
-    # app-id не задан ни аргументом, ни конфигурацией.
-    rc = cli.main(["apps", "get"])
-    assert rc == 1
-    captured = capsys.readouterr()
-    payload = json.loads(captured.err)
-    assert "error" in payload
-    assert "app-id" in payload["error"]
-
-
-def test_deploy_exit_code_reflects_ok(monkeypatch, capsys, project_factory, tmp_path):
-    from tests.test_deploy import FakeDeployClient
-
-    # Успех: применённая версия совпала.
-    monkeypatch.setattr(
-        cli, "make_client", lambda config: FakeDeployClient(applied_version="1.0-1")
-    )
-    rc = cli.main(
-        [
-            "deploy",
-            "--app-id",
-            "app-1",
-            "--project-id",
-            "proj-1",
-            "--project-dir",
-            str(project_factory(repo_name="repo-ok")),
-            "--output",
-            str(tmp_path / "d1"),
-            "--build-version",
-            "1.0-1",
-        ]
-    )
-    captured = capsys.readouterr()
-    assert rc == 0
-    assert json.loads(captured.out)["ok"] is True
-
-    # Откат: версия не совпала – код возврата 1.
-    monkeypatch.setattr(
-        cli, "make_client", lambda config: FakeDeployClient(applied_version="1.0-0")
-    )
-    rc = cli.main(
-        [
-            "deploy",
-            "--app-id",
-            "app-1",
-            "--project-id",
-            "proj-1",
-            "--project-dir",
-            str(project_factory(repo_name="repo-fail")),
-            "--output",
-            str(tmp_path / "d2"),
-            "--build-version",
-            "1.0-1",
-        ]
-    )
-    captured = capsys.readouterr()
-    assert rc == 1
-    report = json.loads(captured.out)
-    assert report["ok"] is False
-    assert report["applied"] is False