Increase test coverage
maypink opened this issue · comments
The current percent of coverage is ~74%, which is relatively small and allows to make more thoughtless and untested changes. Even though some functionality can be covered with integration tests, it is not enough, since unit tests allow to easily track all inconsistencies. Covergae with units must be increased to make GOLEM more stable. So the current files with the smallest percent of coverage can be seen here. These are mainly:
- structural analysis
- graph builder, some visualizations
- bandits
я думаю, что структурный анализ, бандитов и мажорную часть тестов я возьму на себя, но надо будет просто при дальнейшем изменении функциональности побольше обращать внимания на покрытость тестами. при желании -- писать по ходу дела дополнительные.
@YamLyubov @kasyanovse @MorrisNein
Я могу взять ответственность за тесты модулей в golem/core/optimisers/genetic
.
На днях как раз размышлял, как покрыть тестами параметры визуализаций.
Пока что представляю себе такой подход: сгенерить заранее набор референсных визуализаций и сравнивать выходные на тестах с референсными. Пожалуй, стоит мне этим заняться.
Это, однако, не подойдёт для некоторых стохастических визуализаций. Для таких в автоматическом режиме можно только проверить наличие выхода в нужном формате, что уже делается.