Рыбин Леонид

.gitignore

@@ -1,3 +1,5 @@
 bin/
 obj/
 .vs/
+Basic.sln
+Basic.sln.DotSettings.user

Testing/Basic/Homework/1. ObjectComparison/ObjectComparison.cs

@@ -1,4 +1,5 @@
-using NUnit.Framework;
+using FluentAssertions;
+using NUnit.Framework;
 using NUnit.Framework.Legacy;
 
 namespace HomeExercise.Tasks.ObjectComparison;
@@ -14,16 +15,8 @@ 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.
-        ClassicAssert.AreEqual(actualTsar.Name, expectedTsar.Name);
-        ClassicAssert.AreEqual(actualTsar.Age, expectedTsar.Age);
-        ClassicAssert.AreEqual(actualTsar.Height, expectedTsar.Height);
-        ClassicAssert.AreEqual(actualTsar.Weight, expectedTsar.Weight);
-
-        ClassicAssert.AreEqual(expectedTsar.Parent!.Name, actualTsar.Parent!.Name);
-        ClassicAssert.AreEqual(expectedTsar.Parent.Age, actualTsar.Parent.Age);
-        ClassicAssert.AreEqual(expectedTsar.Parent.Height, actualTsar.Parent.Height);
-        ClassicAssert.AreEqual(expectedTsar.Parent.Parent, actualTsar.Parent.Parent);
+        actualTsar.Should().BeEquivalentTo(expectedTsar, options =>
+            options.Excluding(o => o.Name == "Id" && o.DeclaringType == typeof(Person)));
     }
 
     [Test]
@@ -35,6 +28,18 @@ public void CheckCurrentTsar_WithCustomEquality()
             new Person("Vasili III of Russia", 28, 170, 60, null));
 
         // Какие недостатки у такого подхода? 
+        /*
+         1) Метод AreEqual проверяет только фиксированые параметры в Person, поэтому
+            если класс Person будет измненён, то тесты предётся переписывать,
+            добавляя новые или удаляя старые поля.
+         2) Метод AreEqual возвращает True или False, поэтому даже если только одно поле
+            не пройдёт проверку, всё что будет понятно, что проверка не прошла, а
+            что именно было не так, узнать не получится.
+         3) Даже если переписывание тестов не является проблемой (см. 1)), то при добавлении
+            большего количества полей, особенно если типом новых полей будут другие классы
+            со своими полями, метод AreEqual рискует разрастись до монструозных масштабов,
+            из-за чего его станет очень сложно читать.
+        */
         ClassicAssert.True(AreEqual(actualTsar, expectedTsar));
     }
 

Testing/Basic/Homework/2. NumberValidator/NumberValidatorTests.cs

@@ -1,31 +1,52 @@
-
+using FluentAssertions;
 using NUnit.Framework;
-using NUnit.Framework.Legacy;
 
 namespace HomeExercise.Tasks.NumberValidator;
 
 [TestFixture]
 public class NumberValidatorTests
 {
-    [Test]
-    public void Test()
+    [TestCase(-1, 2, true, TestName = "negative precision")]
+    [TestCase(0, 2, false, TestName = "precision == 0")]
+    [TestCase(1, -2, true, TestName = "negative scale")]
+    [TestCase(1, 2, false, TestName = "scale > precision")]
+    [TestCase(1, 1, true, TestName = "scale == precision")]
+    public void NumberValidation_ConstructorHasArgumentException(int precision, int scale, bool onlyPositive)
     {
-        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));
+        var action = () => new NumberValidator(precision, scale, onlyPositive);
+        action.Should().Throw<ArgumentException>();
+    }
+
+    [TestCase(1, 0, true, TestName = "default")]
+    public void NumberValidation_ConstructorDoesNotHaveArgumentException(int precision, int scale, bool onlyPositive)
+    {
+        var action = () => new NumberValidator(precision, scale, onlyPositive);
+        action.Should().NotThrow();
+    }
+
+    [TestCase(1, 0, true, "0",ExpectedResult = true)]
+    [TestCase(2, 1, true, "0.0", ExpectedResult = true)]
+    [TestCase(2, 1, true, "0.1", ExpectedResult = true)]
+    [TestCase(2, 1, true, "0,1", ExpectedResult = true)]
+    [TestCase(2, 0, true, "+1", ExpectedResult = true)]
+    [TestCase(3, 1, true, "+1.1", ExpectedResult = true)]
+    [TestCase(3, 1, true, "+1,1", ExpectedResult = true)]
+    [TestCase(2, 0, false, "-1", ExpectedResult = true)]
+    [TestCase(3, 1, false, "-1.1", ExpectedResult = true)]
+    [TestCase(3, 1, false, "-1,1", ExpectedResult = true)]
+    [TestCase(7, 2, true, "671,23", ExpectedResult = true)]
 
-        ClassicAssert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0.0"));
-        ClassicAssert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0"));
-        ClassicAssert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0.0"));
-        ClassicAssert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("00.00"));
-        ClassicAssert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("-0.00"));
-        ClassicAssert.IsTrue(new NumberValidator(17, 2, true).IsValidNumber("0.0"));
-        ClassicAssert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("+0.00"));
-        ClassicAssert.IsTrue(new NumberValidator(4, 2, true).IsValidNumber("+1.23"));
-        ClassicAssert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("+1.23"));
-        ClassicAssert.IsFalse(new NumberValidator(17, 2, true).IsValidNumber("0.000"));
-        ClassicAssert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("-1.23"));
-        ClassicAssert.IsFalse(new NumberValidator(3, 2, true).IsValidNumber("a.sd"));
+    [TestCase(2, 0, true, "", ExpectedResult = false)]
+    [TestCase(2, 1, true, ".0", ExpectedResult = false)]
+    [TestCase(2, 0, true, "0.", ExpectedResult = false)]
+    [TestCase(2, 0, true, "-1", ExpectedResult = false)]
+    [TestCase(10, 2, true, "abcde.f", ExpectedResult = false)]
+    [TestCase(10, 9, true, "127.0.0.1", ExpectedResult = false)]
+    [TestCase(3, 2, true, "4321.5", ExpectedResult = false)]
+    [TestCase(3, 0, true, "+145", ExpectedResult = false)]
+    [TestCase(3, 0, false, "-124", ExpectedResult = false)]
+    public bool IsValidNumber(int precision, int scale, bool onlyPositive, string value)
+    {
+        return new NumberValidator(precision, scale, onlyPositive).IsValidNumber(value);
     }
 }