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.
91 lines
3.7 KiB
Python
91 lines
3.7 KiB
Python
from __future__ import annotations
|
|
|
|
from pathlib import Path
|
|
|
|
import pytest
|
|
|
|
import insightscli_commands.migration_deletions as md
|
|
from click.testing import CliRunner
|
|
|
|
|
|
@pytest.mark.parametrize(
|
|
"path,expected",
|
|
[
|
|
("insights/migrations/0001_initial.py", True),
|
|
("ee/migrations/0042_thing.py", True),
|
|
("products/tasks/backend/migrations/0003_x.py", True),
|
|
("insights/session/migrations/0001_initial.py", True), # real Django sub-package app
|
|
("insights/migrations/__init__.py", False),
|
|
("insights/migrations/manual_fix.py", False), # non-numbered -> not a flagged migration
|
|
("insights/rbac/migrations/rbac_team_migration.py", False), # standalone RBAC data script, not ORM
|
|
("insights/datastore/migrations/0099_x.py", False),
|
|
("insights/async_migrations/migrations/0005_y.py", False),
|
|
("insights/migrations/0001_initial.txt", False),
|
|
("products/tasks/backend/views.py", False),
|
|
],
|
|
)
|
|
def test_is_django_migration(path: str, expected: bool) -> None:
|
|
assert md.is_django_migration(path) is expected
|
|
|
|
|
|
def test_load_allowlist_ignores_comments_and_blanks(tmp_path: Path) -> None:
|
|
f = tmp_path / "allow.txt"
|
|
f.write_text(
|
|
"# header\n\ninsights/migrations/0500_oops.py # trailing comment\n products/old/backend/migrations/ \n"
|
|
)
|
|
assert md.load_allowlist(f) == ["insights/migrations/0500_oops.py", "products/old/backend/migrations/"]
|
|
|
|
|
|
def test_load_allowlist_missing_file_is_empty(tmp_path: Path) -> None:
|
|
assert md.load_allowlist(tmp_path / "nope.txt") == []
|
|
|
|
|
|
@pytest.mark.parametrize(
|
|
"path,allowlist,expected",
|
|
[
|
|
("insights/migrations/0500_oops.py", ["insights/migrations/0500_oops.py"], True),
|
|
("products/old/backend/migrations/0001_initial.py", ["products/old/backend/migrations/"], True),
|
|
(
|
|
"products/old/backend/migrations/0001_initial.py",
|
|
["products/old/backend/migrations"],
|
|
True,
|
|
), # trailing slash optional
|
|
("products/old_v2/backend/migrations/0001_initial.py", ["products/old/backend/migrations/"], False),
|
|
("insights/migrations/0500_oops.py", [], False),
|
|
],
|
|
)
|
|
def test_is_allowlisted(path: str, allowlist: list[str], expected: bool) -> None:
|
|
assert md.is_allowlisted(path, allowlist) is expected
|
|
|
|
|
|
def test_guarded_deletions_filters_and_respects_allowlist() -> None:
|
|
paths = [
|
|
"insights/migrations/0001_initial.py",
|
|
"insights/migrations/__init__.py",
|
|
"insights/datastore/migrations/0099_x.py",
|
|
"products/old/backend/migrations/0001_x.py",
|
|
"frontend/app.ts",
|
|
]
|
|
assert md.guarded_deletions(paths, ["products/old/backend/migrations/"]) == ["insights/migrations/0001_initial.py"]
|
|
|
|
|
|
@pytest.mark.parametrize(
|
|
"stdin,allowlist,expected_exit",
|
|
[
|
|
("insights/migrations/0001_initial.py\nfrontend/x.ts\n", "", 1), # historical deletion -> block
|
|
("insights/migrations/__init__.py\nfrontend/app.ts\n", "", 0), # only non-migrations -> pass
|
|
# historical migration deleted but acknowledged in the allowlist -> pass
|
|
("products/old/backend/migrations/0001_x.py\n", "products/old/backend/migrations/\n", 0),
|
|
],
|
|
)
|
|
def test_command(
|
|
monkeypatch: pytest.MonkeyPatch, tmp_path: Path, stdin: str, allowlist: str, expected_exit: int
|
|
) -> None:
|
|
allow = tmp_path / "allow.txt"
|
|
allow.write_text(allowlist)
|
|
monkeypatch.setattr(md, "ALLOWLIST_PATH", allow)
|
|
result = CliRunner().invoke(md.cmd_lint_migration_deletions, input=stdin)
|
|
# A crash also exits non-zero — assert the exit is a real verdict, not a trapped exception.
|
|
assert not isinstance(result.exception, Exception)
|
|
assert result.exit_code == expected_exit
|