[TECH] Sortir les erreurs jetées dans les repositories de certification/enrolment (PIX-23893).

[alexandrecoin]

☀️ Problème

Plusieurs repositories du sous-contexte certification/enrolment lèvent des erreurs du domaine lorsqu'ils ne trouvent pas de ligne en base.
L'infrastructure porte ainsi une décision métier alors que son seul rôle devrait être de rapporter ce qu'elle a trouvé, ou non.

⛱️ Proposition

Les repositories renvoient désormais null au lieu de lever une erreur, et la décision métier remonte d'un cran :

Repositories (enrolment/infrastructure)

Repository	Méthode	Avant	Après
session-repository	get	throw NotFoundError	return null
session-repository	remove	throw NotFoundError	return null
session-for-attendance-sheet-repository	getWithCertificationCandidates	throw NotFoundError	return null
candidate-repository	update	throw CertificationCandidateNotFoundError	retourne le candidat mis à jour

🧴 Remarques

• Le refacto porte uniquement sur certification/enrolment

• unfinalize-session introduit un changement de comportement, pas seulement un déplacement d'erreur : finalizedSessionRepository.remove (dans session-management, non touché par la branche) n'a jamais levé d'erreur. Avant, dé-finaliser une session absente de finalized-sessions passait silencieusement et appelait quand même sessionManagementRepository.unfinalize.

• ⚠️ Point d'attention pour la revue : sur GET/PATCH d'une session inexistante, le 404 n'est plus produit par l'error-manager mais directement par le contrôleur. Le statut est identique, mais le corps de la réponse n'est plus un payload d'erreur JSON:API. (Renvoi d'un statusCode 404 au lieu d'une erreur)

  🏊 Pour tester

  • Tests verts.
  • Non-régression sur les erreurs levées dans le sous-contexte enrolment :
    • GET/PATCH d'une session inexistante → 404
    • suppression d'une session inexistante → 404
    • PV de session (feuille d'émargement) sans session ou sans candidat inscrit → 404
    • téléchargement du modèle d'import de candidats sur une session inexistante → 404
    • ajout d'un candidat / enrôlement d'élèves SCO / import ODS sur une session inaccessible → 404
    • modification d'un candidat inexistant → erreur candidat non trouvé
    • dé-finalisation d'une session non finalisée → 404

[pix-bot-github]

Choisir les applications à déployer :

• API | 👩‍💻 Dashboard Scalingo
• App (.fr) – App (.org) | 👩‍💻 Dashboard Scalingo
• Orga (.fr) – Orga (.org) | 👩‍💻 Dashboard Scalingo
• Certif (.fr) – Certif (.org) | 👩‍💻 Dashboard Scalingo
• Junior | 👩‍💻 Dashboard Scalingo
• Admin | 👩‍💻 Dashboard Scalingo
• API MaDDo | 👩‍💻 Dashboard Scalingo
• Audit Logger | 👩‍💻 Dashboard Scalingo

Important

N'oubliez pas de déployer l'API pour pouvoir accéder aux fronts et/ou à l’API MaDDo.