Developer
ddd92e3dd4
- Add ToolType.home_directory column with default /home/user
- Add Alembic migration to add column, set existing rows, and rewrite
/workspace to /home/user/{{WORKSPACE_NAME}} in legacy templates
- Add merge migration fc8f1a20cbf6 to resolve Alembic multiple heads
- Update manifest compiler to honor manifest.home_directory for HOME,
WORKDIR, /workspace symlink, and default repo mount target
- Update legacy dockerfile/compose instance generation to use
tool_type.home_directory
- Thread resolved home_dir through config profile and git mount expansion
- Generate entrypoint permission fixer to chown home/mounts at startup
- Update base.dockerfile with sudo/passwordless sudo for permission fixer
- Add unit tests for manifest compiler, instance service, and migrations
- Add placeholder integration test for container lifecycle
- Update openspec/tasks/home-path-expansion.md task checkboxes
- Update project maps for modified files
Quality gates: py_compile, ruff, mypy, pytest tests/unit (205 passed),
pytest tests/integration (110 passed, 35 skipped). Alembic round-trip
and container lifecycle integration tests require Docker/PostgreSQL.
50 lines
1.7 KiB
Python
50 lines
1.7 KiB
Python
"""Unit tests for Alembic migration structure/import.
|
|
|
|
Actual upgrade/downgrade round-trips require a PostgreSQL database, so these
|
|
tests verify that migrations are importable, have the expected identifiers,
|
|
and declare the expected dependencies.
|
|
"""
|
|
|
|
import importlib.util
|
|
from pathlib import Path
|
|
|
|
import pytest
|
|
|
|
|
|
@pytest.mark.unit
|
|
def test_home_directory_migration_imports_and_rewrites() -> None:
|
|
migration_path = Path(__file__).parent.parent.parent / (
|
|
"alembic/versions/2026_06_14_104415_add_tool_type_home_directory.py"
|
|
)
|
|
assert migration_path.exists()
|
|
|
|
spec = importlib.util.spec_from_file_location("home_dir_migration", migration_path)
|
|
assert spec is not None and spec.loader is not None
|
|
module = importlib.util.module_from_spec(spec)
|
|
spec.loader.exec_module(module)
|
|
|
|
assert module.revision == "2026_06_14_104415"
|
|
assert module.down_revision == "f3d2dc90ba3a"
|
|
assert callable(module.upgrade)
|
|
assert callable(module.downgrade)
|
|
assert module.OLD_WORKSPACE == "/workspace"
|
|
assert module.NEW_WORKSPACE == "/home/user/{{WORKSPACE_NAME}}"
|
|
|
|
|
|
@pytest.mark.unit
|
|
def test_merge_migration_resolves_heads() -> None:
|
|
migration_path = Path(__file__).parent.parent.parent / (
|
|
"alembic/versions/fc8f1a20cbf6_merge_home_directory_and_pi_agent_mount_.py"
|
|
)
|
|
assert migration_path.exists()
|
|
|
|
spec = importlib.util.spec_from_file_location("merge_migration", migration_path)
|
|
assert spec is not None and spec.loader is not None
|
|
module = importlib.util.module_from_spec(spec)
|
|
spec.loader.exec_module(module)
|
|
|
|
assert module.revision == "fc8f1a20cbf6"
|
|
assert "2026_06_14_104415" in module.down_revision
|
|
assert "8c6d1dbd4798" in module.down_revision
|
|
assert callable(module.upgrade)
|