Зиновьева Милана @xsitin #24
Conversation
| actualTsar.Should().BeEquivalentTo(expectedTsar, options => options | ||
| .IncludingNestedObjects() | ||
| .Excluding(p => p.Id) | ||
| .Excluding(p => p.Parent.Id)); |
There was a problem hiding this comment.
А что если наследников станет сильно больше? Посмотри перегрузки метода Exclude
| @@ -36,6 +34,10 @@ public void CheckCurrentTsar_WithCustomEquality() | |||
|
|
|||
| // Какие недостатки у такого подхода? | |||
| ClassicAssert.True(AreEqual(actualTsar, expectedTsar)); | |||
|
|
|||
| //-При появлении новых полей придется переписывать тест | |||
| //Тесты для проверки корректности инициализации | ||
|
|
||
| NumberValidator numberForCheck = new NumberValidator(4, 2, true); | ||
| numberForCheck.IsValidNumber("").Should().BeFalse(); |
There was a problem hiding this comment.
Добавь больше кейсов на невалидные случаи. Так же добавь кейсы для не арабских цифр, например цифр телугу
| numberForCheck.IsValidNumber(null).Should().BeFalse(because: "Ввод пуст"); | ||
|
|
||
| numberForCheck.IsValidNumber("a.sd").Should().BeFalse(because: "Используются не арабские цифры"); | ||
| numberForCheck.IsValidNumber("సున్న.నాలుగు").Should().BeFalse(because: "Используются не арабские цифры"); |
There was a problem hiding this comment.
Это словва, а не числа, для телугу можно посмотреть табличку тут, числа то, что в скобках. Можно поискать ещё различные варианты
| @@ -16,8 +16,7 @@ public void CheckCurrentTsar() | |||
| new Person("Vasili III of Russia", 28, 170, 60, null)); | |||
| actualTsar.Should().BeEquivalentTo(expectedTsar, options => options | |||
| .IncludingNestedObjects() | |||
| .Excluding(p => p.Id) | |||
| .Excluding(p => p.Parent.Id)); | |||
| .Excluding(p => p.Path.Contains("Id"))); | |||
There was a problem hiding this comment.
При такой проверке, будут игнорироваться все поля в пути которых есть Id. Так мы можем пропустить что-то важно, если в классе появяться новые поля
| ClassicAssert.AreEqual(expectedTsar.Parent.Parent, actualTsar.Parent.Parent); | ||
| actualTsar.Should().BeEquivalentTo(expectedTsar, options => options | ||
| .IncludingNestedObjects() | ||
| .Excluding(p => p.Name.Equals(nameof(Person.Id)) && p.DeclaringType == typeof(Person))); |
There was a problem hiding this comment.
В лямбдах тоже лучше давать понятные имена относящиеся к обьекту. Возможно, ты назвала в сокращении от Person, но аргумент MemberInfo
No description provided.