Kibana: Используйте visualizeLoader для информационных панелей

Созданный на 24 нояб. 2017  ·  3Комментарии  ·  Источник: elastic/kibana

Я думаю, мы должны использовать загрузчик визуализации для загрузки визуализаций на приборную панель (т.е. VisualizeEmbeddableFactory ).

В настоящее время мы представляем загрузчик как своего рода API для загрузки визуализаций, но сами нигде не используем его. Я думаю, что его использование гарантирует, что мы тщательно протестируем его, и он также будет постоянно обновляться с внесенными нами изменениями.

Также мы получили бы хорошее представление о том, какие расширения нам нужно сделать для API, что что-то вроде панелей мониторинга может правильно работать с ним. И последнее, но не менее важное: мы могли бы избавиться от другой части Angular на панели инструментов, поскольку загрузчик визуализации предоставляет чистый JavaScript.

Dashboard Vis Loader Visualizations chore

Самый полезный комментарий

@bhavyarm в идеале не должно изменять ЛЮБУЮ функциональность, поскольку это всего лишь внутренний рефакторинг кода. Это означает, что все должно работать так же, как и раньше, и все тесты панели мониторинга должны выполняться так же. Таким образом, это изменение не должно давать вам никаких новых сценариев тестирования, если нам не хватает тестов функциональной панели, это будут те же тесты, которые уже отсутствовали ранее :-)

Все 3 Комментарий

@timroes нужны ли для этого функциональные тесты? Также, если я добавляю каждую визуализацию на панель управления и снова открываю ее - достаточно ли этого тестирования? Спасибо!

@bhavyarm в идеале не должно изменять ЛЮБУЮ функциональность, поскольку это всего лишь внутренний рефакторинг кода. Это означает, что все должно работать так же, как и раньше, и все тесты панели мониторинга должны выполняться так же. Таким образом, это изменение не должно давать вам никаких новых сценариев тестирования, если нам не хватает тестов функциональной панели, это будут те же тесты, которые уже отсутствовали ранее :-)

Большой. Спасибо @timroes

Была ли эта страница полезной?
0 / 5 - 0 рейтинги