mirror of
https://github.com/hanzoai/insights.git
synced 2026-10-30 16:15:31 +00:00
zeekay
1b367e2b62
The tree was a debranded snapshot of upstream taken around 2026-02-20, and upstream has moved about five months since. This replaces it with upstream master, mechanically re-derived: bin/debrand does the rename, and the pipeline and squashed migrations are restored from the shipped tree. Not a merge. The previous import replaced the tree wholesale in a single commit, so there is no shared content history to merge against. Reproducing the rename on the new tree is what actually converges. What survives: .hanzo/workflows, the Dockerfiles and bin/docker* entrypoints, compose files, LLM.md, and insights/migrations. The squashed initial migration is load-bearing — the live database records it by name, so upstream's ~918 individual migrations cannot come back. Nine products are dropped as unreferenced. Twenty-one more that we do not run are kept anyway because the core app still imports them; unpicking those is a refactor and does not belong in a rebase. This base does not build yet, and the reasons are not in the rename: upstream now wants Python 3.13.13 against our 3.12.12 and Django 5.2 against our 4.2, and the dependency manifests still name upstream packages that we republish under our own names. Those are decisions, not renames.
103 lines
4.0 KiB
Python
103 lines
4.0 KiB
Python
"""Tests for insightscli pr:upload-video.
|
|
|
|
The shared upload plumbing (key shape, base64 body, 409 retry, validation gates) is
|
|
pinned in ``test_pr_assets``; these guard what is video-specific: the extension
|
|
allowlist, the link-style markdown on stdout (a plain [label](url), never an image
|
|
embed, since GitHub renders no player for raw-hosted video), the "add video" commit
|
|
message, and the --yes gate.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import re
|
|
from pathlib import Path
|
|
from types import SimpleNamespace
|
|
|
|
import pytest
|
|
from unittest.mock import Mock, patch
|
|
|
|
from click.testing import CliRunner
|
|
from insightscli_commands import pr_assets, upload_video
|
|
|
|
_URL_RE = r"https://raw\.githubusercontent\.com/Insights/pr-assets/deadbeef/\d{4}/\d{2}/[0-9a-f-]+\.mp4"
|
|
|
|
|
|
@pytest.fixture
|
|
def mp4(tmp_path: Path) -> Path:
|
|
path = tmp_path / "frontend-qa.mp4"
|
|
path.write_bytes(b"\x00\x00\x00\x18ftypmp42 fake bytes")
|
|
return path
|
|
|
|
|
|
def _resp(status: int, sha: str | None = None) -> SimpleNamespace:
|
|
return SimpleNamespace(
|
|
status_code=status,
|
|
ok=status < 400,
|
|
json=lambda: {"commit": {"sha": sha}} if sha is not None else {},
|
|
)
|
|
|
|
|
|
def _session(*responses: SimpleNamespace) -> Mock:
|
|
session = Mock()
|
|
session.put.side_effect = responses
|
|
return session
|
|
|
|
|
|
@pytest.mark.parametrize("name", ["reel.gif", "still.png", "notes.txt"], ids=["gif", "png", "txt"])
|
|
def test_video_allowlist_rejects_non_video_extensions(tmp_path: Path, name: str) -> None:
|
|
# The video command's allowlist is its contract: mp4/webm only; images belong to
|
|
# pr:upload-image.
|
|
path = tmp_path / name
|
|
path.write_bytes(b"data")
|
|
with pytest.raises(pr_assets.click.ClickException) as excinfo:
|
|
pr_assets.validate(path, upload_video._ALLOWED_EXTS, upload_video._MAX_MB)
|
|
assert "mp4" in str(excinfo.value) and "webm" in str(excinfo.value)
|
|
|
|
|
|
def test_without_yes_uploads_nothing_and_exits_nonzero(mp4: Path, monkeypatch: pytest.MonkeyPatch) -> None:
|
|
monkeypatch.setenv("GH_TOKEN", "tok")
|
|
session = _session(_resp(201, "x"))
|
|
with patch.object(upload_video.requests, "Session", return_value=session):
|
|
result = CliRunner(mix_stderr=False).invoke(upload_video.upload_video, [str(mp4)])
|
|
|
|
assert result.exit_code == 1
|
|
assert not result.stdout
|
|
session.put.assert_not_called()
|
|
|
|
|
|
def test_stdout_is_link_markdown_not_image_embed(mp4: Path, monkeypatch: pytest.MonkeyPatch) -> None:
|
|
monkeypatch.setenv("GH_TOKEN", "tok")
|
|
with patch.object(upload_video.requests, "Session", return_value=_session(_resp(201, "deadbeef"))):
|
|
result = CliRunner(mix_stderr=False).invoke(upload_video.upload_video, ["--yes", str(mp4)])
|
|
|
|
assert result.exit_code == 0
|
|
lines = result.stdout.strip().splitlines()
|
|
assert len(lines) == 1
|
|
assert re.fullmatch(r"\[frontend-qa\]\(" + _URL_RE + r"\)", lines[0])
|
|
assert not lines[0].startswith("!") # a video embed renders as a broken image on GitHub
|
|
|
|
|
|
def test_upload_uses_video_commit_message(mp4: Path, monkeypatch: pytest.MonkeyPatch) -> None:
|
|
monkeypatch.setenv("GH_TOKEN", "tok")
|
|
session = _session(_resp(201, "deadbeef"))
|
|
with patch.object(upload_video.requests, "Session", return_value=session):
|
|
result = CliRunner(mix_stderr=False).invoke(upload_video.upload_video, ["--yes", str(mp4)])
|
|
|
|
assert result.exit_code == 0
|
|
assert session.put.call_args.kwargs["json"]["message"] == "add video"
|
|
|
|
|
|
def test_label_rejected_for_multiple_files(mp4: Path, tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
|
|
# Isolate the network so a guard regression fails loudly here instead of PUTting to
|
|
# the public repo from a token-bearing machine.
|
|
other = tmp_path / "second.webm"
|
|
other.write_bytes(b"webm fake")
|
|
monkeypatch.setenv("GH_TOKEN", "tok")
|
|
session = _session(_resp(201, "x"))
|
|
with patch.object(upload_video.requests, "Session", return_value=session):
|
|
result = CliRunner(mix_stderr=False).invoke(
|
|
upload_video.upload_video, ["--yes", "--label", "demo", str(mp4), str(other)]
|
|
)
|
|
assert result.exit_code != 0
|
|
session.put.assert_not_called()
|