🐛 Replace Equals with SequenceEqual
continuous-integration/drone/push Build is passing Details

still not foolproof, since Die and Face don't redefine equals etc.

the reason why all the tests pass is because we don't have UTs for DiceGroupManager
pull/200/head
Alexis Drai 2 years ago
parent d259884ba8
commit c61ffeb783

@ -69,7 +69,7 @@ namespace Model.Dice
public Task<DiceGroup> Update(DiceGroup before, DiceGroup after)
{
// pas autorisé de changer les dés, juste le nom
if (!before.Dice.Equals(after.Dice))
if (!before.Dice.SequenceEqual(after.Dice))
{
throw new ArgumentException("the group of dice cannot be updated, only the name", nameof(before));
}

Loading…
Cancel
Save