Gestion des référents depuis la fiche bénévole
Un poste avait bien un `lead_account_id`, lu par l'écran référent
(`/mobile-referent`) et par les droits d'écriture (`ensure_can_write_position`),
mais rien ne permettait de le renseigner : aucun écran ne l'exposait, et
l'API ne validait même pas que la valeur envoyée désignait un compte référent
existant de la bonne édition.
Backend :
- `PATCH /editions/{id}/accounts/{account_id}` (nouveau) : change le `role`
d'un compte, seulement entre LEAD et VOLUNTEER (lien magique dans les deux
cas) — 422 `account.role_change_not_allowed` dès que COORDINATOR est
impliqué (mot de passe en jeu, jamais touché ici). C'est ce qui permet de
faire passer le compte d'un bénévole en référent.
- `POST`/`PATCH /positions` : `lead_account_id` vérifié — doit désigner un
compte `Role.LEAD` de la même édition, sinon 422
`position.lead_must_be_referent_account` (silencieusement accepté
jusqu'ici, y compris un compte d'une autre édition ou du mauvais rôle).
Frontend (`ConfigVolunteers.tsx`) : nouvelle section « Référent des postes »
sur la fiche du bénévole, juste après l'accès de connexion — mêmes badges à
cocher que les vœux de poste/artiste. Cocher un poste crée ou met à niveau
(LEAD) l'accès de connexion du bénévole automatiquement ; décocher retire
seulement le poste, sans jamais rétrograder l'accès. Un bénévole dont le
compte est déjà COORDINATOR ne peut pas être désigné référent (il a déjà
accès à tout). Miroir complet côté mock (`accounts.update`,
`checkLeadAccount`) pour que la démonstration se comporte pareil.
Vérifié : 204 tests backend pertinents (14 nouveaux : 6 sur `PATCH accounts`,
8 sur la validation `lead_account_id`), ruff, alembic check ; typecheck/
lint/build/smoke frontend ; parcours complet testé en direct sur le jeu de
démo (création de compte, promotion LEAD, rejet COORDINATOR, affectation
référent) avant nettoyage des données de test.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017uUsRg7XzopkCPWimwg9ZM
This commit is contained in:
AdamThomas
co-authored by
Claude Sonnet 5
Claude Sonnet 5
parent
db11d9a3ba
commit
8fcc22115f
@@ -9,7 +9,7 @@ from sqlmodel import Session, select
|
||||
|
||||
from app.api._common import owned_or_404
|
||||
from app.api.deps import get_edition
|
||||
from app.api.schemas import AccountCreate, AccountRead
|
||||
from app.api.schemas import AccountCreate, AccountRead, AccountUpdate
|
||||
from app.auth.deps import require_coordinator
|
||||
from app.auth.passwords import MIN_LENGTH, hash_password
|
||||
from app.db import get_session
|
||||
@@ -85,6 +85,30 @@ def list_accounts(
|
||||
)
|
||||
|
||||
|
||||
@router.patch("/{account_id}", response_model=AccountRead)
|
||||
def update_account(
|
||||
account_id: uuid.UUID,
|
||||
payload: AccountUpdate,
|
||||
edition: Edition = Depends(get_edition),
|
||||
_guard: Account = Depends(require_coordinator),
|
||||
session: Session = Depends(get_session),
|
||||
) -> Account:
|
||||
"""Seul le rôle change de main ici, et seulement entre LEAD et VOLUNTEER
|
||||
(lien magique dans les deux cas) : devenir référent d'un poste (voir
|
||||
`positions.py`) met à niveau le compte bénévole existant, sans toucher au
|
||||
mot de passe d'un compte COORDINATION."""
|
||||
account = owned_or_404(session.get(Account, account_id), edition.id)
|
||||
if payload.role is not None and payload.role != account.role:
|
||||
if Role.COORDINATOR in (account.role, payload.role):
|
||||
raise HTTPException(
|
||||
status_code=422, detail={"code": "account.role_change_not_allowed"}
|
||||
)
|
||||
account.role = payload.role
|
||||
session.commit()
|
||||
session.refresh(account)
|
||||
return account
|
||||
|
||||
|
||||
@router.delete("/{account_id}", status_code=status.HTTP_204_NO_CONTENT)
|
||||
def delete_account(
|
||||
account_id: uuid.UUID,
|
||||
|
||||
@@ -1,8 +1,13 @@
|
||||
"""CRUD des postes."""
|
||||
"""CRUD des postes.
|
||||
|
||||
`lead_account_id` (référent) ne peut désigner qu'un compte `Role.LEAD` de la
|
||||
même édition — voir `_check_lead_account`. C'est le seul moyen de donner à un
|
||||
bénévole une vue référent (`/mobile-referent`) à la connexion ; voir
|
||||
`api/accounts.py:update_account` pour le faire passer de VOLUNTEER à LEAD."""
|
||||
|
||||
import uuid
|
||||
|
||||
from fastapi import APIRouter, Depends, Response, status
|
||||
from fastapi import APIRouter, Depends, HTTPException, Response, status
|
||||
from sqlmodel import Session, select
|
||||
|
||||
from app.api._common import owned_or_404
|
||||
@@ -21,13 +26,28 @@ from app.auth.deps import (
|
||||
require_staff,
|
||||
)
|
||||
from app.db import get_session
|
||||
from app.models import Account, Edition, Position
|
||||
from app.models import Account, Edition, Position, Role
|
||||
|
||||
router = APIRouter(
|
||||
prefix="/editions/{edition_id}/positions", tags=["positions"]
|
||||
)
|
||||
|
||||
|
||||
def _check_lead_account(
|
||||
session: Session, edition: Edition, lead_account_id: uuid.UUID | None
|
||||
) -> None:
|
||||
"""`lead_account_id` doit désigner un compte RÉFÉRENT (`Role.LEAD`) de
|
||||
cette même édition — sinon le poste se retrouve sans référent utilisable
|
||||
(cf. `ensure_can_write_position`, `auth/deps.py`)."""
|
||||
if lead_account_id is None:
|
||||
return
|
||||
lead = session.get(Account, lead_account_id)
|
||||
if lead is None or lead.edition_id != edition.id or lead.role is not Role.LEAD:
|
||||
raise HTTPException(
|
||||
status_code=422, detail={"code": "position.lead_must_be_referent_account"}
|
||||
)
|
||||
|
||||
|
||||
@router.post("", response_model=PositionRead, status_code=status.HTTP_201_CREATED)
|
||||
def create_position(
|
||||
payload: PositionCreate,
|
||||
@@ -35,6 +55,7 @@ def create_position(
|
||||
_guard: Account = Depends(require_coordinator),
|
||||
session: Session = Depends(get_session),
|
||||
) -> Position:
|
||||
_check_lead_account(session, edition, payload.lead_account_id)
|
||||
position = Position(edition_id=edition.id, **payload.model_dump())
|
||||
session.add(position)
|
||||
session.flush()
|
||||
@@ -80,7 +101,9 @@ def update_position(
|
||||
) -> Position:
|
||||
position = owned_or_404(session.get(Position, position_id), edition.id)
|
||||
ensure_can_write_position(account, position)
|
||||
for field, value in payload.model_dump(exclude_unset=True).items():
|
||||
changes = payload.model_dump(exclude_unset=True)
|
||||
_check_lead_account(session, edition, changes.get("lead_account_id"))
|
||||
for field, value in changes.items():
|
||||
setattr(position, field, value)
|
||||
session.commit()
|
||||
session.refresh(position)
|
||||
|
||||
@@ -363,6 +363,12 @@ class AccountCreate(ApiModel):
|
||||
password: str | None = None
|
||||
|
||||
|
||||
class AccountUpdate(ApiModel):
|
||||
# Seul le rôle est modifiable, et seulement entre LEAD et VOLUNTEER (pas
|
||||
# de mot de passe en jeu) — voir `api/accounts.py:update_account`.
|
||||
role: Role | None = None
|
||||
|
||||
|
||||
# --------------------------------------------------------------------------- #
|
||||
# Notifications par email #
|
||||
# --------------------------------------------------------------------------- #
|
||||
|
||||
@@ -10,6 +10,7 @@
|
||||
"day_part.duplicate_code": "Un créneau type porte déjà ce code dans l'édition.",
|
||||
"shift.ends_not_after_starts": "La fin du créneau doit être postérieure à son début.",
|
||||
"shift.position_wrong_edition": "Le poste n'appartient pas à cette édition.",
|
||||
"position.lead_must_be_referent_account": "Le référent d'un poste doit être un compte référent (rôle LEAD) de cette édition.",
|
||||
"artist.name_required": "Le nom de l'artiste ne peut pas être vide.",
|
||||
"artist.duplicate": "Un artiste porte déjà ce nom à cette date dans l'édition.",
|
||||
"artist.unknown": "Un des artistes choisis n'appartient pas à cette édition.",
|
||||
@@ -25,6 +26,7 @@
|
||||
"account.volunteer_wrong_edition": "Le bénévole n'appartient pas à cette édition.",
|
||||
"account.password_required": "Un compte coordination requiert un mot de passe d'au moins {min} caractères.",
|
||||
"account.password_not_allowed": "Un compte référent ou bénévole se connecte par lien magique, sans mot de passe.",
|
||||
"account.role_change_not_allowed": "Un compte coordination ne peut pas changer de rôle ainsi.",
|
||||
"import.error.empty_file": "Le fichier est vide.",
|
||||
"import.error.encoding": "L'encodage du fichier n'est pas reconnu.",
|
||||
"import.error.unreadable_file": "Fichier illisible ou format non pris en charge.",
|
||||
|
||||
@@ -0,0 +1,51 @@
|
||||
import uuid
|
||||
|
||||
from app.models import Role
|
||||
|
||||
|
||||
def _url(edition, suffix=""):
|
||||
return f"/editions/{edition.id}/accounts{suffix}"
|
||||
|
||||
|
||||
def test_promotes_a_volunteer_account_to_lead(client, edition, make_account):
|
||||
account = make_account(Role.VOLUNTEER, email="v@ex.fr")
|
||||
resp = client.patch(_url(edition, f"/{account.id}"), json={"role": "LEAD"})
|
||||
assert resp.status_code == 200
|
||||
assert resp.json()["role"] == "LEAD"
|
||||
assert resp.json()["auth_method"] == "magic_link"
|
||||
|
||||
|
||||
def test_demotes_a_lead_account_to_volunteer(client, edition, make_account):
|
||||
account = make_account(Role.LEAD, email="l@ex.fr")
|
||||
resp = client.patch(_url(edition, f"/{account.id}"), json={"role": "VOLUNTEER"})
|
||||
assert resp.status_code == 200
|
||||
assert resp.json()["role"] == "VOLUNTEER"
|
||||
|
||||
|
||||
def test_cannot_change_role_of_a_coordinator_account(client, edition, make_account):
|
||||
account = make_account(Role.COORDINATOR, email="c@ex.fr", password="password123")
|
||||
resp = client.patch(_url(edition, f"/{account.id}"), json={"role": "LEAD"})
|
||||
assert resp.status_code == 422
|
||||
assert resp.json()["detail"]["code"] == "account.role_change_not_allowed"
|
||||
|
||||
|
||||
def test_cannot_promote_a_volunteer_to_coordinator(client, edition, make_account):
|
||||
account = make_account(Role.VOLUNTEER, email="v2@ex.fr")
|
||||
resp = client.patch(_url(edition, f"/{account.id}"), json={"role": "COORDINATOR"})
|
||||
assert resp.status_code == 422
|
||||
assert resp.json()["detail"]["code"] == "account.role_change_not_allowed"
|
||||
|
||||
|
||||
def test_unknown_account_is_404(client, edition):
|
||||
assert (
|
||||
client.patch(_url(edition, f"/{uuid.uuid4()}"), json={"role": "LEAD"}).status_code
|
||||
== 404
|
||||
)
|
||||
|
||||
|
||||
def test_volunteer_role_forbidden(edition, make_account, as_account):
|
||||
volunteer_account = make_account(Role.VOLUNTEER, email="v3@ex.fr")
|
||||
other = make_account(Role.VOLUNTEER, email="v4@ex.fr")
|
||||
api = as_account(volunteer_account)
|
||||
resp = api.patch(_url(edition, f"/{other.id}"), json={"role": "LEAD"})
|
||||
assert resp.status_code == 403
|
||||
@@ -1,5 +1,7 @@
|
||||
import uuid
|
||||
|
||||
from app.models import Role
|
||||
|
||||
|
||||
def _url(edition, suffix=""):
|
||||
return f"/editions/{edition.id}/positions{suffix}"
|
||||
@@ -35,3 +37,75 @@ def test_unknown_edition_is_404(client):
|
||||
assert (
|
||||
client.get(f"/editions/{uuid.uuid4()}/positions").status_code == 404
|
||||
)
|
||||
|
||||
|
||||
def test_sets_a_referent(client, edition, make_account):
|
||||
lead = make_account(Role.LEAD, email="referent@ex.fr")
|
||||
created = client.post(
|
||||
_url(edition),
|
||||
json={"name": "Bar", "category": "BAR"},
|
||||
)
|
||||
pid = created.json()["id"]
|
||||
|
||||
resp = client.patch(_url(edition, f"/{pid}"), json={"lead_account_id": str(lead.id)})
|
||||
assert resp.status_code == 200
|
||||
assert resp.json()["lead_account_id"] == str(lead.id)
|
||||
|
||||
|
||||
def test_referent_must_have_the_lead_role(client, edition, make_account):
|
||||
volunteer_account = make_account(Role.VOLUNTEER, email="v@ex.fr")
|
||||
created = client.post(_url(edition), json={"name": "Bar", "category": "BAR"})
|
||||
pid = created.json()["id"]
|
||||
|
||||
resp = client.patch(
|
||||
_url(edition, f"/{pid}"), json={"lead_account_id": str(volunteer_account.id)}
|
||||
)
|
||||
assert resp.status_code == 422
|
||||
assert resp.json()["detail"]["code"] == "position.lead_must_be_referent_account"
|
||||
|
||||
|
||||
def test_referent_must_belong_to_the_same_edition(client, edition, session, make_account):
|
||||
import datetime as dt
|
||||
|
||||
from app.models import Edition
|
||||
|
||||
other_edition = Edition(year=2027, start_date=dt.date(2027, 7, 1), end_date=dt.date(2027, 7, 4))
|
||||
session.add(other_edition)
|
||||
session.commit()
|
||||
session.refresh(other_edition)
|
||||
|
||||
other_lead = make_account(Role.LEAD, email="stranger@ex.fr")
|
||||
other_lead.edition_id = other_edition.id
|
||||
session.add(other_lead)
|
||||
session.commit()
|
||||
|
||||
created = client.post(_url(edition), json={"name": "Bar", "category": "BAR"})
|
||||
pid = created.json()["id"]
|
||||
|
||||
resp = client.patch(
|
||||
_url(edition, f"/{pid}"), json={"lead_account_id": str(other_lead.id)}
|
||||
)
|
||||
assert resp.status_code == 422
|
||||
assert resp.json()["detail"]["code"] == "position.lead_must_be_referent_account"
|
||||
|
||||
|
||||
def test_unknown_referent_is_422(client, edition):
|
||||
created = client.post(_url(edition), json={"name": "Bar", "category": "BAR"})
|
||||
pid = created.json()["id"]
|
||||
|
||||
resp = client.patch(
|
||||
_url(edition, f"/{pid}"), json={"lead_account_id": str(uuid.uuid4())}
|
||||
)
|
||||
assert resp.status_code == 422
|
||||
assert resp.json()["detail"]["code"] == "position.lead_must_be_referent_account"
|
||||
|
||||
|
||||
def test_clearing_the_referent_is_allowed(client, edition, make_account):
|
||||
lead = make_account(Role.LEAD, email="referent2@ex.fr")
|
||||
created = client.post(_url(edition), json={"name": "Bar", "category": "BAR"})
|
||||
pid = created.json()["id"]
|
||||
client.patch(_url(edition, f"/{pid}"), json={"lead_account_id": str(lead.id)})
|
||||
|
||||
resp = client.patch(_url(edition, f"/{pid}"), json={"lead_account_id": None})
|
||||
assert resp.status_code == 200
|
||||
assert resp.json()["lead_account_id"] is None
|
||||
|
||||
Reference in New Issue
Block a user