diff --git a/Sources/Tests/Model_UTs/GameRunnerTest.cs b/Sources/Tests/Model_UTs/GameRunnerTest.cs index 9bf6f20..6bf4626 100644 --- a/Sources/Tests/Model_UTs/GameRunnerTest.cs +++ b/Sources/Tests/Model_UTs/GameRunnerTest.cs @@ -73,7 +73,7 @@ namespace Tests.Model_UTs } [Fact] - public void TestAddwhenNullThenThrowsException() + public void TestAddWhenNullThenThrowsException() { // Arrange @@ -90,7 +90,7 @@ namespace Tests.Model_UTs [InlineData("")] [InlineData(null)] [InlineData(" ")] - public void TestGetOneByNameIfInvalidThrowsException(string name) + public void TestGetOneByNameWhenInvalidThenThrowsException(string name) { // Arrange @@ -103,7 +103,7 @@ namespace Tests.Model_UTs } [Fact] - public void TestGetOneByNameIfValidButNotExistThenReturnNull() + public void TestGetOneByNameWhenValidButNotExistThenReturnNull() { // Arrange @@ -116,7 +116,7 @@ namespace Tests.Model_UTs } [Fact] - public void TestGetOneByNameIfValidThenReturnGame() + public void TestGetOneByNameWhenValidThenReturnGame() { // Arrange GameRunner gameRunner = new(new PlayerManager(), new DieManager()); @@ -131,42 +131,45 @@ namespace Tests.Model_UTs } [Fact] - public void TestRemoveWorksIfExists() + public void TestWhenRemoveExistsThenSucceeds() { // Arrange - + Game game = new("blargh", new PlayerManager(), stubGameRunner.GetAll().First().Dice); + stubGameRunner.Add(game); // Act - + stubGameRunner.Remove(game); // Assert - + Assert.DoesNotContain(game, stubGameRunner.GetAll()); } [Fact] - public void TestRemoveThrowsExceptionIfGivenNull() + public void TestRemoveWhenGivenNullThenThrowsException() { // Arrange // Act - + void action() => stubGameRunner.Remove(null); // Assert - + Assert.Throws(action); } [Fact] - public void TestRemoveFailsSilentlyIfGivenNonExistent() + public void TestRemoveWhenGiveenNonExistentThenFailsSilently() { // Arrange - + Game notGame = new("blargh", new PlayerManager(), stubGameRunner.GetAll().First().Dice); + IEnumerable expected = stubGameRunner.GetAll(); // Act - + stubGameRunner.Remove(notGame); + IEnumerable actual = stubGameRunner.GetAll(); // Assert - + Assert.Equal(actual, expected); } [Fact] @@ -182,23 +185,6 @@ namespace Tests.Model_UTs } - [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(" ")]