PD-238 — Revue des Tests (v2 - Corrections post-gate)¶

Résumé¶

Verdict : ✅ CONFORME

Points positifs¶

• Suppression des tests constantTimeEqual cohérente avec l’usage de crypto.timingSafeEqual (primitive native fiable).
• Assertions unifiées vers INVALID_CREDENTIALS, conforme à l’objectif anti‑énumération.
• Correction de mockResolvedValue → mockReturnValue pour une méthode synchrone évite des faux positifs.
• Nommage et structure Jest (describe/it) clairs et alignés sur TC‑ERR‑13 / TC‑INV‑01.

Points à améliorer¶

Analyse détaillée¶

Suppression tests constantTimeEqual¶

• Justifiée : la méthode custom n’existe plus et la primitive native ne nécessite pas de test unitaire interne.
• Pas de perte de couverture métier : les tests d’auth/reauth continuent de valider les comportements attendus.

Assertions INVALID_CREDENTIALS¶

• Alignement correct sur la règle ECT‑06 (raison unique pour éviter l’énumération).
• Les assertions vérifient bien l’événement emitSensitiveActionBlocked avec la nouvelle raison.

Correction mockReturnValue¶

• Appropriée pour verifyPassword synchrone : la promesse implicite de mockResolvedValue pouvait masquer des erreurs.
• Les tests restent déterministes et valident l’absence de JWT en cas d’échec.