Files
benevoles/backend/tests/test_api_solver.py
T
AdamThomasandClaude Sonnet 5 11bc8cbe36 Vœu de compagnon : traité comme réciproque par le solveur
Le vœu reste à sens unique en base (A choisit B sur sa fiche, B n'a rien à
faire) mais si A souhaite B, B doit désormais profiter aussi de la présence
de A — dans les deux sens : affecter A d'abord fait remonter B comme
candidat, et affecter B d'abord fait remonter A, même si B n'a rien demandé.

- Solveur (`loader.py`) : le vœu (A → B) alimente maintenant l'ensemble
  effectif des deux bénévoles (A et B), pas seulement celui du demandeur.
  Aucun changement dans `engine.py` : il ne fait que lire un ensemble déjà
  symétrisé.
- Mock (`buildEffectiveCompanions`) et classement par défaut
  (`buildEffectiveCompanionIds` dans `lib/planning.ts`, utilisé par
  `AssignmentPanel.tsx`) : même symétrisation, pour rester cohérents avec le
  vrai solveur.
- La fiche bénévole (vœux affichés/édités) n'est pas touchée : c'est le seul
  endroit qui reste à sens unique, par construction (la réciprocité n'est
  qu'une lecture, jamais une donnée enregistrée — sinon éditer sa propre
  liste risquerait d'effacer le vœu de quelqu'un d'autre).
- Nouveau test d'intégration : le bénévole qui n'a rien demandé profite quand
  même de la présence de celui qui l'a choisi.
- CLAUDE.md à jour.

Vérifié : 193 tests backend pertinents (dont le nouveau), ruff, alembic
check ; typecheck/lint/build/smoke frontend.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017uUsRg7XzopkCPWimwg9ZM
2026-09-15 22:32:02 +02:00

189 lines
7.0 KiB
Python

import datetime as dt
import uuid
from app.models import Role
from tests.factories import dt_at, make_artist, make_position, make_shift, make_volunteer
def _url(edition, suffix=""):
return f"/editions/{edition.id}/solver{suffix}"
def test_suggests_a_free_volunteer_for_an_understaffed_shift(client, edition, session):
position = make_position(session, edition)
shift = make_shift(
session, edition, position,
start=dt_at(10, 14), end=dt_at(10, 18), min_staff=1, ideal_staff=1,
)
volunteer = make_volunteer(session, edition, email="v@ex.fr")
resp = client.post(_url(edition, "/suggestions"))
assert resp.status_code == 200
body = resp.json()
assert body["shifts_considered"] == 1
assert body["volunteers_considered"] == 1
suggestion = next(s for s in body["suggestions"] if s["shift_id"] == str(shift.id))
assert suggestion["volunteer_ids"] == [str(volunteer.id)]
def test_fully_staffed_shift_has_no_suggestion(client, edition, session):
from app.models import Assignment
position = make_position(session, edition)
shift = make_shift(
session, edition, position,
start=dt_at(10, 14), end=dt_at(10, 18), min_staff=1, ideal_staff=1,
)
volunteer = make_volunteer(session, edition, email="v@ex.fr")
session.add(
Assignment(edition_id=edition.id, volunteer_id=volunteer.id, shift_id=shift.id)
)
session.commit()
resp = client.post(_url(edition, "/suggestions"))
assert resp.status_code == 200
shift_ids = {s["shift_id"] for s in resp.json()["suggestions"]}
assert str(shift.id) not in shift_ids
def test_avoids_volunteer_whose_wished_artist_would_be_missed(client, edition, session):
position = make_position(session, edition)
shift = make_shift(
session, edition, position,
start=dt_at(10, 14), end=dt_at(10, 18), min_staff=1, ideal_staff=1,
)
artist = make_artist(
session, edition,
performance_date=dt.date(2026, 7, 10),
start_time=dt.time(15, 0), end_time=dt.time(16, 0),
)
fan = make_volunteer(session, edition, email="fan@ex.fr")
assert client.patch(
f"/editions/{edition.id}/volunteers/{fan.id}",
json={"artist_ids": [str(artist.id)]},
).status_code == 200
indiff = make_volunteer(session, edition, email="indiff@ex.fr")
resp = client.post(_url(edition, "/suggestions"))
suggestion = next(
s for s in resp.json()["suggestions"] if s["shift_id"] == str(shift.id)
)
assert suggestion["volunteer_ids"] == [str(indiff.id)]
def test_prefers_volunteer_whose_wished_companion_is_already_on_the_shift(
client, edition, session
):
from app.models import Assignment
position = make_position(session, edition)
shift = make_shift(
session, edition, position,
start=dt_at(10, 14), end=dt_at(10, 18), min_staff=1, ideal_staff=2,
)
copain = make_volunteer(session, edition, email="copain@ex.fr")
session.add(
Assignment(edition_id=edition.id, volunteer_id=copain.id, shift_id=shift.id)
)
session.commit()
buddy = make_volunteer(session, edition, email="buddy@ex.fr")
assert client.patch(
f"/editions/{edition.id}/volunteers/{buddy.id}",
json={"wished_companion_ids": [str(copain.id)]},
).status_code == 200
stranger = make_volunteer(session, edition, email="stranger@ex.fr")
resp = client.post(_url(edition, "/suggestions"))
suggestion = next(
s for s in resp.json()["suggestions"] if s["shift_id"] == str(shift.id)
)
assert suggestion["volunteer_ids"] == [str(buddy.id)]
assert str(stranger.id) not in suggestion["volunteer_ids"]
def test_companion_bonus_applies_even_to_the_one_who_did_not_ask(client, edition, session):
# Le vœu de compagnon est à sens unique en base, mais traité comme
# réciproque par le solveur : « wisher » (déjà affecté) a souhaité être
# avec « wished », qui lui n'a rien demandé — il en profite quand même.
from app.models import Assignment
position = make_position(session, edition)
shift = make_shift(
session, edition, position,
start=dt_at(10, 14), end=dt_at(10, 18), min_staff=1, ideal_staff=2,
)
wisher = make_volunteer(session, edition, email="wisher@ex.fr")
session.add(
Assignment(edition_id=edition.id, volunteer_id=wisher.id, shift_id=shift.id)
)
session.commit()
wished = make_volunteer(session, edition, email="wished@ex.fr")
assert client.patch(
f"/editions/{edition.id}/volunteers/{wisher.id}",
json={"wished_companion_ids": [str(wished.id)]},
).status_code == 200
stranger = make_volunteer(session, edition, email="stranger@ex.fr")
resp = client.post(_url(edition, "/suggestions"))
suggestion = next(
s for s in resp.json()["suggestions"] if s["shift_id"] == str(shift.id)
)
assert suggestion["volunteer_ids"] == [str(wished.id)]
assert str(stranger.id) not in suggestion["volunteer_ids"]
def test_chain_penalty_applies_across_different_positions(client, edition, session):
from app.models import Assignment, PositionCategory
# Deux postes différents, même jour, repos suffisant (8 h pile) entre les
# deux créneaux : seul le chaînage (n'importe quel poste) doit
# différencier les deux bénévoles pour le second créneau.
bar = make_position(session, edition, name="Bar")
entrance = make_position(session, edition, name="Entrée", category=PositionCategory.WELCOME)
morning_shift = make_shift(
session, edition, bar,
start=dt_at(10, 8), end=dt_at(10, 12), min_staff=1, ideal_staff=1,
)
evening_shift = make_shift(
session, edition, entrance,
start=dt_at(10, 20), end=dt_at(10, 23), min_staff=1, ideal_staff=1,
)
chained = make_volunteer(session, edition, email="chained@ex.fr")
session.add(
Assignment(edition_id=edition.id, volunteer_id=chained.id, shift_id=morning_shift.id)
)
session.commit()
fresh = make_volunteer(session, edition, email="fresh@ex.fr")
resp = client.post(_url(edition, "/suggestions"))
suggestion = next(
s for s in resp.json()["suggestions"] if s["shift_id"] == str(evening_shift.id)
)
assert suggestion["volunteer_ids"] == [str(fresh.id)]
def test_unknown_edition_is_404(client):
assert client.post(f"/editions/{uuid.uuid4()}/solver/suggestions").status_code == 404
def test_volunteer_role_forbidden(edition, session, make_account, as_account):
account = make_account(Role.VOLUNTEER, email="v@e.fr")
api = as_account(account)
assert api.post(_url(edition, "/suggestions")).status_code == 403
def test_does_not_persist_any_assignment(client, edition, session):
from sqlmodel import select
from app.models import Assignment
position = make_position(session, edition)
make_shift(session, edition, position, start=dt_at(10, 14), end=dt_at(10, 18))
make_volunteer(session, edition, email="v@ex.fr")
client.post(_url(edition, "/suggestions"))
assert session.exec(select(Assignment)).all() == []