diff --git a/Sources/BowlingLib/Model/Equipe.cs b/Sources/BowlingLib/Model/Equipe.cs index 402c045..f2fb05d 100644 --- a/Sources/BowlingLib/Model/Equipe.cs +++ b/Sources/BowlingLib/Model/Equipe.cs @@ -54,9 +54,8 @@ namespace BowlingLib.Model return true; }else { - throw new ArgumentException("Le joueur existe déjà dans l'équipe"); - } return false; + } } public void SupprimerJoueur(Joueur joueur) diff --git a/Sources/BowlingLib/Model/Joueur.cs b/Sources/BowlingLib/Model/Joueur.cs index 70f95aa..660acd0 100644 --- a/Sources/BowlingLib/Model/Joueur.cs +++ b/Sources/BowlingLib/Model/Joueur.cs @@ -27,5 +27,11 @@ namespace BowlingLib.Model private set { pseudo = value; } } + public override bool Equals(object obj) + { + return obj is Joueur joueur && + pseudo == joueur.pseudo && + Pseudo == joueur.Pseudo; + } } } diff --git a/Sources/Tests/BowlingAppUnitTest/UTestEquipe.cs b/Sources/Tests/BowlingAppUnitTest/UTestEquipe.cs index 0ca632e..e0287bc 100644 --- a/Sources/Tests/BowlingAppUnitTest/UTestEquipe.cs +++ b/Sources/Tests/BowlingAppUnitTest/UTestEquipe.cs @@ -44,7 +44,6 @@ namespace Test.BowlingAppUnitTest new Joueur("Bénita"), new Joueur("Regis"), new Joueur("Mania")), - new Joueur("Mania") }; } @@ -53,14 +52,14 @@ namespace Test.BowlingAppUnitTest [Theory] [MemberData(nameof(Data_AddJoueurToEquipe))] public void Test_AddJoueurToEquipe(bool expectedResult, - IEnumerable expectedJoueurs, + Joueur[] expectedJoueurs, Equipe equipe, Joueur joueur) { bool result = equipe.AjouterJoueur(joueur); Assert.Equal(expectedResult, result); - Assert.Equal(expectedJoueurs., equipe.GetJoueurs()); + Assert.Equal(expectedJoueurs.Length, equipe.GetJoueurs()); Assert.All(expectedJoueurs, j => equipe.Joueurs.Contains(j)); }