Rollen-differenzierte Verwaltungs-UI (Material-/Leitungsverantwortliche)
CI / backend-tests (push) Failing after 1m19s
CI / frontend-build (push) Successful in 23s

Bug: Frontend prüfte nur istAdmin, Materialverantwortliche und Leitungs-
verantwortliche sahen exakt dieselbe UI wie Mitarbeiter - obwohl das Backend
längst granular unterscheidet (require_roles je Endpunkt).

- AuthContext gibt volle Rollenliste + istVerantwortlich/istMaterial-
  verantwortlich/istLeitungsverantwortlich weiter.
- /admin-Route für alle drei Rollen offen, AdminPage filtert Tabs nach
  Rolle (Material/Leitung: Objekte-Pflege, Kontrollverantwortung,
  Änderungslog; nur Administration: Stammdaten/Benutzer/Eskalation/
  Zuständigkeit).
- ObjektSection blendet Anlegen/Duplizieren/Kopplung für Nicht-Admins aus
  (waren admin-only im Backend, hätten 403 geworfen).
- Promise.allSettled statt Promise.all beim Laden, damit ein admin-only
  Endpunkt (z.B. /benutzer) nicht die ganze Seite für niedrigere Rollen
  blockiert.
- Backend: GET /benutzer jetzt auch für Material-/Leitungsverantwortliche
  lesbar (für Kontrollverantwortung-Auswahl im Frontend nötig), Schreiben
  bleibt admin-only.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CVgbozhYmuEhiEJHffRXCV
This commit is contained in:
2026-09-04 22:25:32 +02:00
co-authored by Claude Sonnet 5
parent cf97b2e13d
commit 7f5bbb65d5
8 changed files with 180 additions and 51 deletions
+5 -1
View File
@@ -11,10 +11,14 @@ from app.schemas.benutzer import BenutzerCreate, BenutzerRead, BenutzerUpdate
router = APIRouter()
_admin_only = require_roles(RolleTyp.administration)
# Lesend auch für Material-/Leitungsverantwortliche (Rollen-Vorgabe): brauchen
# die Liste u.a. für Kontrollverantwortung/Zuständigkeit-Auswahl im Frontend -
# Anlegen/Ändern bleibt admin-only.
_lesend = require_roles(RolleTyp.administration, RolleTyp.materialverantwortlicher, RolleTyp.leitungsverantwortlicher)
@router.get("/benutzer", response_model=list[BenutzerRead])
async def liste_benutzer(db: AsyncSession = Depends(get_db), _=Depends(_admin_only)) -> list[BenutzerRead]:
async def liste_benutzer(db: AsyncSession = Depends(get_db), _=Depends(_lesend)) -> list[BenutzerRead]:
result = await db.execute(select(Benutzer))
return [BenutzerRead.from_orm_benutzer(b) for b in result.scalars().all()]
+37
View File
@@ -54,3 +54,40 @@ async def test_admin_can_update_rollen(client, admin_user, mitarbeiter_user):
)
assert response.status_code == 200
assert response.json()["rollen"] == ["leitungsverantwortlicher"]
@pytest.mark.asyncio
async def test_materialverantwortlicher_darf_benutzer_lesen_nicht_anlegen(
client, materialverantwortlicher_user
):
"""Rollen-Vorgabe: Material-/Leitungsverantwortliche brauchen die Liste
lesend (z.B. Kontrollverantwortung-Auswahl im Frontend), aber kein
Schreibrecht - anders als Administration."""
token = await login(client, "materialverantwortlicher1")
gelesen = await client.get("/api/v1/benutzer", headers=auth_header(token))
assert gelesen.status_code == 200
angelegt = await client.post(
"/api/v1/benutzer",
json={"name": "X", "login": "x1", "passwort": "sicheres-passwort", "rollen": ["mitarbeiter"]},
headers=auth_header(token),
)
assert angelegt.status_code == 403
@pytest.mark.asyncio
async def test_leitungsverantwortlicher_darf_benutzer_lesen(client, db_session):
from app.core.security import hash_password
from app.models.auth import Benutzer, BenutzerRolle, RolleTyp
benutzer = Benutzer(
name="Test Leitung", login="leitung1", passwort_hash=hash_password("test-passwort-123"), aktiv=True
)
db_session.add(benutzer)
await db_session.flush()
db_session.add(BenutzerRolle(benutzer_id=benutzer.id, rolle=RolleTyp.leitungsverantwortlicher))
await db_session.flush()
token = await login(client, "leitung1")
response = await client.get("/api/v1/benutzer", headers=auth_header(token))
assert response.status_code == 200