🚧 WIP

Sources/Model/Games/GameRunner.cs

@@ -22,6 +22,10 @@ namespace Model.Games
             this.games = games ?? new();
         }
 
+        public GameRunner(IManager<Player> globalPlayerManager, IManager<KeyValuePair<string, IEnumerable<AbstractDie<AbstractDieFace>>>> globalDieManager)
+            : this(globalPlayerManager, globalDieManager, null){ }
+
+
         public IEnumerable<Game> GetAll() => games.AsEnumerable();
 
         /// <summary>
@@ -44,18 +48,21 @@ namespace Model.Games
         /// <summary>
         /// saves a given game -- does not allow copies yet: if a game with the same name exists, it is overwritten
         /// </summary>
-        /// <param name="game">a game to save</param>
+        /// <param name="toAdd">a game to save</param>
         /// <exception cref="NotSupportedException"></exception>
-        public Game Add(Game game)
+        public Game Add(Game toAdd)
         {
-            if (game != null)
+            if (toAdd is null)
+            {
+                throw new ArgumentNullException(nameof(toAdd), "param should not be null");
+            }
+            else
             {
-                games.Remove(games.FirstOrDefault(g => g.Name == game.Name));
+                games.Remove(games.FirstOrDefault(g => g.Name == toAdd.Name));
                 // will often be an update: if game with that name exists, it is removed, else, nothing happens above
-                games.Add(game);
+                games.Add(toAdd);
-                return game;
+                return toAdd;
             }
-            return null;
         }
 
         /// <summary>

Sources/Tests/Model_UTs/GameRunnerTest.cs

@@ -0,0 +1,262 @@
+using Model.Games;
+using Model.Dice;
+using Model.Dice.Faces;
+using Model.Players;
+using Data;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Xunit;
+
+namespace Tests.Model_UTs
+{
+    public class GameRunnerTest
+    {
+        private readonly GameRunner stubGameRunner;
+        public GameRunnerTest()
+        {
+            stubGameRunner = new Stub().LoadApp();
+        }
+
+        [Fact]
+        public void TestConstructorWhenNoGamesThenNewIEnumerable()
+        {
+            // Arrange
+            GameRunner gameRunner = new(new PlayerManager(), new DieManager());
+            IEnumerable<Game> expected;
+            IEnumerable<Game> actual;
+
+            // Act
+            expected = new List<Game>().AsEnumerable();
+            actual = gameRunner.GetAll();
+
+            // Assert
+            Assert.Equal(expected, actual);
+        }
+
+        [Fact]
+        public void TestConstructorWhenGamesThenGamesIEnumerable()
+        {
+            // Arrange
+            GameRunner gameRunner = new(new PlayerManager(), new DieManager(), stubGameRunner.GetAll().ToList());
+            IEnumerable<Game> expected;
+            IEnumerable<Game> actual;
+
+            // Act
+            expected = stubGameRunner.GetAll();
+            actual = gameRunner.GetAll();
+
+            // Assert
+            Assert.Equal(expected, actual);
+        }
+
+        [Fact]
+        public void TestAddWhenGamesThenDoAddAndReturnGames()
+        {
+            // Arrange
+            GameRunner gameRunner = new(new PlayerManager(), new DieManager());
+            Game game1 = stubGameRunner.GetAll().First();
+            Game game2 = stubGameRunner.GetAll().Last();
+
+            // Act
+            IEnumerable<Game> expected = new List<Game>() { game1, game2 }.AsEnumerable();
+            IEnumerable<Game> actual = new List<Game>()
+            {
+                gameRunner.Add(game1),
+                gameRunner.Add(game2)
+            };
+
+            // Assert
+            Assert.Equal(expected, actual);
+        }
+
+        [Fact]
+        public void TestAddwhenNullThenThrowsException()
+        {
+            // Arrange
+
+
+            // Act
+            void action() => stubGameRunner.Add(null);// Add() returns the added element if succesful
+
+            // Assert
+            Assert.Throws<ArgumentNullException>(action);
+            Assert.DoesNotContain(null, stubGameRunner.GetAll());
+        }
+
+        [Theory]
+        [InlineData("")]
+        [InlineData(null)]
+        [InlineData(" ")]
+        public void TestGetOneByNameIfInvalidThrowsException(string name)
+        {
+            // Arrange
+
+
+            // Act
+            void action() => stubGameRunner.GetOneByName(name);
+
+            // Assert
+            Assert.Throws<ArgumentException>(action);
+        }
+
+        [Fact]
+        public void TestGetOneByNameIfValidButNotExistThenReturnNull()
+        {
+            // Arrange
+
+
+            // Act
+            Game result = stubGameRunner.GetOneByName("thereisbasicalllynowaythatthisgamenamealreadyexists");
+
+            // Assert
+            Assert.Null(result);
+        }
+
+        [Theory]
+        [InlineData("Bob")]
+        [InlineData("bob")]
+        [InlineData("bob ")]
+        [InlineData(" boB ")]
+        public void TestGetOneByNameIfValidThenReturnPlayer(string name)
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Fact]
+        public void TestRemoveWorksIfExists()
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Fact]
+        public void TestRemoveThrowsExceptionIfGivenNull()
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Fact]
+        public void TestRemoveFailsSilentlyIfGivenNonExistent()
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Fact]
+        public void TestUpdateWorksIfValid()
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Theory]
+        [InlineData("Filibert", "filibert")]
+        [InlineData("Filibert", " fiLibert")]
+        [InlineData("Filibert", "FIlibert ")]
+        [InlineData(" Filibert", " filiBErt ")]
+        public void TestUpdateDiscreetlyUpdatesCaseAndIgnoresExtraSpaceIfOtherwiseSame(string n1, string n2)
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Theory]
+        [InlineData("")]
+        [InlineData(" ")]
+        [InlineData(null)]
+        public void TestUpdateDoesNotGoWithValidBeforeAndInvalidAfter(string badName)
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Fact]
+        public void TestUpdateDoesNotGoWithValidBeforeAndNullAfter()
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Fact]
+        public void TestUpdateDoesNotGoWithValidAfterAndNullBefore()
+        {
+            // Arrange
+
+
+            // Act
+
+
+            // Assert
+
+        }
+
+        [Theory]
+        [InlineData("")]
+        [InlineData(" ")]
+        [InlineData(null)]
+        public void TestUpdateDoesNotGoWithValidAfterAndInvalidBefore(string name)
+        {
+            // Arrange
+            
+
+            // Act
+            
+
+            // Assert
+            
+        }
+    }
+}

Sources/Tests/Model_UTs/PlayerManagerTest.cs

@@ -59,8 +59,8 @@ namespace Tests.Model_UTs
 
             // Assert
             Assert.Null(expected);
-            Assert.DoesNotContain(expected, playerManager.GetAll());
             Assert.Throws<ArgumentNullException>(action);
+            Assert.DoesNotContain(expected, playerManager.GetAll());
         }
 
         [Fact]

Sources/Tests/Tests.csproj

@@ -20,6 +20,7 @@
   </ItemGroup>
 
   <ItemGroup>
+    <ProjectReference Include="..\Data\Data.csproj" />
     <ProjectReference Include="..\Model\Model.csproj" />
   </ItemGroup>