diff --git a/2_Uncle_Bob_Full/Soat.CleanCode.VideoStore.UncleBobFull.Tests/VideoStoreTests.cs b/2_Uncle_Bob_Full/Soat.CleanCode.VideoStore.UncleBobFull.Tests/VideoStoreTests.cs index 64ab89a..8c02870 100644 --- a/2_Uncle_Bob_Full/Soat.CleanCode.VideoStore.UncleBobFull.Tests/VideoStoreTests.cs +++ b/2_Uncle_Bob_Full/Soat.CleanCode.VideoStore.UncleBobFull.Tests/VideoStoreTests.cs @@ -12,57 +12,58 @@ public VideoStoreTests() } [Fact] - public void TestSingleNewReleaseStatement() + public void TestSingleNewReleaseStatementTotals() { _statement.AddRental(new Rental(new Movie("The cell", Movie.NEW_RELEASE), 3)); _statement.Generate(); Assert.Equal(9.0m, _statement.TotalAmount); - Assert.Equal(2, _statement.FrequentRenterPoints); + Assert.Equal(2, _statement.FrequentRenterPoints); } [Fact] - public void TestDualNewReleaseStatement() + public void TestDualNewReleaseStatementTotals() { _statement.AddRental(new Rental(new Movie("The cell", Movie.NEW_RELEASE), 3)); _statement.AddRental(new Rental(new Movie("The Tigger Movie", Movie.NEW_RELEASE), 3)); - - Assert.Equal( - "Rental Record for Fred\n" + - "\tThe cell\t9.0\n" + - "\tThe Tigger Movie\t9.0\n" + - "You owed 18.0\n" + - "You earned 4 frequent renter points \n", - _statement.Generate()); + _statement.Generate(); + Assert.Equal(18.0m, _statement.TotalAmount); + Assert.Equal(4, _statement.FrequentRenterPoints); } [Fact] - public void TestSingleChildrensStatement() + public void TestSingleChildrensStatementTotals() { _statement.AddRental(new Rental(new Movie("The Tigger Movie", Movie.CHILDREN), 3)); + _statement.Generate(); + Assert.Equal(1.5m, _statement.TotalAmount); + Assert.Equal(1, _statement.FrequentRenterPoints); + } - Assert.Equal( - "Rental Record for Fred\n" + - "\tThe Tigger Movie\t1.5\n" + - "You owed 1.5\n" + - "You earned 1 frequent renter points \n", - _statement.Generate()); + [Fact] + public void TestMultipleRegularStatementTotals() + { + _statement.AddRental(new Rental(new Movie("Plan 9 from Outer Space", Movie.REGULAR), 1)); + _statement.AddRental(new Rental(new Movie("8 1/2", Movie.REGULAR), 2)); + _statement.AddRental(new Rental(new Movie("Eraserhead", Movie.REGULAR), 3)); + _statement.Generate(); + Assert.Equal(7.5m, _statement.TotalAmount); + Assert.Equal(3, _statement.FrequentRenterPoints); } [Fact] - public void TestMultipleRegularStatement() + public void TestMultipleRegularStatementFormat() { _statement.AddRental(new Rental(new Movie("Plan 9 from Outer Space", Movie.REGULAR), 1)); _statement.AddRental(new Rental(new Movie("8 1/2", Movie.REGULAR), 2)); _statement.AddRental(new Rental(new Movie("Eraserhead", Movie.REGULAR), 3)); - Assert.Equal( - "Rental Record for Fred\n" + - "\tPlan 9 from Outer Space\t2.0\n" + - "\t8 1/2\t2.0\n" + - "\tEraserhead\t3.5\n" + - "You owed 7.5\n" + - "You earned 3 frequent renter points \n", - _statement.Generate()); + Assert.Equal(@"Rental Record for Fred + Plan 9 from Outer Space 2.0 + 8 1/2 2.0 + Eraserhead 3.5 +You owed 7.5 +You earned 3 frequent renter points +", _statement.Generate()); } } } diff --git a/2_Uncle_Bob_Full/Soat.CleanCode.VideoStore.UncleBobFull/Statement.cs b/2_Uncle_Bob_Full/Soat.CleanCode.VideoStore.UncleBobFull/Statement.cs index 9446c9f..c157747 100644 --- a/2_Uncle_Bob_Full/Soat.CleanCode.VideoStore.UncleBobFull/Statement.cs +++ b/2_Uncle_Bob_Full/Soat.CleanCode.VideoStore.UncleBobFull/Statement.cs @@ -63,7 +63,7 @@ public string Generate() } result += "You owed " + TotalAmount.ToString("0.0", CultureInfo.InvariantCulture) + "\n"; - result += "You earned " + FrequentRenterPoints.ToString() + " frequent renter points \n"; + result += "You earned " + FrequentRenterPoints.ToString() + " frequent renter points\n"; return result; }