Conversation
…NumberValidator classes, removed comments, comment from NumberValidator.IsValidNumber moved to method documentation
| @@ -0,0 +1,23 @@ | |||
| namespace HomeExercises | |||
| { | |||
| public class Person | |||
There was a problem hiding this comment.
хороший тон
обычно тесты выносятся в отдельный проект от кода который будет тестироваться,, но здесь можно хотя бы попробовать разнести по папкам тесты и код который тестируется (не забудь после этого чекнуть неймпсейсы)
обычно когда работаешь в большом проекте то навигируешься конечно по именам файлов с помощью хоткеев, но иногда бывает такое что имя файла который хочешь открыть забываешь, и тут на помощь приходит solution explorer, при хорошей папочной структуре проекта можно будет легко, интуитивно найти все что нужно не только тем кто забыл название того или иного класса, но и поможет новичкам в проекте быстрее разобраться, ибо будет наглядное разделение классов на какие-то категории
|
|
||
| public bool IsValidNumber(string value) | ||
|
|
||
| [Test, TestCaseSource(nameof(ScaleTestCases))] |
There was a problem hiding this comment.
codestyle
стоит размещать аттрибуты в одном стиле, как - выбери сам
| { | ||
| [Test] | ||
| public void Test() | ||
| [TestCase(-3, 2, true, TestName = "WhenPrecisionIsNegative")] |
There was a problem hiding this comment.
codestyle
почему бы не написать все в общем стиле через testcase либо через testcasesource
|
|
||
| namespace HomeExercises | ||
| { | ||
| public class NumberValidatorTests |
There was a problem hiding this comment.
я бы накидал еще различные тесты на пограничные значения
ну например разве мы не можем передать null или emty string?
когда scale и prescision равны
когда не указаны необязательные параметры
и.т.д.
2 participants
fixes refactors e.t.c.