Про тесты
Closed this issue · 1 comments
K1ola commented
-
Мне не понравился тест TestServer_Start по след причинам
- начинка теста ничего не проверяет сама по себе, ее можно спокойно заменить на if в самом приложении. Данный тест по сути проверяет, не забыли ли вы положить нужный конфиг в сервис. Это нужно проверять на старте приложения, а не на прогоне тестов
- при тестировании методов, которые работают с ос, нужно работать более осторожно, там своя специфика в отличие от тестирования ваших ручек или бизнес логики. Работу с ОС в тестах вы не сможете обеспечить на 100: надежно в рамках веб-приложения, поэтому лучше тестами максимально покрывать бизнес логику и интеграционное взаимодействие с другими сервисами, а работу с ос проверять в самом приложении
-
В табличных тестах использовать уникальные говорящие имена. Это поможет, когда у вас будет 100+ тестов и какой-нибудь один рухнет.
zeronethunter commented
We are closing the issue because we have fixed the remarks 😄