Improve Update() test after bugfix

pull/52/head
Alexis Drai 2 years ago
parent d36afce173
commit a5483b3850

@ -200,15 +200,19 @@ namespace Tests.Model_UTs
Assert.Contains(newPlayer, playerManager.GetAll()); Assert.Contains(newPlayer, playerManager.GetAll());
} }
[Fact] [Theory]
public void TestUpdateDoesNothingIfSame() [InlineData("Filibert", "filibert")]
[InlineData("Filibert", " fiLibert")]
[InlineData("Filibert", "FIlibert ")]
[InlineData(" Filibert", " filiBErt ")]
public void TestUpdateDiscreetlyUpdatesCaseAndIgnoresExtraSpaceIfOtherwiseSame(string n1, string n2)
{ {
// Arrange // Arrange
string name = "Filibert"; string name = "Filibert";
PlayerManager playerManager = new(); PlayerManager playerManager = new();
Player oldPlayer = new(name); Player oldPlayer = new(n1);
playerManager.Add(oldPlayer); playerManager.Add(oldPlayer);
Player newPlayer = new(name); Player newPlayer = new(n2);
// Act // Act
playerManager.Update(oldPlayer, newPlayer); playerManager.Update(oldPlayer, newPlayer);
@ -216,6 +220,8 @@ namespace Tests.Model_UTs
// Assert // Assert
Assert.Contains(oldPlayer, playerManager.GetAll()); Assert.Contains(oldPlayer, playerManager.GetAll());
Assert.Contains(newPlayer, playerManager.GetAll()); Assert.Contains(newPlayer, playerManager.GetAll());
Assert.Equal(n2.Trim(), playerManager.GetAll().First().Name);
// uses Equals(), which is made to be case-insensitive
} }
} }
} }

Loading…
Cancel
Save