2026-04-06 08:02:13 -04:00
|
|
|
"""Tests for project registry resolution and refresh behavior."""
|
|
|
|
|
|
|
|
|
|
import json
|
|
|
|
|
|
|
|
|
|
import atocore.config as config
|
|
|
|
|
from atocore.projects.registry import (
|
2026-04-06 09:11:11 -04:00
|
|
|
build_project_registration_proposal,
|
2026-04-06 08:02:13 -04:00
|
|
|
get_registered_project,
|
2026-04-06 08:46:37 -04:00
|
|
|
get_project_registry_template,
|
2026-04-06 08:02:13 -04:00
|
|
|
list_registered_projects,
|
2026-04-06 09:52:19 -04:00
|
|
|
register_project,
|
2026-04-06 08:02:13 -04:00
|
|
|
refresh_registered_project,
|
2026-04-06 12:31:24 -04:00
|
|
|
update_project,
|
2026-04-06 08:02:13 -04:00
|
|
|
)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_project_registry_lists_projects_with_resolved_roots(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
vault_dir.mkdir()
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
(vault_dir / "incoming" / "projects" / "p04-gigabit").mkdir(parents=True)
|
|
|
|
|
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p04-gigabit",
|
|
|
|
|
"aliases": ["p04", "gigabit"],
|
|
|
|
|
"description": "P04 docs",
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{
|
|
|
|
|
"source": "vault",
|
|
|
|
|
"subpath": "incoming/projects/p04-gigabit",
|
|
|
|
|
"label": "P04 staged docs",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
}
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
projects = list_registered_projects()
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert len(projects) == 1
|
|
|
|
|
assert projects[0]["id"] == "p04-gigabit"
|
|
|
|
|
assert projects[0]["ingest_roots"][0]["exists"] is True
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_project_registry_resolves_alias(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
vault_dir.mkdir()
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p05-interferometer",
|
|
|
|
|
"aliases": ["p05", "interferometer"],
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p05-interferometer"}
|
|
|
|
|
],
|
|
|
|
|
}
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
project = get_registered_project("p05")
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert project is not None
|
|
|
|
|
assert project.project_id == "p05-interferometer"
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_refresh_registered_project_ingests_registered_roots(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
project_dir = vault_dir / "incoming" / "projects" / "p06-polisher"
|
|
|
|
|
project_dir.mkdir(parents=True)
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p06-polisher",
|
|
|
|
|
"aliases": ["p06", "polisher"],
|
|
|
|
|
"description": "P06 docs",
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p06-polisher"}
|
|
|
|
|
],
|
|
|
|
|
}
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
calls = []
|
|
|
|
|
|
|
|
|
|
def fake_ingest_folder(path, purge_deleted=True):
|
|
|
|
|
calls.append((str(path), purge_deleted))
|
|
|
|
|
return [{"file": str(path / "README.md"), "status": "ingested"}]
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
monkeypatch.setattr("atocore.projects.registry.ingest_folder", fake_ingest_folder)
|
|
|
|
|
result = refresh_registered_project("polisher")
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert result["project"] == "p06-polisher"
|
|
|
|
|
assert len(calls) == 1
|
|
|
|
|
assert calls[0][0].endswith("p06-polisher")
|
|
|
|
|
assert calls[0][1] is False
|
|
|
|
|
assert result["roots"][0]["status"] == "ingested"
|
feat: tunable ranking, refresh status, chroma backup + admin endpoints
Three small improvements that move the operational baseline forward
without changing the existing trust model.
1. Tunable retrieval ranking weights
- rank_project_match_boost, rank_query_token_step,
rank_query_token_cap, rank_path_high_signal_boost,
rank_path_low_signal_penalty are now Settings fields
- all overridable via ATOCORE_* env vars
- retriever no longer hard-codes 2.0 / 1.18 / 0.72 / 0.08 / 1.32
- lets ranking be tuned per environment as Wave 1 is exercised
without code changes
2. /projects/{name}/refresh status
- refresh_registered_project now returns an overall status field
("ingested", "partial", "nothing_to_ingest") plus roots_ingested
and roots_skipped counters
- ProjectRefreshResponse advertises the new fields so callers can
rely on them
- covers the case where every configured root is missing on disk
3. Chroma cold snapshot + admin backup endpoints
- create_runtime_backup now accepts include_chroma and writes a
cold directory copy of the chroma persistence path
- new list_runtime_backups() and validate_backup() helpers
- new endpoints:
- POST /admin/backup create snapshot (optional chroma)
- GET /admin/backup list snapshots
- GET /admin/backup/{stamp}/validate structural validation
- chroma snapshots are taken under exclusive_ingestion() so a refresh
or ingest cannot race with the cold copy
- backup metadata records what was actually included and how big
Tests:
- 8 new tests covering tunable weights, refresh status branches
(ingested / partial / nothing_to_ingest), chroma snapshot, list,
validate, and the API endpoints (including the lock-acquisition path)
- existing fake refresh stubs in test_api_storage.py updated for the
expanded ProjectRefreshResponse model
- full suite: 105 passing (was 97)
next-steps doc updated to reflect that the chroma snapshot + restore
validation gap from current-state.md is now closed in code; only the
operational retention policy remains.
2026-04-06 18:42:19 -04:00
|
|
|
assert result["status"] == "ingested"
|
|
|
|
|
assert result["roots_ingested"] == 1
|
|
|
|
|
assert result["roots_skipped"] == 0
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_refresh_registered_project_reports_nothing_to_ingest_when_all_missing(
|
|
|
|
|
tmp_path, monkeypatch
|
|
|
|
|
):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
vault_dir.mkdir()
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p07-ghost",
|
|
|
|
|
"aliases": ["ghost"],
|
|
|
|
|
"description": "Project whose roots do not exist on disk",
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p07-ghost"}
|
|
|
|
|
],
|
|
|
|
|
}
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
def fail_ingest_folder(path, purge_deleted=True):
|
|
|
|
|
raise AssertionError(f"ingest_folder should not be called for missing root: {path}")
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
monkeypatch.setattr("atocore.projects.registry.ingest_folder", fail_ingest_folder)
|
|
|
|
|
result = refresh_registered_project("ghost")
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert result["status"] == "nothing_to_ingest"
|
|
|
|
|
assert result["roots_ingested"] == 0
|
|
|
|
|
assert result["roots_skipped"] == 1
|
|
|
|
|
assert result["roots"][0]["status"] == "missing"
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_refresh_registered_project_reports_partial_status(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
real_root = vault_dir / "incoming" / "projects" / "p08-mixed"
|
|
|
|
|
real_root.mkdir(parents=True)
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p08-mixed",
|
|
|
|
|
"aliases": ["mixed"],
|
|
|
|
|
"description": "One root present, one missing",
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p08-mixed"},
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p08-mixed-missing"},
|
|
|
|
|
],
|
|
|
|
|
}
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
def fake_ingest_folder(path, purge_deleted=True):
|
|
|
|
|
return [{"file": str(path / "README.md"), "status": "ingested"}]
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
monkeypatch.setattr("atocore.projects.registry.ingest_folder", fake_ingest_folder)
|
|
|
|
|
result = refresh_registered_project("mixed")
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert result["status"] == "partial"
|
|
|
|
|
assert result["roots_ingested"] == 1
|
|
|
|
|
assert result["roots_skipped"] == 1
|
|
|
|
|
statuses = sorted(root["status"] for root in result["roots"])
|
|
|
|
|
assert statuses == ["ingested", "missing"]
|
2026-04-06 08:46:37 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_project_registry_template_has_expected_shape():
|
|
|
|
|
template = get_project_registry_template()
|
|
|
|
|
assert "projects" in template
|
|
|
|
|
assert template["projects"][0]["id"] == "p07-example"
|
|
|
|
|
assert template["projects"][0]["ingest_roots"][0]["source"] == "vault"
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_project_registry_rejects_alias_collision(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
vault_dir.mkdir()
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p04-gigabit",
|
|
|
|
|
"aliases": ["shared"],
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p04-gigabit"}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"id": "p05-interferometer",
|
|
|
|
|
"aliases": ["shared"],
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p05-interferometer"}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
try:
|
|
|
|
|
list_registered_projects()
|
|
|
|
|
except ValueError as exc:
|
|
|
|
|
assert "collision" in str(exc)
|
|
|
|
|
else:
|
|
|
|
|
raise AssertionError("Expected project registry collision to raise")
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
2026-04-06 09:11:11 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_project_registration_proposal_normalizes_and_resolves_paths(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
staged = vault_dir / "incoming" / "projects" / "p07-example"
|
|
|
|
|
staged.mkdir(parents=True)
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(json.dumps({"projects": []}), encoding="utf-8")
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
proposal = build_project_registration_proposal(
|
|
|
|
|
project_id="p07-example",
|
|
|
|
|
aliases=["p07", "example-project", "p07"],
|
|
|
|
|
description="Example project",
|
|
|
|
|
ingest_roots=[
|
|
|
|
|
{
|
|
|
|
|
"source": "vault",
|
|
|
|
|
"subpath": "incoming/projects/p07-example",
|
|
|
|
|
"label": "Primary docs",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
)
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert proposal["project"]["aliases"] == ["p07", "example-project"]
|
|
|
|
|
assert proposal["resolved_ingest_roots"][0]["exists"] is True
|
|
|
|
|
assert proposal["valid"] is True
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_project_registration_proposal_reports_collisions(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
vault_dir.mkdir()
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p05-interferometer",
|
|
|
|
|
"aliases": ["p05", "interferometer"],
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p05-interferometer"}
|
|
|
|
|
],
|
|
|
|
|
}
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
proposal = build_project_registration_proposal(
|
|
|
|
|
project_id="p08-example",
|
|
|
|
|
aliases=["interferometer"],
|
|
|
|
|
ingest_roots=[
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p08-example"}
|
|
|
|
|
],
|
|
|
|
|
)
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert proposal["valid"] is False
|
|
|
|
|
assert proposal["collisions"][0]["existing_project"] == "p05-interferometer"
|
2026-04-06 09:52:19 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_register_project_persists_new_entry(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
staged = vault_dir / "incoming" / "projects" / "p07-example"
|
|
|
|
|
staged.mkdir(parents=True)
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(json.dumps({"projects": []}), encoding="utf-8")
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
result = register_project(
|
|
|
|
|
project_id="p07-example",
|
|
|
|
|
aliases=["p07", "example-project"],
|
|
|
|
|
description="Example project",
|
|
|
|
|
ingest_roots=[
|
|
|
|
|
{
|
|
|
|
|
"source": "vault",
|
|
|
|
|
"subpath": "incoming/projects/p07-example",
|
|
|
|
|
"label": "Primary docs",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
)
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert result["status"] == "registered"
|
|
|
|
|
payload = json.loads(registry_path.read_text(encoding="utf-8"))
|
|
|
|
|
assert payload["projects"][0]["id"] == "p07-example"
|
|
|
|
|
assert payload["projects"][0]["aliases"] == ["p07", "example-project"]
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_register_project_rejects_collisions(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
vault_dir.mkdir()
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p05-interferometer",
|
|
|
|
|
"aliases": ["p05", "interferometer"],
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p05-interferometer"}
|
|
|
|
|
],
|
|
|
|
|
}
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
try:
|
|
|
|
|
register_project(
|
|
|
|
|
project_id="p07-example",
|
|
|
|
|
aliases=["interferometer"],
|
|
|
|
|
ingest_roots=[
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p07-example"}
|
|
|
|
|
],
|
|
|
|
|
)
|
|
|
|
|
except ValueError as exc:
|
|
|
|
|
assert "collisions" in str(exc)
|
|
|
|
|
else:
|
|
|
|
|
raise AssertionError("Expected collision to prevent project registration")
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
2026-04-06 12:31:24 -04:00
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_update_project_persists_description_and_aliases(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
staged = vault_dir / "incoming" / "projects" / "p04-gigabit"
|
|
|
|
|
staged.mkdir(parents=True)
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p04-gigabit",
|
|
|
|
|
"aliases": ["p04", "gigabit"],
|
|
|
|
|
"description": "Old description",
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{
|
|
|
|
|
"source": "vault",
|
|
|
|
|
"subpath": "incoming/projects/p04-gigabit",
|
|
|
|
|
"label": "Primary docs",
|
|
|
|
|
}
|
|
|
|
|
],
|
|
|
|
|
}
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
result = update_project(
|
|
|
|
|
"p04",
|
|
|
|
|
aliases=["p04", "gigabit", "gigabit-project"],
|
|
|
|
|
description="Updated P04 project docs",
|
|
|
|
|
)
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|
|
|
|
|
|
|
|
|
|
assert result["status"] == "updated"
|
|
|
|
|
assert result["project"]["id"] == "p04-gigabit"
|
|
|
|
|
assert result["project"]["aliases"] == ["p04", "gigabit", "gigabit-project"]
|
|
|
|
|
assert result["project"]["description"] == "Updated P04 project docs"
|
|
|
|
|
|
|
|
|
|
payload = json.loads(registry_path.read_text(encoding="utf-8"))
|
|
|
|
|
assert payload["projects"][0]["aliases"] == ["p04", "gigabit", "gigabit-project"]
|
|
|
|
|
assert payload["projects"][0]["description"] == "Updated P04 project docs"
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
def test_update_project_rejects_colliding_aliases(tmp_path, monkeypatch):
|
|
|
|
|
vault_dir = tmp_path / "vault"
|
|
|
|
|
drive_dir = tmp_path / "drive"
|
|
|
|
|
config_dir = tmp_path / "config"
|
|
|
|
|
vault_dir.mkdir()
|
|
|
|
|
drive_dir.mkdir()
|
|
|
|
|
config_dir.mkdir()
|
|
|
|
|
registry_path = config_dir / "project-registry.json"
|
|
|
|
|
registry_path.write_text(
|
|
|
|
|
json.dumps(
|
|
|
|
|
{
|
|
|
|
|
"projects": [
|
|
|
|
|
{
|
|
|
|
|
"id": "p04-gigabit",
|
|
|
|
|
"aliases": ["p04", "gigabit"],
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p04-gigabit"}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
{
|
|
|
|
|
"id": "p05-interferometer",
|
|
|
|
|
"aliases": ["p05", "interferometer"],
|
|
|
|
|
"ingest_roots": [
|
|
|
|
|
{"source": "vault", "subpath": "incoming/projects/p05-interferometer"}
|
|
|
|
|
],
|
|
|
|
|
},
|
|
|
|
|
]
|
|
|
|
|
}
|
|
|
|
|
),
|
|
|
|
|
encoding="utf-8",
|
|
|
|
|
)
|
|
|
|
|
|
|
|
|
|
monkeypatch.setenv("ATOCORE_VAULT_SOURCE_DIR", str(vault_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_DRIVE_SOURCE_DIR", str(drive_dir))
|
|
|
|
|
monkeypatch.setenv("ATOCORE_PROJECT_REGISTRY_PATH", str(registry_path))
|
|
|
|
|
|
|
|
|
|
original_settings = config.settings
|
|
|
|
|
try:
|
|
|
|
|
config.settings = config.Settings()
|
|
|
|
|
try:
|
|
|
|
|
update_project(
|
|
|
|
|
"p04-gigabit",
|
|
|
|
|
aliases=["p04", "interferometer"],
|
|
|
|
|
)
|
|
|
|
|
except ValueError as exc:
|
|
|
|
|
assert "collisions" in str(exc)
|
|
|
|
|
else:
|
|
|
|
|
raise AssertionError("Expected collision to prevent project update")
|
|
|
|
|
finally:
|
|
|
|
|
config.settings = original_settings
|