...

Qwirkle/QwirkleClassLibrary/Games/Game.cs

@@ -58,6 +58,12 @@ namespace QwirkleClassLibrary.Games
         /// <returns>boolean to check it</returns>
         public bool AddPlayerInGame(List<string> PlayersTag)
         {
+            if (GameRunning)
+            {
+                OnPlayerNotified(new AddPlayerNotifiedEventArgs("ERROR : The game is running."));
+                return false;
+            }
+
             for (int i = PlayersTag.Count - 1; i >= 0; i--)
             {
                 if (string.IsNullOrWhiteSpace(PlayersTag[i]))
@@ -100,12 +106,6 @@ namespace QwirkleClassLibrary.Games
                 return false;
             }
 
-            if (GameRunning)
-            {
-                OnPlayerNotified(new AddPlayerNotifiedEventArgs("ERROR with " + (pos + 1) + " entry : The game is running."));
-                return false;
-            }
-
             for(int i=0; i<PlayersTag.Count; i++)
             {
                 if (i == pos)

Qwirkle/TestBase/TestGame.cs

@@ -10,12 +10,12 @@ public class TestGame
 {
     [Theory]
     [InlineData(true, false, "test", "test1")]
-    [InlineData(false, false, "test", "test1")]
+    [InlineData(false, false, "test", "test")]
-    [InlineData(false, true, "test", "test1")]
     public void Test_GameAddPlayerIngame(bool result, bool gamestate, string p1, string p2)
     {
         Game game = new Game();
         List<string> players = [p1, p2];
+        List<string> players2 = ["patrick", "jeanclaude"];
 
         if (!result)
         {
@@ -29,8 +29,9 @@ public class TestGame
 
         if (gamestate)
         {
+            game.AddPlayerInGame(players);
             game.StartGame();
-            Assert.False(game.AddPlayerInGame(players));
+            Assert.False(game.AddPlayerInGame(players2));
         }
     }