Косторной Дмитрий

Testing/Basic/Basic.csproj

@@ -1,19 +1,27 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
+    <TargetFramework>net7.0</TargetFramework>
     <ImplicitUsings>enable</ImplicitUsings>
     <Nullable>enable</Nullable>
   </PropertyGroup>
 
+  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|AnyCPU'">
+    <LangVersion>preview</LangVersion>
+  </PropertyGroup>
+
+  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
+    <LangVersion>preview</LangVersion>
+  </PropertyGroup>
+
   <ItemGroup>
     <PackageReference Include="coverlet.collector" Version="6.0.2">
       <PrivateAssets>all</PrivateAssets>
       <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
     </PackageReference>
-    <PackageReference Include="FluentAssertions" Version="6.12.0" />
+    <PackageReference Include="FluentAssertions" Version="6.12.1" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" Version="17.10.0" />
-    <PackageReference Include="NUnit" Version="4.1.0" />
+    <PackageReference Include="NUnit" Version="4.2.2" />
     <PackageReference Include="NUnit3TestAdapter" Version="4.5.0" />
     <PackageReference Include="xunit" Version="2.8.1" />
     <PackageReference Include="xunit.runner.visualstudio" Version="2.8.1">

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,9 @@ 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, properties => properties
+            .Excluding(person => person.Id)
+            .Excluding(person => person.Parent.Id));
     }
 
     [Test]
@@ -34,7 +28,15 @@ public void CheckCurrentTsar_WithCustomEquality()
         var expectedTsar = new Person("Ivan IV The Terrible", 54, 170, 70,
             new Person("Vasili III of Russia", 28, 170, 60, null));
 
-        // Какие недостатки у такого подхода? 
+        /* Данное решение имеет следующие недостатки:
+            При добавлении новых свойств в класс Person придётся вручную обновлять метод AreEqual и
+            включать новое свойство в сравнение, что сделает код очень громоздким и замедлит разработку.
+            В целом, используя собственный метод AreEqual, мы уменьшаем читаемость (особенно если подобных тестов и методов станет много,
+            а разбираться в этом будет сторонний разработчик), а также
+            отказываемся от более информативной обработки исключений FluentAssertions (можем увидеть, какое конкретное поле не совпало, 
+            а не просто получим возврат false) 
+            Таким образом моё решение является более устойчивым к изменениям, читабельным и поддерживаемым по сравнению с данным.
+        */
         ClassicAssert.True(AreEqual(actualTsar, expectedTsar));
     }
 

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

@@ -1,31 +1,99 @@
-
+using FluentAssertions;
 using NUnit.Framework;
 using NUnit.Framework.Legacy;
+using System.ComponentModel.DataAnnotations;
 
 namespace HomeExercise.Tasks.NumberValidator;
 
 [TestFixture]
 public class NumberValidatorTests
 {
     [Test]
-    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.DoesNotThrow(() => new NumberValidator(1, 0, 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"));
+    public void WhenPrecisionIsNegative_ShouldReturnArgumentException()
+    {
+        Action action = () => new NumberValidator(-1, 2, true);
+        action.Should().Throw<ArgumentException>().WithMessage("precision must be a positive number");
+    }
+
+    [Test]
+    public void WhenScaleIsNegative_ShouldThrowArgumentException()
+    {
+        Action act = () => new NumberValidator(1, -1, false);
+        act.Should().Throw<ArgumentException>().WithMessage("precision must be a non-negative number less or equal than precision");
+    }
+
+    [Test]
+    public void WhenScaleIsMoreThanPrecision_ShouldThrowArgumentException()
+    {
+        Action act = () => new NumberValidator(2, 3, false);
+        act.Should().Throw<ArgumentException>().WithMessage("precision must be a non-negative number less or equal than precision");
+    }
+
+    [Test]
+    public void WhenValid_ShouldNotThrow()
+    {
+        Action act = () => new NumberValidator(1, 0, true);
+        act.Should().NotThrow();
+    }
+
+    [Test]
+    public void WhenInputIsEmpty_ShouldReturnFalse()
+    {
+        NumberValidator validator = new NumberValidator(5, 2, true);
+        ValidateNumber(validator, "", false);
+    }
+
+    [Test]
+    public void WhenInputIsNull_ShouldReturnFalse()
+    {
+        NumberValidator validator = new NumberValidator(5, 2, true);
+        ValidateNumber(validator, null, false);
+    }
+
+
+    [Test]
+    [TestCase("12345", true)]
+    [TestCase("123.45", true)]
+    [TestCase("-123.45", false)]
+    [TestCase("001.23", true)]
+    [TestCase("0", true)]
+    [TestCase("0.0", true)]
+    [TestCase("+123.45", false)]
+    [TestCase("123456", false)]
+    [TestCase("123.456", false)]
+    [TestCase("0000.00", false)]
+    [TestCase("0.000", false)]
+    [TestCase("ab.c", false)]
+    [TestCase("123.45.67", false)]
+    [TestCase("123,67", true)]
+    [TestCase("123 45", false)]
+    [TestCase("++1", false)]
+    [TestCase("1.", false)]
+    public void WhenInputIsVarious_ShouldReturnExpected_PositiveValidator(string input, bool expected)
+    {
+        NumberValidator validator = new NumberValidator(5, 2, true);
+        ValidateNumber(validator, input, expected);
+    }
+
+    [Test]
+    [TestCase("12345", true)]
+    [TestCase("123.45", true)]
+    [TestCase("123.45a", false)]
+    [TestCase("12.3.4", false)]
+    [TestCase("-0", true)]
+    [TestCase("-12.45", true)]
+    [TestCase("-123.45", false)]
+    [TestCase("--1", false)]
+    [TestCase("-1.", false)]
+    public void WhenInputIsVarious_ShouldReturnExpected_NegativeValidator(string input, bool expected)
+    {
+        NumberValidator validator = new NumberValidator(5, 2, false);
+        ValidateNumber(validator, input, expected);
+    }
+
+    private void ValidateNumber(NumberValidator validator, string input, bool expected)
+    {
+        bool result = validator.IsValidNumber(input);
+        result.Should().Be(expected, $"Input '{input}' did not return expected result '{expected}'.");
     }
 }