savetest-cli 0.3.0rc3__tar.gz → 0.3.0rc4__tar.gz

{savetest_cli-0.3.0rc3 → savetest_cli-0.3.0rc4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: savetest-cli
-Version: 0.3.0rc3
+Version: 0.3.0rc4
 Summary: CLI for SaveTest: upload Allure results as a ZIP, as raw files, or by watching a directory
 License-Expression: MIT
 Requires-Python: >=3.8
@@ -196,6 +196,8 @@ savetest upload_files ./allure-results/*-result.json ./allure-results/executor.j
 
 Первая пачка создаёт тест-ран, если не передан `--test-run-id`. Следующие файлы пишутся в тот же ран: пока ран не создан, вторая пачка ждёт. Код возврата `1`, если хотя бы один импорт не завершился как `completed`.
 
+Пачка уходит одним запросом, статусы задач в очереди тоже читаются одним запросом. Если сервер ещё старый и отвечает 404 или 405, до конца этого запуска watch сам возвращается к прежней схеме: три запроса на пачку и опрос задач по одной, с головы очереди до первой незавершённой. Уже выпущенные версии утилиты с новым сервером работают как раньше.
+
 Вложение уезжает вместе с `*-result.json`, который на него ссылается. Результат подождёт вложение до `--attachment-wait` секунд. Пачка без файла результата не отправляется.
 
 Привязки к фреймворку нет. После `--` watch запускает любую команду как есть и не разбирает её аргументы: pytest, JUnit, Maven, Gradle, npm или свой скрипт. Ему всё равно, кто пишет файлы. Нужны только `*-result.json` в каталоге, который указан первым аргументом. `pytest` в примерах ниже — один из вариантов, не требование.
@@ -220,7 +222,7 @@ savetest watch ./allure-results \
 - Если тесты и загрузка — разные джобы, отдайте каталог `allure-results` артефактом тоже с `when: always`.
 - `--wait` оставляйте включённым: джоба должна завершиться только когда импорт дошёл до конца.
 - Вложения включайте только если они реально нужны. По умолчанию CLI их не шлёт.
-- `0.3.0rc3` — предварительная версия для SaveTest 1.5.0: `pip install savetest-cli==0.3.0rc3`. Обычный `pip install savetest-cli` остаётся на последней стабильной версии.
+- `0.3.0rc4` — предварительная версия для SaveTest 1.5.0: `pip install savetest-cli==0.3.0rc4`. Обычный `pip install savetest-cli` остаётся на последней стабильной версии.
 
 ### Результаты по мере прогона
 

savetest_cli-0.3.0rc3/savetest_cli.egg-info/PKG-INFO → savetest_cli-0.3.0rc4/README.md

@@ -1,13 +1,3 @@
-Metadata-Version: 2.4
-Name: savetest-cli
-Version: 0.3.0rc3
-Summary: CLI for SaveTest: upload Allure results as a ZIP, as raw files, or by watching a directory
-License-Expression: MIT
-Requires-Python: >=3.8
-Description-Content-Type: text/markdown
-Requires-Dist: click>=8.0
-Requires-Dist: requests>=2.28
-
 <p align="center">
   <img src="savetest_cli/pwa-512x512.png" alt="SaveTest" width="128">
 </p>
@@ -196,6 +186,8 @@ savetest upload_files ./allure-results/*-result.json ./allure-results/executor.j
 
 Первая пачка создаёт тест-ран, если не передан `--test-run-id`. Следующие файлы пишутся в тот же ран: пока ран не создан, вторая пачка ждёт. Код возврата `1`, если хотя бы один импорт не завершился как `completed`.
 
+Пачка уходит одним запросом, статусы задач в очереди тоже читаются одним запросом. Если сервер ещё старый и отвечает 404 или 405, до конца этого запуска watch сам возвращается к прежней схеме: три запроса на пачку и опрос задач по одной, с головы очереди до первой незавершённой. Уже выпущенные версии утилиты с новым сервером работают как раньше.
+
 Вложение уезжает вместе с `*-result.json`, который на него ссылается. Результат подождёт вложение до `--attachment-wait` секунд. Пачка без файла результата не отправляется.
 
 Привязки к фреймворку нет. После `--` watch запускает любую команду как есть и не разбирает её аргументы: pytest, JUnit, Maven, Gradle, npm или свой скрипт. Ему всё равно, кто пишет файлы. Нужны только `*-result.json` в каталоге, который указан первым аргументом. `pytest` в примерах ниже — один из вариантов, не требование.
@@ -220,7 +212,7 @@ savetest watch ./allure-results \
 - Если тесты и загрузка — разные джобы, отдайте каталог `allure-results` артефактом тоже с `when: always`.
 - `--wait` оставляйте включённым: джоба должна завершиться только когда импорт дошёл до конца.
 - Вложения включайте только если они реально нужны. По умолчанию CLI их не шлёт.
-- `0.3.0rc3` — предварительная версия для SaveTest 1.5.0: `pip install savetest-cli==0.3.0rc3`. Обычный `pip install savetest-cli` остаётся на последней стабильной версии.
+- `0.3.0rc4` — предварительная версия для SaveTest 1.5.0: `pip install savetest-cli==0.3.0rc4`. Обычный `pip install savetest-cli` остаётся на последней стабильной версии.
 
 ### Результаты по мере прогона
 

{savetest_cli-0.3.0rc3 → savetest_cli-0.3.0rc4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "savetest-cli"
-version = "0.3.0rc3"
+version = "0.3.0rc4"
 description = "CLI for SaveTest: upload Allure results as a ZIP, as raw files, or by watching a directory"
 readme = "README.md"
 requires-python = ">=3.8"

savetest_cli-0.3.0rc4/savetest_cli/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.3.0rc4"

{savetest_cli-0.3.0rc3 → savetest_cli-0.3.0rc4}/savetest_cli/cli.py

@@ -537,17 +537,24 @@ def _is_result_file(path: Path) -> bool:
     return name.endswith("-result.json") or (name.startswith("test-result-") and name.endswith(".json"))
 
 
+def _raw_file_name_reason(name: str, *, include_attachments: bool) -> Optional[str]:
+    """Причина пропустить файл по одному имени, без stat."""
+    if name.startswith(".") or name.lower() in _JUNK_NAMES or name == "__MACOSX":
+        return "служебный файл"
+    if name.lower().endswith(".zip"):
+        return "ZIP загружается командой upload"
+    if not include_attachments and _is_attachment(name):
+        return "вложение"
+    return None
+
+
 def _skip_raw_file(path: Path, *, include_attachments: bool) -> Optional[str]:
     """None — файл берём. Иначе короткая причина пропуска."""
     if not path.is_file():
         return "не файл"
-    if "__MACOSX" in path.parts or path.name.startswith(".") or path.name.lower() in _JUNK_NAMES:
+    if "__MACOSX" in path.parts:
         return "служебный файл"
-    if path.suffix.lower() == ".zip":
-        return "ZIP загружается командой upload"
-    if not include_attachments and _is_attachment(path.name):
-        return "вложение"
-    return None
+    return _raw_file_name_reason(path.name, include_attachments=include_attachments)
 
 
 def _collect_directory(root: Path, include_attachments: bool) -> list:
@@ -835,18 +842,50 @@ def _load_json_if_complete(path: Path) -> Optional[Any]:
         return None
 
 
-def _snapshot_raw_files(root: Path, include_attachments: bool) -> Dict[str, Tuple[Path, int, int]]:
+def _snapshot_raw_files(
+    root: Path,
+    include_attachments: bool,
+    uploaded: Optional[set] = None,
+) -> Dict[str, Tuple[Path, int, int]]:
+    """Файлы каталога, которые ещё не отправлены.
+
+    Уже отправленные (rel in uploaded) не stat'ятся и в снимок не попадают.
+    Имя отсекается до stat. Ключ — относительный posix-путь.
+    """
     found: Dict[str, Tuple[Path, int, int]] = {}
-    if not root.is_dir():
-        return found
-    for file_path in root.rglob("*"):
-        if _skip_raw_file(file_path, include_attachments=include_attachments):
-            continue
+    done = uploaded or ()
+
+    def walk(directory: str, prefix: str) -> None:
         try:
-            stat = file_path.stat()
+            iterator = os.scandir(directory)
         except OSError:
-            continue
-        found[file_path.relative_to(root).as_posix()] = (file_path, stat.st_size, stat.st_mtime_ns)
+            return
+        with iterator:
+            for entry in iterator:
+                name = entry.name
+                rel = f"{prefix}/{name}" if prefix else name
+                if rel in done:
+                    continue
+                try:
+                    if entry.is_dir(follow_symlinks=False):
+                        if name != "__MACOSX":
+                            walk(entry.path, rel)
+                        continue
+                except OSError:
+                    continue
+                if _raw_file_name_reason(name, include_attachments=include_attachments):
+                    continue
+                if "__MACOSX" in rel.split("/"):
+                    continue
+                try:
+                    if not entry.is_file(follow_symlinks=False):
+                        continue
+                    file_stat = entry.stat(follow_symlinks=False)
+                except OSError:
+                    continue
+                found[rel] = (Path(entry.path), file_stat.st_size, file_stat.st_mtime_ns)
+
+    walk(os.fspath(root), "")
     return found
 
 
@@ -951,12 +990,23 @@ def _chunk_watch_selection(
     return chunks
 
 
-def _enqueue_watch_batch(
-    selected: List[Tuple[str, Path]],
+class _WatchApi:
+    """Какие быстрые ручки ещё пробуем в этом запуске watch.
+
+    404/405 значит, что сервер старый: до конца запуска остаётся прежний путь.
+    """
+
+    def __init__(self) -> None:
+        self.files_batch = True
+        self.tasks_batch = True
+
+
+_TASK_STATUS_BATCH_LIMIT = 500
+_MISSING_ROUTE = frozenset({404, 405})
+
+
+def _watch_form_data(
     *,
-    host: str,
-    token: str,
-    project_id: str,
     test_run_id: Optional[str],
     test_run_name: Optional[str],
     description: Optional[str],
@@ -969,18 +1019,17 @@ def _enqueue_watch_batch(
     configuration_name: Optional[str],
     branch: Optional[str],
     include_attachments: bool,
-) -> str:
-    headers = {"Authorization": f"Bearer {token}"}
-    init_data: Dict[str, str] = {
+) -> Dict[str, str]:
+    data: Dict[str, str] = {
         "include_attachments": "true" if include_attachments else "false",
     }
     if branch:
-        init_data["git_branch"] = branch
+        data["git_branch"] = branch
     if test_run_id:
-        init_data["test_run_id"] = str(test_run_id)
+        data["test_run_id"] = str(test_run_id)
     else:
         _append_new_test_run_fields(
-            init_data,
+            data,
             test_run_name=test_run_name,
             description=description,
             version=version,
@@ -991,7 +1040,58 @@ def _enqueue_watch_batch(
             configuration_id=configuration_id,
             configuration_name=configuration_name,
         )
+    return data
+
+
+def _require_task_id(payload: Dict[str, Any]) -> str:
+    task_id = payload.get("task_id")
+    if not task_id:
+        click.echo("Файлы приняты, но task_id не получен.", err=True)
+        sys.exit(1)
+    return str(task_id)
 
+
+def _try_watch_files_batch(
+    selected: List[Tuple[str, Path]],
+    *,
+    host: str,
+    project_id: str,
+    headers: Dict[str, str],
+    init_data: Dict[str, str],
+    api: _WatchApi,
+) -> Optional[str]:
+    """task_id новой ручки. None — сервер её не знает, флаг уже снят."""
+    if not api.files_batch:
+        return None
+    url = f"{host}/api/v1/projects/{project_id}/allure-results/files/batch"
+    handles = []
+    multipart = []
+    try:
+        for rel, path in selected:
+            handle = open(path, "rb")
+            handles.append(handle)
+            multipart.append(("files", (path.name, handle, "application/octet-stream")))
+            multipart.append(("relative_paths", (None, rel)))
+        resp = requests.post(url, headers=headers, data=init_data, files=multipart, timeout=180)
+    finally:
+        for handle in handles:
+            handle.close()
+    if resp.status_code in _MISSING_ROUTE:
+        api.files_batch = False
+        click.echo("Сервер без пакетной загрузки, отправляю пачку прежним способом.")
+        return None
+    _check_response(resp, "watch batch")
+    return _require_task_id(resp.json())
+
+
+def _enqueue_watch_batch_steps(
+    selected: List[Tuple[str, Path]],
+    *,
+    host: str,
+    project_id: str,
+    headers: Dict[str, str],
+    init_data: Dict[str, str],
+) -> str:
     resp = requests.post(
         f"{host}/api/v1/projects/{project_id}/allure-results/files/init",
         headers=headers,
@@ -1018,11 +1118,64 @@ def _enqueue_watch_batch(
 
     resp = requests.post(f"{url}/complete", headers=headers, timeout=60)
     _check_response(resp, "watch complete")
-    task_id = resp.json().get("task_id")
-    if not task_id:
-        click.echo("Файлы приняты, но task_id не получен.", err=True)
-        sys.exit(1)
-    return str(task_id)
+    return _require_task_id(resp.json())
+
+
+def _enqueue_watch_batch(
+    selected: List[Tuple[str, Path]],
+    *,
+    host: str,
+    token: str,
+    project_id: str,
+    test_run_id: Optional[str],
+    test_run_name: Optional[str],
+    description: Optional[str],
+    version: Optional[str],
+    revision: Optional[str],
+    test_object: Optional[str],
+    epic_id: Optional[str],
+    epic_name: Optional[str],
+    configuration_id: Optional[str],
+    configuration_name: Optional[str],
+    branch: Optional[str],
+    include_attachments: bool,
+    api: Optional[_WatchApi] = None,
+) -> str:
+    if api is None:
+        api = _WatchApi()
+    headers = {"Authorization": f"Bearer {token}"}
+    init_data = _watch_form_data(
+        test_run_id=test_run_id,
+        test_run_name=test_run_name,
+        description=description,
+        version=version,
+        revision=revision,
+        test_object=test_object,
+        epic_id=epic_id,
+        epic_name=epic_name,
+        configuration_id=configuration_id,
+        configuration_name=configuration_name,
+        branch=branch,
+        include_attachments=include_attachments,
+    )
+    if api.files_batch:
+        task_id = _try_watch_files_batch(
+            selected,
+            host=host,
+            project_id=project_id,
+            headers=headers,
+            init_data=init_data,
+            api=api,
+        )
+        if task_id:
+            return task_id
+    return _enqueue_watch_batch_steps(
+        selected,
+        host=host,
+        project_id=project_id,
+        headers=headers,
+        init_data=init_data,
+    )
 
 
 def _fetch_task(host: str, project_id: str, task_id: str, token: str) -> Dict[str, Any]:
@@ -1035,6 +1188,83 @@ def _fetch_task(host: str, project_id: str, task_id: str, token: str) -> Dict[st
     return resp.json()
 
 
+def _task_status_items(body: Any) -> List[Dict[str, Any]]:
+    if isinstance(body, list):
+        items = body
+    elif isinstance(body, dict) and isinstance(body.get("tasks"), list):
+        items = body["tasks"]
+    else:
+        items = []
+    return [item for item in items if isinstance(item, dict)]
+
+
+def _fetch_task_status_map(
+    task_ids: List[str],
+    *,
+    host: str,
+    project_id: str,
+    token: str,
+    api: _WatchApi,
+) -> Optional[Dict[str, Dict[str, Any]]]:
+    """Карта task_id -> статус. None — сервер без пакетной ручки."""
+    if not api.tasks_batch or not task_ids:
+        return None
+    found: Dict[str, Dict[str, Any]] = {}
+    headers = {"Authorization": f"Bearer {token}"}
+    url = f"{host}/api/v1/projects/{project_id}/allure-results/tasks/status"
+    for offset in range(0, len(task_ids), _TASK_STATUS_BATCH_LIMIT):
+        chunk = task_ids[offset:offset + _TASK_STATUS_BATCH_LIMIT]
+        resp = requests.post(url, headers=headers, json={"task_ids": chunk}, timeout=60)
+        if resp.status_code in _MISSING_ROUTE:
+            api.tasks_batch = False
+            click.echo("Сервер без пакетного статуса задач, опрашиваю по одной.")
+            return None
+        _check_response(resp, "watch tasks")
+        for item in _task_status_items(resp.json()):
+            task_id = item.get("task_id")
+            if task_id:
+                found[str(task_id)] = item
+    return found
+
+
+def _poll_watch_tasks(
+    inflight: List[str],
+    *,
+    host: str,
+    project_id: str,
+    token: str,
+    api: _WatchApi,
+) -> List[Tuple[str, Dict[str, Any]]]:
+    """Статусы inflight.
+
+    Новая ручка читает все сразу. Старый сервер — с головы до первой незавершённой.
+    """
+    if not inflight:
+        return []
+    batched = _fetch_task_status_map(
+        inflight,
+        host=host,
+        project_id=project_id,
+        token=token,
+        api=api,
+    )
+    if batched is not None:
+        pairs: List[Tuple[str, Dict[str, Any]]] = []
+        for task_id in inflight:
+            status = batched.get(task_id)
+            if status is None:
+                status = _fetch_task(host, project_id, task_id, token)
+            pairs.append((task_id, status))
+        return pairs
+    pairs = []
+    for task_id in inflight:
+        status = _fetch_task(host, project_id, task_id, token)
+        pairs.append((task_id, status))
+        if status.get("status", "") not in TERMINAL_STATUSES:
+            break
+    return pairs
+
+
 def _watch_session_url(host: str, project_id: str, session_id: Optional[str] = None, action: str = "") -> str:
     url = f"{host}/api/v1/projects/{project_id}/allure-results/watch/sessions"
     if session_id:
@@ -1152,7 +1382,9 @@ def watch(
     Готовые файлы не уходят по одному. Если они появляются подряд, watch держит их
     --batch-window секунд после последнего нового и отправляет вместе. Дольше
     --batch-max-wait секунд пачка не копится. Предел одной отправки — 200 файлов
-    или 32 МБ.
+    или 32 МБ. Если сервер умеет, пачка и статусы задач уходят одним запросом.
+    Ответ 404 или 405 — до конца этого запуска прежний путь: три запроса на пачку
+    и опрос задач с головы очереди до первой незавершённой.
     """
     command = [arg for arg in ctx.args if arg != "--"]
     host = host.rstrip("/")
@@ -1166,6 +1398,7 @@ def watch(
     had_failure = False
     run_create_failed = False
     sent_batches = 0
+    watch_api = _WatchApi()
     drain_until = 0.0
     announced_flush = False
     hold_opened = 0.0
@@ -1240,7 +1473,7 @@ def watch(
             timed_out = bool(timeout) and (now - started) >= timeout
             flushing = stop["requested"] or timed_out
 
-            snapshot = _snapshot_raw_files(root, include_attachments) if root.is_dir() else {}
+            snapshot = _snapshot_raw_files(root, include_attachments, uploaded)
             if flushing and not announced_flush:
                 pending = sum(1 for rel in snapshot if rel not in uploaded)
                 click.echo(f"После остановки осталось файлов: {pending}.")
@@ -1319,6 +1552,7 @@ def watch(
                         configuration_name=configuration_name,
                         branch=branch,
                         include_attachments=include_attachments,
+                        api=watch_api,
                     )
                     for rel, _path in chunk:
                         uploaded.add(rel)
@@ -1329,9 +1563,15 @@ def watch(
                 hold_opened = 0.0
                 hold_key = None
 
+            pairs = _poll_watch_tasks(
+                inflight,
+                host=host,
+                project_id=project_id,
+                token=token,
+                api=watch_api,
+            )
             still_running: List[str] = []
-            for task_id in inflight:
-                status = _fetch_task(host, project_id, task_id, token)
+            for task_id, status in pairs:
                 state = status.get("status", "")
                 if state not in TERMINAL_STATUSES:
                     still_running.append(task_id)
@@ -1359,6 +1599,8 @@ def watch(
                     if not known_run_id:
                         run_create_failed = True
                         click.echo("Тест-ран не создан, новые пачки не отправляю.", err=True)
+            if len(pairs) < len(inflight):
+                still_running.extend(inflight[len(pairs):])
             inflight = still_running
 
             if session_id and (
@@ -1434,3 +1676,7 @@ def watch(
             test_status = child["proc"].wait()
     if test_status or watch_code:
         sys.exit(test_status or watch_code)
+
+
+if __name__ == "__main__":
+    cli()

savetest_cli-0.3.0rc3/README.md → savetest_cli-0.3.0rc4/savetest_cli.egg-info/PKG-INFO

@@ -1,3 +1,13 @@
+Metadata-Version: 2.4
+Name: savetest-cli
+Version: 0.3.0rc4
+Summary: CLI for SaveTest: upload Allure results as a ZIP, as raw files, or by watching a directory
+License-Expression: MIT
+Requires-Python: >=3.8
+Description-Content-Type: text/markdown
+Requires-Dist: click>=8.0
+Requires-Dist: requests>=2.28
+
 <p align="center">
   <img src="savetest_cli/pwa-512x512.png" alt="SaveTest" width="128">
 </p>
@@ -186,6 +196,8 @@ savetest upload_files ./allure-results/*-result.json ./allure-results/executor.j
 
 Первая пачка создаёт тест-ран, если не передан `--test-run-id`. Следующие файлы пишутся в тот же ран: пока ран не создан, вторая пачка ждёт. Код возврата `1`, если хотя бы один импорт не завершился как `completed`.
 
+Пачка уходит одним запросом, статусы задач в очереди тоже читаются одним запросом. Если сервер ещё старый и отвечает 404 или 405, до конца этого запуска watch сам возвращается к прежней схеме: три запроса на пачку и опрос задач по одной, с головы очереди до первой незавершённой. Уже выпущенные версии утилиты с новым сервером работают как раньше.
+
 Вложение уезжает вместе с `*-result.json`, который на него ссылается. Результат подождёт вложение до `--attachment-wait` секунд. Пачка без файла результата не отправляется.
 
 Привязки к фреймворку нет. После `--` watch запускает любую команду как есть и не разбирает её аргументы: pytest, JUnit, Maven, Gradle, npm или свой скрипт. Ему всё равно, кто пишет файлы. Нужны только `*-result.json` в каталоге, который указан первым аргументом. `pytest` в примерах ниже — один из вариантов, не требование.
@@ -210,7 +222,7 @@ savetest watch ./allure-results \
 - Если тесты и загрузка — разные джобы, отдайте каталог `allure-results` артефактом тоже с `when: always`.
 - `--wait` оставляйте включённым: джоба должна завершиться только когда импорт дошёл до конца.
 - Вложения включайте только если они реально нужны. По умолчанию CLI их не шлёт.
-- `0.3.0rc3` — предварительная версия для SaveTest 1.5.0: `pip install savetest-cli==0.3.0rc3`. Обычный `pip install savetest-cli` остаётся на последней стабильной версии.
+- `0.3.0rc4` — предварительная версия для SaveTest 1.5.0: `pip install savetest-cli==0.3.0rc4`. Обычный `pip install savetest-cli` остаётся на последней стабильной версии.
 
 ### Результаты по мере прогона
 

{savetest_cli-0.3.0rc3 → savetest_cli-0.3.0rc4}/savetest_cli.egg-info/SOURCES.txt

@@ -9,4 +9,5 @@ savetest_cli.egg-info/SOURCES.txt
 savetest_cli.egg-info/dependency_links.txt
 savetest_cli.egg-info/entry_points.txt
 savetest_cli.egg-info/requires.txt
-savetest_cli.egg-info/top_level.txt
+savetest_cli.egg-info/top_level.txt
+tests/test_watch_fastpath.py

savetest_cli-0.3.0rc4/tests/test_watch_fastpath.py

@@ -0,0 +1,219 @@
+"""Снимок watch и фолбэк на старые ручки."""
+
+import os
+from pathlib import Path
+from unittest.mock import patch
+
+import pytest
+
+from savetest_cli.cli import (
+    _WatchApi,
+    _enqueue_watch_batch,
+    _poll_watch_tasks,
+    _snapshot_raw_files,
+)
+
+
+class _Resp:
+    def __init__(self, status_code, payload):
+        self.status_code = status_code
+        self._payload = payload
+        self.ok = 200 <= status_code < 300
+        self.text = payload if isinstance(payload, str) else ""
+
+    def json(self):
+        if isinstance(self._payload, str):
+            raise ValueError("not json")
+        return self._payload
+
+
+def _send(selected, api):
+    return _enqueue_watch_batch(
+        selected,
+        host="https://example.test",
+        token="tok",
+        project_id="7",
+        test_run_id=None,
+        test_run_name="Прогон",
+        description=None,
+        version=None,
+        revision=None,
+        test_object=None,
+        epic_id=None,
+        epic_name=None,
+        configuration_id=None,
+        configuration_name=None,
+        branch="main",
+        include_attachments=False,
+        api=api,
+    )
+
+
+def _kind(url: str) -> str:
+    if url.endswith("/files/batch"):
+        return "batch"
+    if url.endswith("/files/init"):
+        return "init"
+    if url.endswith("/complete"):
+        return "complete"
+    if "/allure-results/files/" in url:
+        return "upload"
+    return url
+
+
+def test_snapshot_does_not_stat_uploaded_files(tmp_path: Path):
+    (tmp_path / "sent-result.json").write_text("{}", encoding="utf-8")
+    (tmp_path / "new-result.json").write_text('{"status": "passed"}', encoding="utf-8")
+    nested = tmp_path / "sub" / "deep"
+    nested.mkdir(parents=True)
+    (nested / "nested-result.json").write_text("{}", encoding="utf-8")
+    (tmp_path / "sub" / "old-result.json").write_text("{}", encoding="utf-8")
+    (tmp_path / ".DS_Store").write_text("x", encoding="utf-8")
+    (tmp_path / "shot.png").write_text("x", encoding="utf-8")
+    (tmp_path / "pack.zip").write_text("x", encoding="utf-8")
+    macos = tmp_path / "__MACOSX"
+    macos.mkdir()
+    (macos / "junk-result.json").write_text("{}", encoding="utf-8")
+
+    uploaded = {"sent-result.json", "sub/old-result.json"}
+    uploaded_names = {"sent-result.json", "old-result.json"}
+    skipped_names = uploaded_names | {".DS_Store", "shot.png", "pack.zip", "junk-result.json"}
+    stat_names = []
+    real_dir_stat = os.DirEntry.stat
+    real_os_stat = os.stat
+
+    def dir_stat(self, *args, **kwargs):
+        stat_names.append(self.name)
+        if self.name in uploaded_names:
+            raise AssertionError(f"DirEntry.stat для уже отправленного {self.name}")
+        return real_dir_stat(self, *args, **kwargs)
+
+    def os_stat(path, *args, **kwargs):
+        base = os.path.basename(os.fspath(path))
+        if base in uploaded_names:
+            raise AssertionError(f"os.stat для уже отправленного {base}")
+        return real_os_stat(path, *args, **kwargs)
+
+    with patch.object(os.DirEntry, "stat", dir_stat), patch("os.stat", os_stat):
+        found = _snapshot_raw_files(tmp_path, False, uploaded)
+
+    assert set(found) == {"new-result.json", "sub/deep/nested-result.json"}
+    assert "new-result.json" in stat_names
+    assert "nested-result.json" in stat_names
+    assert not (set(stat_names) & skipped_names)
+    path, size, mtime = found["sub/deep/nested-result.json"]
+    assert path.name == "nested-result.json"
+    assert size > 0
+    assert mtime > 0
+    assert _snapshot_raw_files(tmp_path / "missing", False, set()) == {}
+
+
+def test_files_batch_404_falls_back_once(tmp_path: Path):
+    result = tmp_path / "a-result.json"
+    result.write_text("{}", encoding="utf-8")
+    selected = [("a-result.json", result)]
+    api = _WatchApi()
+    calls = []
+
+    def fake_post(url, **kwargs):
+        calls.append(_kind(url))
+        if url.endswith("/files/batch"):
+            assert kwargs["data"]["include_attachments"] == "false"
+            assert kwargs["data"]["git_branch"] == "main"
+            assert kwargs["data"]["test_run_name"] == "Прогон"
+            return _Resp(404, "missing")
+        if url.endswith("/files/init"):
+            return _Resp(200, {"upload_id": "up1"})
+        if url.endswith("/complete"):
+            return _Resp(200, {"task_id": "task-1", "status": "pending"})
+        if "/allure-results/files/" in url:
+            return _Resp(200, {"ok": True})
+        raise AssertionError(url)
+
+    with patch("savetest_cli.cli.requests.post", side_effect=fake_post):
+        assert _send(selected, api) == "task-1"
+        assert _send(selected, api) == "task-1"
+
+    assert calls == [
+        "batch", "init", "upload", "complete",
+        "init", "upload", "complete",
+    ]
+    assert api.files_batch is False
+
+
+def test_task_status_404_polls_until_first_open():
+    api = _WatchApi()
+    calls = []
+    states = {"a": "completed", "b": "processing", "c": "completed"}
+
+    def fake_post(url, **kwargs):
+        calls.append(("POST", url))
+        assert kwargs["json"] == {"task_ids": ["a", "b", "c"]}
+        return _Resp(404, "missing")
+
+    def fake_get(url, **kwargs):
+        task_id = url.rstrip("/").split("/")[-1]
+        calls.append(("GET", task_id))
+        return _Resp(200, {"task_id": task_id, "status": states[task_id]})
+
+    with patch("savetest_cli.cli.requests.post", side_effect=fake_post), patch(
+        "savetest_cli.cli.requests.get", side_effect=fake_get
+    ):
+        first = _poll_watch_tasks(
+            ["a", "b", "c"],
+            host="https://example.test",
+            project_id="7",
+            token="tok",
+            api=api,
+        )
+        second = _poll_watch_tasks(
+            ["a", "b", "c"],
+            host="https://example.test",
+            project_id="7",
+            token="tok",
+            api=api,
+        )
+
+    assert [task_id for task_id, _status in first] == ["a", "b"]
+    assert first[-1][1]["status"] == "processing"
+    assert [task_id for task_id, _status in second] == ["a", "b"]
+    assert calls == [
+        ("POST", "https://example.test/api/v1/projects/7/allure-results/tasks/status"),
+        ("GET", "a"),
+        ("GET", "b"),
+        ("GET", "a"),
+        ("GET", "b"),
+    ]
+    assert api.tasks_batch is False
+
+
+def test_task_status_batch_reads_all_without_get(monkeypatch: pytest.MonkeyPatch):
+    api = _WatchApi()
+    posts = []
+    monkeypatch.setattr("savetest_cli.cli._TASK_STATUS_BATCH_LIMIT", 2)
+
+    def fake_post(url, **kwargs):
+        posts.append(kwargs["json"]["task_ids"])
+        payload = [
+            {"task_id": task_id, "status": "completed"}
+            for task_id in kwargs["json"]["task_ids"]
+        ]
+        return _Resp(200, payload)
+
+    def fake_get(url, **kwargs):
+        raise AssertionError(url)
+
+    with patch("savetest_cli.cli.requests.post", side_effect=fake_post), patch(
+        "savetest_cli.cli.requests.get", side_effect=fake_get
+    ):
+        pairs = _poll_watch_tasks(
+            ["a", "b", "c"],
+            host="https://example.test",
+            project_id="7",
+            token="tok",
+            api=api,
+        )
+
+    assert posts == [["a", "b"], ["c"]]
+    assert [task_id for task_id, _status in pairs] == ["a", "b", "c"]
+    assert api.tasks_batch is True

savetest_cli-0.3.0rc3/savetest_cli/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.3.0rc3"