Яценко Ирина

cs/HomeExercises/NumberValidatorTests.cs

@@ -1,34 +1,80 @@
 using System;
+using System.Collections.Generic;
+using System.Runtime.CompilerServices;
 using System.Text.RegularExpressions;
-using FluentAssertions;
 using NUnit.Framework;
 
 namespace HomeExercises
 {
 	public class NumberValidatorTests
 	{
+		private static IEnumerable<TestCaseData> ArgumentExceptionTestCases
+        {
+			get
+			{
+				yield return new TestCaseData(-1, 2, true).SetName("WhenPassNegativePrecision");
+				yield return new TestCaseData(1, -2, true).SetName("WhenPassNegativeScale");
+				yield return new TestCaseData(2, 2, true).SetName("WhenPrecisionIsEqualToTheScale");
+			}
+		}
+
+        [TestCaseSource(nameof(ArgumentExceptionTestCases))]
+        public void NumberValidatorCtor_WhenPassInvalidArguments_ShouldThrowArgumentException(int precision,
+	        int scale, bool onlyPositive)
+		{
+			TestDelegate testDelegate = () => new NumberValidator(precision, scale, onlyPositive);
+
+			Assert.Throws<ArgumentException>(testDelegate);
+		}
+
 		[Test]
-		public void Test()
+		public void NumberValidatorCtor_WhenPassValidArguments_ShouldNotThrows()
 		{
-			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"));
+			TestDelegate testDelegate = () => new NumberValidator(1, 0, true);
+
+			Assert.DoesNotThrow(testDelegate);
 		}
-	}
+
+		private static IEnumerable<TestCaseData> InvalidArgumentTestCases
+        {
+			get
+			{
+				yield return new TestCaseData("a.sd", false).SetName("WhenLettersInsteadOfNumber");
+				yield return new TestCaseData("2.!", false).SetName("WhenSymbolsInsteadOfNumber");
+				yield return new TestCaseData(null!, false).SetName("WhenPassNumberIsNull");
+				yield return new TestCaseData("", false).SetName("WhenPassNumberIsEmpty");
+				yield return new TestCaseData("-0.00", false).SetName("WhenIntPartWithNegativeSignMoreThanPrecision");
+				yield return new TestCaseData("+1.23", false).SetName("WhenIntPartWithPositiveSignMoreThanPrecision");
+				yield return new TestCaseData("0.000", false).SetName("WhenFractionalPartMoreThanScale");
+            }
+		}
+		private static IEnumerable<TestCaseData> ValidArgumentTestCases
+		{
+			get
+			{
+				yield return new TestCaseData("0", true).SetName("WhenFractionalPartIsMissing");
+				yield return new TestCaseData("0.0", true).SetName("WhenNumberIsValid");
+			}
+		}
+
+		private NumberValidator numberValidator;
+
+		[SetUp]
+		public void SetUp()
+		{
+			numberValidator = new NumberValidator(3, 2, true);
+		}
+
+		[TestOf(nameof(NumberValidator.IsValidNumber))]
+        [TestCaseSource(nameof(ValidArgumentTestCases))]
+		[TestCaseSource(nameof(InvalidArgumentTestCases))]
+        public void WhenPassInvalidArguments_ShouldReturnFalse(string number, bool expectedResult)
+		{
+			var actualResult = numberValidator.IsValidNumber(number);
+
+			Assert.AreEqual(expectedResult, actualResult);
+		}
+    }
 
 	public class NumberValidator
 	{

cs/HomeExercises/ObjectComparison.cs

@@ -3,81 +3,82 @@
 
 namespace HomeExercises
 {
-	public class ObjectComparison
-	{
-		[Test]
-		[Description("Проверка текущего царя")]
-		[Category("ToRefactor")]
-		public void CheckCurrentTsar()
-		{
-			var actualTsar = TsarRegistry.GetCurrentTsar();
+    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));
+		    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);
+		    actualTsar.Should()
+			    .BeEquivalentTo(expectedTsar, x => x
+				    .Excluding(x => x.Id)
+				    .Excluding(x => x.Parent!.Id));
+	    }
 
-			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);
-		}
+        [Test]
+        [Description("Альтернативное решение. Какие у него недостатки?")]
+        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));
 
-		[Test]
-		[Description("Альтернативное решение. Какие у него недостатки?")]
-		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));
+	        // Какие недостатки у такого подхода? 
+	        // Данный подход делает класс труднорасширяемым, ведь при добавлении новых полей в класс Person
+	        // нужно переписывать и метод сравнения для всех этих полей, так же новые поля cможет добавить
+	        // другой разработчик, который не знает о таком методе сравнения, что приведет к новым, неотловоенным ошибкам - 
+	        // новые поля не будут сравниваться.
+	        // В моем решении (CheckCurrentTsar), такой ошибки не возникнет и класс Person сможет без ошибок расширяться
+	        // при условии, что сравниваться будут все поля, кроме Id (так же и их Parent), так как код написан не перебиранием
+	        // всех полей для сравнения, а сравнением объекта в целом с исключением его Id.
+	        Assert.True(AreEqual(actualTsar, expectedTsar));
+        }
 
-			// Какие недостатки у такого подхода? 
-			Assert.True(AreEqual(actualTsar, expectedTsar));
-		}
+        private bool AreEqual(Person? actual, Person? expected)
+        {
+            if (actual == expected) return true;
+            if (actual == null || expected == null) return false;
+            return
+                actual.Name == expected.Name
+                && actual.Age == expected.Age
+                && actual.Height == expected.Height
+                && actual.Weight == expected.Weight
+                && AreEqual(actual.Parent, expected.Parent);
+        }
+    }
 
-		private bool AreEqual(Person? actual, Person? expected)
-		{
-			if (actual == expected) return true;
-			if (actual == null || expected == null) return false;
-			return
-				actual.Name == expected.Name
-				&& actual.Age == expected.Age
-				&& actual.Height == expected.Height
-				&& actual.Weight == expected.Weight
-				&& AreEqual(actual.Parent, expected.Parent);
-		}
-	}
+    public class TsarRegistry
+    {
+        public static Person GetCurrentTsar()
+        {
+            return new Person(
+                "Ivan IV The Terrible", 54, 170, 70,
+                new Person("Vasili III of Russia", 28, 170, 60, null));
+        }
+    }
 
-	public class TsarRegistry
-	{
-		public static Person GetCurrentTsar()
-		{
-			return new Person(
-				"Ivan IV The Terrible", 54, 170, 70,
-				new Person("Vasili III of Russia", 28, 170, 60, null));
-		}
-	}
+    public class Person
+    {
+        public static int IdCounter = 0;
+        public int Age, Height, Weight;
+        public string Name;
+        public Person? Parent;
+        public int Id;
 
-	public class Person
-	{
-		public static int IdCounter = 0;
-		public int Age, Height, Weight;
-		public string Name;
-		public Person? Parent;
-		public int Id;
-
-		public Person(string name, int age, int height, int weight, Person? parent)
-		{
-			Id = IdCounter++;
-			Name = name;
-			Age = age;
-			Height = height;
-			Weight = weight;
-			Parent = parent;
-		}
-	}
+        public Person(string name, int age, int height, int weight, Person? parent)
+        {
+            Id = IdCounter++;
+            Name = name;
+            Age = age;
+            Height = height;
+            Weight = weight;
+            Parent = parent;
+        }
+    }
 }