Коробицын Александр, "Тестирование"#61
Коробицын Александр, "Тестирование"#61AlexKorobitsyn wants to merge 3 commits intokontur-courses:masterfrom
Conversation
| public void CheckCurrentTsar() | ||
| { | ||
| var actualTsar = TsarRegistry.GetCurrentTsar(); | ||
| private Person actualTsar = TsarRegistry.GetCurrentTsar(); |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
There was a problem hiding this comment.
Нет, это же в тестах поля
There was a problem hiding this comment.
Тогда лучше в коде это явно обозначить
| [Test] | ||
| public void Constructor_InvalidScale_Fail() | ||
| { | ||
| Assert.Throws<ArgumentException>(() => new NumberValidator(5, -1)); |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
There was a problem hiding this comment.
Еще подскажи, пожалуйста, почему решил писать без использования fluent'ов?
There was a problem hiding this comment.
fluent'ов
Маленькая структура, маленькие числовые проверки, Fluent как я понял, для чего потяжелее используется. Как в первом большая вложенность... Ну и в описании ко второй задаче, не говорилось про использование FluentAssert
There was a problem hiding this comment.
Тут и в тестах ниже строки проверок дублируются. Как можно этого избежать?
Исправлю сейчас
| Assert.Throws<ArgumentException>(() => new NumberValidator(-1, 2)); | ||
| Assert.Throws<ArgumentException>(() => new NumberValidator(0, 1)); | ||
| } | ||
| [Test] |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
There was a problem hiding this comment.
В message или комментарием?
There was a problem hiding this comment.
Если хочешь оставить [Test], то можно посмотреть в ObjectComparison, там описание есть
Но я бы порекомендовал посмотреть альтернативы, которые закроют и проблему дублирования, и эту
|
Вопрос общий, напрямую не относящийся к коду |
всегда так делаю, одна домашка - один комит ( так в шаде научили) |
В рамках обучения в Контуре, вроде, должны были провести занятия по гиту и рассказать, как нужно оформлять ДЗ |
Скорее всего рассказывали, сегодня уточню у коллег. |
|
Всё исправил |
2 participants
No description provided.