From cf895b870f301091af8aa85d661ca9468efc47ef Mon Sep 17 00:00:00 2001 From: Jade_VAN_BRABANDT Date: Tue, 6 Jun 2023 18:24:04 +0200 Subject: [PATCH] feat : coverage --- Sources/Stim.Model/Game.cs | 2 +- Sources/Stim.Model/Manager.cs | 6 ++++-- Sources/TestProject1/TestManager.cs | 28 +++++++++++++++++++++++++++- Sources/TestProject1/TestUser.cs | 2 ++ 4 files changed, 34 insertions(+), 4 deletions(-) diff --git a/Sources/Stim.Model/Game.cs b/Sources/Stim.Model/Game.cs index 9cde10f..4f44a57 100644 --- a/Sources/Stim.Model/Game.cs +++ b/Sources/Stim.Model/Game.cs @@ -87,7 +87,7 @@ namespace Model } } } - private ObservableCollection tags = default!; + private ObservableCollection tags; [DataMember] public List Reviews { get; private init; } diff --git a/Sources/Stim.Model/Manager.cs b/Sources/Stim.Model/Manager.cs index 99c8552..b2ee887 100644 --- a/Sources/Stim.Model/Manager.cs +++ b/Sources/Stim.Model/Manager.cs @@ -1,4 +1,5 @@ using System.Collections.ObjectModel; +using System.Diagnostics.CodeAnalysis; using System.Linq; namespace Model @@ -24,7 +25,7 @@ namespace Model IEnumerable retList; retList = GameList; if (filterName != null) retList = retList - .Where(game => game.Name.IndexOf(filterName, StringComparison.OrdinalIgnoreCase) >= 0 + .Where(game => game.Name.Contains(filterName, StringComparison.OrdinalIgnoreCase) ); if (filterTag1 != null) retList = retList .Where(game => game.Tags != null && game.Tags.Any(tag => tag != null && tag.Contains(filterTag1, StringComparison.OrdinalIgnoreCase)) @@ -51,7 +52,7 @@ namespace Model gameList.Remove(game); mgrpersistance.SaveGame(gameList); } - + [ExcludeFromCodeCoverage] public void SaveGames() { mgrpersistance.SaveGame(gameList); @@ -64,6 +65,7 @@ namespace Model } return null; } + [ExcludeFromCodeCoverage] public void SaveUser() { mgrpersistance.SaveUser(Users); diff --git a/Sources/TestProject1/TestManager.cs b/Sources/TestProject1/TestManager.cs index c93279f..036907f 100644 --- a/Sources/TestProject1/TestManager.cs +++ b/Sources/TestProject1/TestManager.cs @@ -121,7 +121,8 @@ namespace Test compListAsList = compList.ToList(); compListAsList.Clear(); - compList = compListAsList; list = manager.FilterGames("Elden Ring", "Action", "Solo"); + compList = compListAsList; + list = manager.FilterGames("Elden Ring", "Action", "Solo"); foreach (var game in manager.GameList) { if (game.Name=="Elden Ring" && game.Tags.Any(tag => tag == "Action" && game.Tags.Any(tag => tag == "Solo"))) @@ -133,6 +134,31 @@ namespace Test } } Assert.Equal(compList, list); + list = manager.FilterGames(null, "Action", "Solo"); + compListAsList = compList.ToList(); + compListAsList.Clear(); + compList = compListAsList; + compList = compListAsList; list = manager.FilterGames("Elden Ring", "Action", null); + foreach (var game in manager.GameList) + { + if (game.Name == "Elden Ring" && game.Tags.Any(tag => tag == "Action")) + { + compListAsList = compList.ToList(); + compListAsList.Add(game); + compList = compListAsList; + break; + } + } + Assert.Equal(compList, list); + } + [Fact] + public void Search() + { + IPersistance persistance = new Stub(); + Manager manager = new(persistance); + User user = new(null, "username", "biographie", "adresse.mail@gmail.com", "Azerty123*"); + manager.AddUsertoUserList(user); + Assert.Equal(user, manager.SearchUser("username")); } } } diff --git a/Sources/TestProject1/TestUser.cs b/Sources/TestProject1/TestUser.cs index 0f65799..b554399 100644 --- a/Sources/TestProject1/TestUser.cs +++ b/Sources/TestProject1/TestUser.cs @@ -108,5 +108,7 @@ namespace Test Assert.False(user.Equals(user4 as object)); Assert.False(user.Equals(user2 as object)); } + [Fact] + public void Hashcode } }