Conversation
MrTimeChip
left a comment
MrTimeChip
left a comment
There was a problem hiding this comment.
В целом, неплохо, но какие-то серьезные проблемы с форматированием - где-то много пробелов, где-то их нет, где-то странный отступ, перенос строки. Это все нужно поправить, можно даже авторефакторингом.
И ещё, на будущее - надо писать в телеграмм наставнику, как только получил задачу. Я даже не в курсе был, что ты сделал что-то)
cs/HomeExercises/ObjectComparison.cs
Outdated
| Assert.AreEqual(expectedTsar.Parent.Parent, actualTsar.Parent.Parent); | ||
| } | ||
| // Cравниваем все свойства, кроме исключенных | ||
| actualTsar.Should().BeEquivalentTo(expectedTsar,options => options.Excluding(x=>x.Id).Excluding((x=>x.Parent!.Id))); |
There was a problem hiding this comment.
Кажется, здесь есть лишняя пара скобочек.
|
|
||
| [TestCase(17, 2, "0.0", TestName = "Return True on correct values")] | ||
| public void NumberValidator_CorrectValue_ShouldReturnTrue(int precision, int scale,string number) | ||
| { | ||
| new NumberValidator(precision, scale,true).IsValidNumber(number).Should().Be(true); | ||
| } |
There was a problem hiding this comment.
Мало кейсов на позитивные сценарии. Мы никогда не знаем, какая часть логики поменяется, поэтому - фиксация позитивных сценариев важна точно так же, как и негативных.
Плюс один из параметров вообще в тестах нигде не учитывается, с ним бы тоже написать тестов.
|
Все хорошо, 2 балла (это максимум). |
2 participants
@DonielPankek