Холстинин Егор

cs/HomeExercises/NumberValidatorTests.cs

@@ -8,25 +8,89 @@ namespace HomeExercises
 	public class NumberValidatorTests
 	{
 		[Test]
-		public void Test()
+		public void CreatesWithNoExceptions()
 		{
-			Assert.Throws<ArgumentException>(() => new NumberValidator(-1, 2, true));
-			Assert.DoesNotThrow(() => new NumberValidator(1, 0, true));
-			Assert.Throws<ArgumentException>(() => new NumberValidator(-1, 2, false));
-			Assert.DoesNotThrow(() => new NumberValidator(1, 0, true));
-
-			Assert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0.0"));
-			Assert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0"));
-			Assert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0.0"));
-			Assert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("00.00"));
-			Assert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("-0.00"));
-			Assert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0.0"));
-			Assert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("+0.00"));
-			Assert.IsTrue(new NumberValidator(4, 2, true).IsValidNumber("+1.23"));
-			Assert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("+1.23"));
-			Assert.IsFalse(new NumberValidator(17, 2, true).IsValidNumber("0.000"));
-			Assert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("-1.23"));
-			Assert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("a.sd"));
+			var creationOfValidator = new Action(() => { new NumberValidator(2, 1, true); });
+			creationOfValidator.Should().NotThrow();
+		}
+
+		[TestCase(-1, 2, false, TestName = "Precision < 0")]
+		[TestCase(3, -1, false, TestName = "Scale < 0")]
+		[TestCase(3, 4, false, TestName = "Precision < Scale")]
+		public void ShouldThrowException_WhenInitWithIncorrectData(int precision, int scale, bool onlyPositive)
+		{
+			var creationOfValidatorOnlyPositive = new Action(() => { new NumberValidator(precision, scale, onlyPositive); });
+			creationOfValidatorOnlyPositive.Should().Throw<ArgumentException>();
+		}
+
+		[Test]
+		public void ShouldNotValidate_Null()
+		{
+			string number = null!;
+			var validator = new NumberValidator(5, 2);
+			validator.IsValidNumber(number).Should().BeFalse("null should be false");
+		}
+
+		[Test]
+		public void ShouldNotValidate_Empty()
+		{
+			const string number = "";
+			var validator = new NumberValidator(5, 2);
+			validator.IsValidNumber(number).Should().BeFalse("empty should be false");
+		}
+
+		[Test]
+		public void ShouldNotValidate_NonNumeric()
+		{
+			const string notNumber = "a.bc";
+			var validator = new NumberValidator(5, 2);
+			validator.IsValidNumber(notNumber).Should().BeFalse($"{notNumber} is not a number");
+		}
+
+		[TestCase("12.34", true)]
+		[TestCase("12", true)]
+		[TestCase("1.000", false)]
+		[TestCase("123.000", false)]
+		public void ShouldValidate_NumbersWithoutSign(string inputValue, bool expected)
+		{
+			var validator = new NumberValidator(5, 2, true);
+			validator.IsValidNumber(inputValue).Should().Be(expected, $"number {inputValue} is {(expected ? "correct" : "incorrect")}");
+		}
+
+		[TestCase("+1.23", true)]
+		[TestCase("+1", true)]
+		[TestCase("+0.5", true)]
+		[TestCase("+0.567", false)]
+		[TestCase("+123.45", false)]
+		public void ShouldValidate_NumbersWithPositiveSign(string inputValue, bool expected)
+		{
+			var validator = new NumberValidator(5, 2, true);
+			validator.IsValidNumber(inputValue).Should().Be(expected, $"number {inputValue} is {(expected ? "correct" : "incorrect")}");
+		}
+
+		[TestCase("-1.23", true)]
+		[TestCase("-1", true)]
+		[TestCase("-0.5", true)]
+		[TestCase("-0.567", false)]
+		[TestCase("-123.45", false)]
+		public void ShouldValidate_NumbersWithNegativeSign(string inputValue, bool expected)
+		{
+			var positiveValidator = new NumberValidator(5, 2, true);
+			positiveValidator.IsValidNumber(inputValue).Should().Be(false, "only positive validator should not validate negatives");
+
+			var validator = new NumberValidator(5, 2);
+			validator.IsValidNumber(inputValue).Should().Be(expected, $"number {inputValue} is {(expected ? "correct" : "incorrect")}");
+		}
+
+		[TestCase("1,23", true)]
+		[TestCase("+1,23", true)]
+		[TestCase("-1,23", true)]
+		[TestCase("-123,45", false)]
+		[TestCase("0,456", false)]
+		public void ShouldValidate_DifferentDelimiter(string inputValue, bool expected)
+		{
+			var validator = new NumberValidator(5, 2);
+			validator.IsValidNumber(inputValue).Should().Be(expected, $"{expected} should validate comma");
 		}
 	}
 

cs/HomeExercises/ObjectComparison.cs

@@ -1,4 +1,5 @@
 using FluentAssertions;
+using FluentAssertions.Equivalency;
 using NUnit.Framework;
 
 namespace HomeExercises
@@ -7,24 +8,16 @@ public class ObjectComparison
 	{
 		[Test]
 		[Description("Проверка текущего царя")]
-		[Category("ToRefactor")]
 		public void CheckCurrentTsar()
 		{
 			var actualTsar = TsarRegistry.GetCurrentTsar();
 
 			var expectedTsar = new Person("Ivan IV The Terrible", 54, 170, 70,
 				new Person("Vasili III of Russia", 28, 170, 60, null));
 
-			// Перепишите код на использование Fluent Assertions.
-			Assert.AreEqual(actualTsar.Name, expectedTsar.Name);
-			Assert.AreEqual(actualTsar.Age, expectedTsar.Age);
-			Assert.AreEqual(actualTsar.Height, expectedTsar.Height);
-			Assert.AreEqual(actualTsar.Weight, expectedTsar.Weight);
-
-			Assert.AreEqual(expectedTsar.Parent!.Name, actualTsar.Parent!.Name);
-			Assert.AreEqual(expectedTsar.Parent.Age, actualTsar.Parent.Age);
-			Assert.AreEqual(expectedTsar.Parent.Height, actualTsar.Parent.Height);
-			Assert.AreEqual(expectedTsar.Parent.Parent, actualTsar.Parent.Parent);
+			actualTsar.Should().BeEquivalentTo(expectedTsar, options => options
+				.IncludingFields()
+				.Excluding(info => info.SelectedMemberPath.EndsWith("Id")));
 		}
 
 		[Test]
@@ -36,6 +29,15 @@ public void CheckCurrentTsar_WithCustomEquality()
 				new Person("Vasili III of Russia", 28, 170, 60, null));
 
 			// Какие недостатки у такого подхода? 
+			/*
+			 * Недостатками такого подхода являются:
+			 * 1. Если тест падает то единственное, что можно из этого понять, что цари не одинаковые
+			 * Никакой информации об их различиях он не выводит
+			 * 2. Из названия теста тоже никак не получится понять что именно тестируется
+			 * 3. При любом расширении класса Person придется дописывать/переписывать компаратор
+			 * 4. Еще ужаснее будет если нововведение в Person будет необязательным и нам придется
+			 * В компараторе поддерживать это опциональное сравнение, а учитывая еще и первый пункт...
+			 */
 			Assert.True(AreEqual(actualTsar, expectedTsar));
 		}