Phase 0 milestone 0.6: 501-stub routers for remaining entities
26 stub entities (every matrix entity except Registrant, User, Role, AuditLog) get a router from framework/stub_router.py: one route per matrix action (List, Search, Create, View, Edit, Delete, Restore, VersionHistory). Each route runs the real RBAC check, writes an audit row, then returns 501. Source RFP C.5.7.27. - Registry in src/ypms/stubs.py; test fails if a matrix entity is neither real nor stubbed. - tests/test_stubs.py sweeps entity x role x route against the matrix JSON: 501 + exactly one audit row when granted, 403 + none when not. FERPA/CFSA stubs pinned closed to non-examiner roles. - Registrant restore now gated on the matrix's Restore action (was Edit, the vendored-factory convention); only the super-role holds it. - Test fixtures (roles, make_user, api) lifted to conftest; one app per session (route construction dominated runtime) and a shared password hash for minted-token users. 138 tests passing, ruff + mypy clean. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
1 parent
e5720440fb
commit
98b783b6b5
8 files changed
+381
-58
No files matched your search
@@ -6,7 +6,7 @@ grants), so these tests also pin the matrix's Registrant rows.
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
from collections.abc import Callable, Iterator
|
||||
from collections.abc import Callable
|
||||
from datetime import date
|
||||
from typing import Any
|
||||
|
||||
@@ -16,15 +16,10 @@ from sqlalchemy import select
|
||||
from sqlalchemy.orm import Session
|
||||
|
||||
from ypms.audit.models import AuditLog
|
||||
from ypms.auth.security import create_access_token, hash_password
|
||||
from ypms.db import get_db
|
||||
from ypms.framework.crud import REDACTED
|
||||
from ypms.framework.pii_audit import PII_READ_ACTION
|
||||
from ypms.framework.version_history import VersionHistory
|
||||
from ypms.main import create_app
|
||||
from ypms.seed.permissions import seed_permissions
|
||||
from ypms.seed.run import seed_roles
|
||||
from ypms.users.models import ASSURANCE_STANDARD, Role, User, UserRole
|
||||
from ypms.users.models import User
|
||||
from ypms.youth.models import Registrant
|
||||
|
||||
SSN = "123-45-6789"
|
||||
@@ -42,56 +37,11 @@ PAYLOAD: dict[str, Any] = {
|
||||
Headers = dict[str, str]
|
||||
|
||||
|
||||
@pytest.fixture
|
||||
def roles(db: Session) -> dict[str, Role]:
|
||||
r = seed_roles(db)
|
||||
seed_permissions(db, r)
|
||||
db.flush()
|
||||
return r
|
||||
|
||||
|
||||
@pytest.fixture
|
||||
def make_user(db: Session, roles: dict[str, Role]) -> Callable[..., tuple[User, Headers]]:
|
||||
def _make(username: str, *role_codes: str) -> tuple[User, Headers]:
|
||||
user = User(
|
||||
username=username,
|
||||
password_hash=hash_password("unused-in-these-tests"),
|
||||
is_active=True,
|
||||
assurance_level=ASSURANCE_STANDARD,
|
||||
)
|
||||
db.add(user)
|
||||
db.flush()
|
||||
for code in role_codes:
|
||||
db.add(UserRole(user_id=user.id, role_id=roles[code].id))
|
||||
db.flush()
|
||||
db.refresh(user)
|
||||
token = create_access_token(
|
||||
sub=username, uid=user.id, roles=list(role_codes),
|
||||
assurance_level=ASSURANCE_STANDARD, mfa_verified=False,
|
||||
)
|
||||
return user, {"Authorization": f"Bearer {token}"}
|
||||
|
||||
return _make
|
||||
|
||||
|
||||
@pytest.fixture
|
||||
def staff(make_user: Callable[..., tuple[User, Headers]]) -> tuple[User, Headers]:
|
||||
return make_user("oyp.staff", "OYP_STAFF")
|
||||
|
||||
|
||||
@pytest.fixture
|
||||
def api(db: Session) -> Iterator[TestClient]:
|
||||
app = create_app()
|
||||
|
||||
def _override() -> Iterator[Session]:
|
||||
yield db
|
||||
|
||||
app.dependency_overrides[get_db] = _override
|
||||
with TestClient(app) as c:
|
||||
yield c
|
||||
app.dependency_overrides.clear()
|
||||
|
||||
|
||||
def _create(api: TestClient, headers: Headers, **overrides: Any) -> dict[str, Any]:
|
||||
r = api.post("/registrants", json={**PAYLOAD, **overrides}, headers=headers)
|
||||
assert r.status_code == 201, r.text
|
||||
@@ -256,7 +206,9 @@ def test_delete_requires_delete_permission_then_restore(
|
||||
assert r.status_code == 200 and r.json()["deleted_at"] is not None
|
||||
assert api.get(f"/registrants/{rid}", headers=staff[1]).status_code == 404
|
||||
|
||||
r = api.post(f"/registrants/{rid}/restore", headers=staff[1])
|
||||
# Restore is its own matrix action; only the super-role holds it today.
|
||||
assert api.post(f"/registrants/{rid}/restore", headers=staff[1]).status_code == 403
|
||||
r = api.post(f"/registrants/{rid}/restore", headers=admin)
|
||||
assert r.status_code == 200 and r.json()["deleted_at"] is None
|
||||
assert [a.action for a in db.execute(select(AuditLog).order_by(AuditLog.id)).scalars()][-2:] == [
|
||||
"delete",
|
||||
@@ -365,3 +317,21 @@ def test_host_user_denied(
|
||||
_, hh = make_user("host", "HOST_USER")
|
||||
rid = _create(api, staff[1])["id"]
|
||||
assert api.get(f"/registrants/{rid}", headers=hh).status_code == 403
|
||||
|
||||
|
||||
def test_staff_guard_holds_even_if_matrix_grants_portal_role_list(
|
||||
api: TestClient,
|
||||
db: Session,
|
||||
roles: dict[str, Any],
|
||||
make_user: Callable[..., tuple[User, Headers]],
|
||||
) -> None:
|
||||
"""Defense in depth: a future matrix edit granting YOUTH_USER List must not
|
||||
expose every registrant."""
|
||||
from ypms.security.models import Permission
|
||||
|
||||
db.add(Permission(role_id=roles["YOUTH_USER"].id, entity="Registrant", action="List"))
|
||||
db.flush()
|
||||
_, yh = make_user("youth", "YOUTH_USER")
|
||||
r = api.get("/registrants", headers=yh)
|
||||
assert r.status_code == 403
|
||||
assert r.json()["detail"]["reason"] == "staff-only route"
|
||||
Reference in new issue
Block a user