Feat : Review | Fix : Coverage Test

Sources/Stim.Model/Review.cs

@@ -48,10 +48,9 @@ namespace Model
         }
         private string authorName;
 
-        public Review(/*string username,*/ float rate, string text)
+        public Review(string username, float rate, string text)
         {
-            //AuthorName = username;
-            AuthorName = authorName;
+            AuthorName = username;
             Rate = rate;
             Text = text;
         }

Sources/Stim.Model/User.cs

@@ -66,16 +66,21 @@ namespace Model
             Email = email;
             Password = password;
             Followed_Games = new List<Game>();
+            Role = 0;
         }
         public void AddReview(Game game, float rate, string text)
         {
-            game.AddReview(new Review(/*username,*/ rate, text));
+            game.AddReview(new Review(username, rate, text));
         }
         public void RemoveSelfReview(Game game, float rate, string text)
         {
-            foreach (Review review in game.Reviews)
+            for (int i = game.Reviews.Count - 1; i >= 0; i--)
             {
-                if (review.Rate == rate && review.Text == text && review.AuthorName == username) game.RemoveReview(review);
+                Review review = game.Reviews[i];
+                if (review.Rate == rate && review.Text == text && review.AuthorName == username)
+                {
+                    game.RemoveReview(review);
+                }
             }
         }
         public void FollowAGame(Game game)

Sources/TestProject1/TestGame.cs

@@ -72,13 +72,13 @@ namespace Test
         [Fact]
         public void AddReview()
         {
-            Review r1 = new(2.5f, "cool"), r2 = new(4, "tres cool"), r3 = new(1, "pas cool");
+            User user = new("username", "biographie", "email@email.com", "password");
 
             Game game = new("name", "description", 2012, new List<String> {"1","2","3"}, "cover");
 
-            game.AddReview(r1);
-            game.AddReview(r2);
-            game.AddReview(r3);
+            user.AddReview(game, 2.5f, "bof");
+            user.AddReview(game, 4f, "bof++");
+            user.AddReview(game, 3f, "bof+");
 
             Assert.NotEmpty(game.Reviews);
         }
@@ -86,16 +86,17 @@ namespace Test
         [Fact]
         public void RemoveReview()
         {
-            Review r1 = new(2.5f, "cool"), r2 = new(4, "tres cool"), r3 = new(1, "pas cool");
-
+            User user = new("username", "biographie", "email@email.com", "password");
+            User user2 = new("username2", "biographie", "email2@email.com", "password");
             Game game = new("name", "description", 2012, new List<String> {"1","2","3"}, "cover");
 
-            game.AddReview(r1);
-            game.AddReview(r2);
-            game.AddReview(r3);
-            game.RemoveReview(r2);
-
-            Assert.DoesNotContain(r2, game.Reviews);
+            user.AddReview(game, 2.5f, "bof");
+            user.AddReview(game, 4f, "bof++");
+            user.AddReview(game, 3f, "bof+");
+            user2.RemoveSelfReview(game, 2.5f, "bof");
+            Assert.Equal(3, game.Reviews.Count);
+            user.RemoveSelfReview(game, 2.5f, "bof");
+            Assert.Equal(2, game.Reviews.Count);
         }
 
         [Fact]
@@ -140,13 +141,13 @@ namespace Test
         [Fact]
         public void Average()
         {
-            Review r1 = new(2.5f, "cool"), r2 = new(4, "tres cool"), r3 = new(1, "pas cool");
+            User user = new("username", "biographie", "email@email.com", "password");
 
             Game game = new("name", "description", 2012, new List<String> {"1","2","3"}, "cover");
 
-            game.AddReview(r1);
-            game.AddReview(r2);
-            game.AddReview(r3);
+            user.AddReview(game, 2.5f, "bof");
+            user.AddReview(game, 0f, "bof--");
+            user.AddReview(game, 5f, "bof++");
 
             Assert.Equal(2.5f, game.GetAvgRate());
         }

Sources/TestProject1/TestUser.cs

@@ -26,7 +26,7 @@ namespace Test
             Assert.Null(user2.Username);
         }
 
-        /*[Fact]
+        [Fact]
         public void Biographie()
         {
             User user = new("username", "", "adresse.mail@gmail.com", "Azerty123*");
@@ -34,7 +34,12 @@ namespace Test
 
             User user2 = new("username", null, "adresse.mail@gmail.com", "Azerty123*");
             Assert.Null(user2.Biographie);
-        }*/
+
+            User user3 = new("username", "biographie", "adresse.mail@gmail.com", "Azerty123*");
+
+            string biographieOfAnUser = user3.Biographie;
+            Assert.NotNull(biographieOfAnUser);
+        }
 
         [Fact]
         public void Email()
@@ -73,13 +78,21 @@ namespace Test
             user.RemoveAGame(game);
             Assert.Empty(user.Followed_Games);
         }
+        [Fact]
+        public void Role()
+        {
+            User user = new("username", "biographie", "adresse.mail@gmail.com", "Azerty123*");
+            Assert.Equal(0,user.Role);
 
-        /*
+            int Perm = user.Role;
+            Assert.True(user.Role == Perm);
+        }
+        
         [Fact]
         public void ReviewAddingAndRemovingFromAGameViaUser()
         {
             User user = new("username", "biographie", "adresse.mail@gmail.com", "Azerty123*");
-            Game game = new("name", "description", 2012, new string[] { "1", "2", "3" });
+            Game game = new("name", "description", 2012, new List<String> { "1", "2", "3" },"cover");
             Assert.NotNull(user);
             Assert.NotNull(game);
 
@@ -90,6 +103,6 @@ namespace Test
             user.RemoveSelfReview(game, 2.5f, "UwU");
             Assert.Empty(game.Reviews);
         }
-        */
+        
     }
 }