PD-238 — Revue de Code (v2 - Corrections post-gate)¶

Résumé¶

Critère	Statut
Correction ECT-04	✅
Correction ECT-06	✅
Correction mockReturnValue	✅
Qualité code	✅

Verdict : ✅ CONFORME

Suppression des tests constantTimeEqual cohérente avec la migration vers crypto.timingSafeEqual (natif Node.js).
Unification des raisons d’échec en INVALID_CREDENTIALS conforme à l’objectif anti‑énumération.
Correction du mock (mockReturnValue pour méthode synchrone) élimine un risque de faux positifs.
Commentaires explicites sur les changements ECT‑04/ECT‑06 et maintien des assertions clés.

ID	Description	Ligne	Gravité
—	Aucun écart résiduel identifié sur les corrections demandées.	—	—

✅ Les tests dédiés ont été supprimés avec un commentaire justifiant la non‑nécessité de tests unitaires pour une primitive native.
✅ Pas de perte de couverture fonctionnelle sur le flux reauth (les tests vérifient toujours le résultat final).

✅ Les assertions emitSensitiveActionBlocked ont été mises à jour sur INVALID_CREDENTIALS.
✅ Les descriptions de tests reflètent correctement la nouvelle sémantique.

✅ verifyPassword étant synchrone, le passage à mockReturnValue(false) est correct et évite des Promises implicites.
✅ Les tests d’erreur restent déterministes (échec reauth + absence de JWT).