Зубакин Ренат

cs/HomeExercises/NumberValidatorTests.cs

@@ -12,7 +12,7 @@ public void Test()
 		{
 			Assert.Throws<ArgumentException>(() => new NumberValidator(-1, 2, true));
 			Assert.DoesNotThrow(() => new NumberValidator(1, 0, true));
-			Assert.Throws<ArgumentException>(() => new NumberValidator(-1, 2, false));
+			Assert.Throws<ArgumentException>(() => new NumberValidator(-1, 2));
 			Assert.DoesNotThrow(() => new NumberValidator(1, 0, true));
 
 			Assert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0.0"));
@@ -28,6 +28,35 @@ public void Test()
 			Assert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("-1.23"));
 			Assert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("a.sd"));
 		}
+
+		[TestCase(-1, 2, true, TestName = "When_Precision_Is_Not_Positive")]
+		[TestCase(1, -2, true, TestName = "When_Scale_Is_Negative(")]
+		[TestCase(12, 14, true, TestName = "When_Scale_More_Than_Precision")]
+		public void NumberValidator_Should_Throw_ArgumentException(int precision, int scale, bool onlyPositive)
+		{
+			Assert.Throws<ArgumentException>(() => new NumberValidator(precision, scale, onlyPositive));
+		}
+
+		[TestCase(3, 1, false, null, TestName = "When_Value_Is_Null")]
+		[TestCase(3, 1, false, "", TestName = "When_Value_Is_Empty")]
+		[TestCase(3, 1, true, "ab.c", TestName = "When_Value_Is_Not_Match_Regex")]
+		[TestCase(3, 1, true, "000.0", TestName = "When_Value_Length_More_Than_Precision")]
+		[TestCase(3, 1, true, "-0.0", TestName = "When_OnlyPositive_Is_True_And_Value_Is_Negative")]
+		public void IsValidNumber_Should_Return_False(int precision, int scale, bool onlyPositive, string value)
+		{
+			var numberValidator = new NumberValidator(precision, scale, onlyPositive);
+			numberValidator.IsValidNumber(value).Should().BeFalse();
+		}
+
+		[TestCase(4, 2, false, "-0.0", TestName = "When_OnlyPositive_Is_False_And_Value_Is_Negative")]
+		[TestCase(4, 2, true, "+0.0", TestName = "When_Value_Starts_With_Plus")]
+		[TestCase(2, 2, true, "0.0", TestName = "When_Value_Is_Positive")]
+		public void IsValidNumeber_Should_Return_True(int precision, int scale, bool onlyPositive, string value)
+		{
+			var numberValidator = new NumberValidator(precision, scale, onlyPositive);
+			numberValidator.IsValidNumber(value).Should().BeTrue();
+		}
+
 	}
 
 	public class NumberValidator

cs/HomeExercises/ObjectComparison.cs

@@ -1,4 +1,6 @@
-using FluentAssertions;
+using System;
+using FluentAssertions;
+using FluentAssertions.Equivalency;
 using NUnit.Framework;
 
 namespace HomeExercises
@@ -15,16 +17,16 @@ public void CheckCurrentTsar()
 			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);
+			/*
+			 *  Данный тест рекурсивно пройдется по всем полям объекта и сравнит их по значению рекурсивно
+			 *  По умолчанию рекурсия проходит на глубину 10, для того чтобы снять ограничение
+			 *  используется опция AllowingInfiniteRecursion, но она может стать бесконечной, если цепь проверки замкнется.
+			 *  Мой способ лучше тем, что в случае добавления новых полей в объект Person, будут проверяться все поля без изменения теста.
+			 *  Изменения нужно будет вносить только в случае если мы хотим убрать какие-то поля из проверки.
+			 */
+			actualTsar.Should().BeEquivalentTo(expectedTsar, options =>
+				options.Excluding((IMemberInfo o) => o.SelectedMemberInfo.Name == "Id" &&
+				                                     o.SelectedMemberInfo.DeclaringType == typeof(Person)));
 		}
 
 		[Test]
@@ -34,8 +36,13 @@ public void CheckCurrentTsar_WithCustomEquality()
 			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));
-
-			// Какие недостатки у такого подхода? 
+			/*
+			 * Какие недостатки у такого подхода?
+			 * 1. Придется менять метод AreEqual, если в объект добавятся еще поля.
+			 * 2. Рекурсия может стать бесконечной, если в каком-то Person Parent будет равен Person, который уже был в цепи проверки
+			 * 3. Имя теста никак не сигнализирует о том какой особый случай оно проверяет
+			 * 4. Тест никак не показывает в чем проблема, тест просто становится красным
+			 */
 			Assert.True(AreEqual(actualTsar, expectedTsar));
 		}