[G1TS23]

Contexte

Yams.computeScore ne vérifiait jamais que le roll passé en paramètre contenait exactement 5 dés. Un roll de taille différente (4, 6, 0...) était accepté silencieusement et produisait un score calculé sur une base erronée.

Closes #1

Changements

• Démarche TDD conservée : FAIL -> PASS -> REFACTOR
• Ajout de la validation roll.size() != DICES_AMOUNT en tout début de computeScore, qui lève désormais une exception avant tout calcul de score
• Création de InvalidRollSizeError (extends IllegalArgumentException), avec un message incluant à la fois la taille attendue et la taille réellement reçue
• DICES_AMOUNT passée de private final à static final dans Yams, pour servir de source unique de vérité partagée avec InvalidRollSizeError
• Ajout de tests couvrant les rolls trop courts, trop longs, et vides

Test plan

• Nouveau test shouldThrowIllegalArgumentExceptionWhenGivenLessThanFiveDices
• Nouveau test shouldThrowIllegalArgumentExceptionWhenGivenMoreThanFiveDices
• Nouveau test shouldThrowIllegalArgumentExceptionWhenGivenNoDice
• Suite de tests existante toujours verte

[vbetsch]

reviewed ✔️