pytest-devant-cloud 0.1.1__tar.gz → 0.1.2__tar.gz

{pytest_devant_cloud-0.1.1 → pytest_devant_cloud-0.1.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pytest-devant-cloud
-Version: 0.1.1
+Version: 0.1.2
 Summary: pytest plugin that streams runs, results, and step trees to Devant Cloud's /v1/runs API.
 Project-URL: Homepage, https://github.com/devant-net/devq-cloud/tree/main/packages/pytest-devant-cloud
 Project-URL: Repository, https://github.com/devant-net/devq-cloud

{pytest_devant_cloud-0.1.1 → pytest_devant_cloud-0.1.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pytest-devant-cloud"
-version = "0.1.1"
+version = "0.1.2"
 description = "pytest plugin that streams runs, results, and step trees to Devant Cloud's /v1/runs API."
 readme = "README.md"
 license = { file = "LICENSE" }

{pytest_devant_cloud-0.1.1 → pytest_devant_cloud-0.1.2}/src/pytest_devant_cloud/client.py

@@ -8,6 +8,7 @@ wire-level concerns (auth, retries, JSON encoding).
 
 from __future__ import annotations
 
+import os
 import time
 from dataclasses import dataclass
 from typing import Any
@@ -52,10 +53,14 @@ class DevqClient:
         # Re-use one Connection pool for the whole session — pytest can
         # easily emit hundreds of POSTs in a fast suite, and TCP setup
         # cost dwarfs the request itself otherwise.
+        # Content-Type intentionally NOT set here: httpx infers
+        # `application/json` from the `json=` kwarg used by _request(), and
+        # leaves multipart boundaries alone for `files=`-based uploads. A
+        # static client-level Content-Type would clobber the multipart
+        # boundary header on artifact uploads.
         self._http = httpx.Client(
             headers={
                 "Authorization": f"Bearer {api_token}",
-                "Content-Type": "application/json",
                 "Accept": "application/json",
             },
             timeout=timeout,
@@ -210,6 +215,36 @@ class DevqClient:
         body = resp.json()
         return body.get("results", [])
 
+    def upload_artifact(
+        self,
+        run_id: int,
+        result_id: int,
+        attempt_id: int,
+        *,
+        path: str,
+        name: str,
+        content_type: str,
+    ) -> None:
+        """Multipart POST a single file to the run/result/attempt's artifacts.
+
+        Mirrors @devant-net/reporter-core's `uploadArtifacts` wire shape:
+        one `file` part with the bytes, one `name` text part with the
+        display name shown in the Devant Cloud UI.
+        """
+        with open(path, "rb") as fh:
+            file_bytes = fh.read()
+        # Reuse the configured httpx client (auth header, base url, timeouts)
+        # but bypass `_request`'s JSON-only path.
+        resp = self._http.post(
+            f"{self.api_url}/v1/runs/{run_id}/results/{result_id}/attempts/{attempt_id}/artifacts",
+            files={"file": (os.path.basename(path), file_bytes, content_type)},
+            data={"name": name},
+        )
+        if resp.status_code >= 400:
+            raise RuntimeError(
+                f"[devant-reporter] artifact upload {resp.status_code}: {resp.text[:200]}"
+            )
+
     def submit_coverage(
         self,
         run_id: int,

{pytest_devant_cloud-0.1.1 → pytest_devant_cloud-0.1.2}/src/pytest_devant_cloud/plugin.py

@@ -502,4 +502,101 @@ def _submit_one(
             }
         ],
     }
-    state.client.submit_results(state.run_id, [submit_payload])
+    inserted = state.client.submit_results(state.run_id, [submit_payload])
+    _maybe_upload_playwright_artifacts(state, item, inserted)
+
+
+# ── Playwright (pytest-playwright) artifact integration ─────────────────────
+
+
+def _maybe_upload_playwright_artifacts(
+    state: _State,
+    item: pytest.Item,
+    inserted: list[dict[str, Any]],
+) -> None:
+    """If pytest-playwright wrote artifacts for this test, upload them.
+
+    pytest-playwright stores per-test artifacts at:
+        <--output dir>/<slugified-nodeid>/
+            trace.zip         (--tracing on / retain-on-failure)
+            video.webm        (--video on / retain-on-failure)
+            test-failed-N.png (--screenshot only-on-failure / on)
+
+    We compute the same path using pytest-playwright's own helpers (so the
+    slug formula stays in lockstep with upstream) and POST every file
+    inside as a multipart artifact attached to attempt 1.
+
+    Silent no-op when pytest-playwright isn't installed.
+    """
+    try:
+        from pytest_playwright.pytest_playwright import _truncate_file_name  # type: ignore
+        from slugify import slugify  # type: ignore
+    except ImportError:
+        return
+
+    if not inserted:
+        return
+    record = inserted[0]
+    result_id = record.get("result_id")
+    attempts = record.get("attempt_ids", [])
+    if not result_id or not attempts:
+        return
+    attempt_id = attempts[0].get("id")
+    if not attempt_id:
+        return
+
+    try:
+        output_root = item.session.config.getoption("--output", default="test-results")
+    except (ValueError, AttributeError):
+        return
+    from pathlib import Path
+
+    output_dir = Path(output_root).absolute() / _truncate_file_name(slugify(item.nodeid))
+    if not output_dir.is_dir():
+        return  # no artifacts captured for this test (passed cleanly, etc.)
+
+    for path in sorted(output_dir.iterdir()):
+        if not path.is_file():
+            continue
+        try:
+            content_type = _guess_content_type(path)
+            display = _artifact_display_name(path)
+            state.client.upload_artifact(
+                state.run_id or 0,
+                int(result_id),
+                int(attempt_id),
+                path=str(path),
+                name=display,
+                content_type=content_type,
+            )
+        except Exception as exc:  # noqa: BLE001
+            sys.stderr.write(
+                f'[devant] failed to upload "{path.name}" for "{item.nodeid}": {exc}\n'
+            )
+
+
+def _guess_content_type(path: "Path") -> str:  # noqa: F821  (Path imported above)
+    import mimetypes
+
+    suffix = path.suffix.lower()
+    if suffix == ".zip":
+        return "application/zip"
+    if suffix == ".webm":
+        return "video/webm"
+    if suffix == ".png":
+        return "image/png"
+    if suffix in (".jpg", ".jpeg"):
+        return "image/jpeg"
+    guessed, _ = mimetypes.guess_type(str(path))
+    return guessed or "application/octet-stream"
+
+
+def _artifact_display_name(path: "Path") -> str:  # noqa: F821
+    """UI-friendly artifact name. `trace*.zip` gets normalised to `trace.zip`
+    so the Devant Cloud UI's trace-viewer detection works regardless of
+    upstream filename quirks (Playwright writes `trace-1.zip` for multi-page
+    tests)."""
+    name = path.name
+    if path.suffix.lower() == ".zip" and "trace" in name.lower():
+        return "trace.zip"
+    return name