Добавлены тесты производительности с помощью Reassure#1
Добавлены тесты производительности с помощью Reassure#1isokolovskii wants to merge 42 commits intocdek-it:mainfrom
Conversation
…водительности компонент
…пуск тестов и reassure
isokolovskii
changed the title
|
Проверка Performance Tests будет падать так как её суть в том что идёт запуск тестов в baseline ветке(main), а потом в текущей ветке и генерируется сравнение А пока reassure в main ветке нет он падает как раз на попытке на ней запуститься |
grevtsovna
left a comment
grevtsovna
left a comment
There was a problem hiding this comment.
Много коммитов с типом feat, может поменяем на test? Просто кажется, что не стоит это все в changelog записать, а для test я отключил добавление changelog
|
|
||
| on: | ||
| push: | ||
| branches: [main] |
There was a problem hiding this comment.
А нужно ли отдельно запускать на пуш в main? Я бы просто добавил все проверки в релизном воркфлоу
There was a problem hiding this comment.
Ну... Вообще если допустить что никогда никто и никак в main не запушит - то может и не стоит
В релизном как раз кажется лишнее - релиз на то и релиз, зачем там проверять линтер
| push: | ||
| branches: [main] | ||
| pull_request: | ||
| branches: ["**"] |
There was a problem hiding this comment.
По моему это не обязательно, и так должен на всех ветках запускаться
There was a problem hiding this comment.
Над проверить.
Я вообще вынесу экшоны из этого PR я думаю
Я в целом на тему перф. тестов пока гонял так и не понял как они работают - есть ощущение что результаты имеют определённый рандом, 0 информативности и полезности в итоге. Сомнительная в общем штука. Так что может быть этот PR и не поедет дальше никуда. |
2 participants
Добавлены тесты производительности компонент в рамках UI kit
Для тестирования производительности используется reassure
Так же в рамках работы поправлены замечания линтера по поводу тестов и добавлени GitHub Workflows для запуска тестов, линтера, тайпчека
Внедрение reassure - эксперимент, тесты написанные в общем тоже, вся эта история скорее всего потребует допила самих тестов.
Проверка производительности пригодится чтобы сравнивать не портят ли перфоманс изменения в новой ветке
Так как код весь в __tests__ аффекта от изменений на работу библиотеки не ожидается