Kibana: Use o visualizeLoader para painéis

Criado em 24 nov. 2017 · 3Comentários · Fonte: elastic/kibana

Acho que devemos usar o visualizador de carregamento para carregar visualizações no painel (ou seja, o VisualizeEmbeddableFactory ).

Atualmente, expomos o carregador como uma espécie de API para carregar visualizações, mas não o usamos em nenhum lugar. Acho que usá-lo garantiria, nós o testamos exaustivamente e também ficará atualizado com as mudanças que fazemos.

Além disso, teríamos uma boa ideia sobre quais extensões precisamos fazer na API, se algo como painéis podem funcionar corretamente com ela. Por último, mas não menos importante, poderíamos nos livrar de outra parte do Angular no Dashboard, uma vez que o visualizador de carregamento expõe JavaScript puro.

Dashboard Vis Loader Visualizations chore

Fonte

timroes

Comentários muito úteis

@bhavyarm idealmente, isso não deve alterar NENHUMA funcionalidade, já que é apenas uma refatoração de código interno. Isso significa que tudo ainda deve funcionar da mesma forma que antes e todos os testes de painel devem ser executados da mesma forma. Portanto, esta mudança não deve fornecer a você nenhum novo cenário de teste, se estivermos perdendo os testes funcionais do painel, seria o mesmo que já estava faltando anteriormente :-)

timroes em 8 jan. 2018

🎉1 😄1 👍1

Todos 3 comentários

@timroes precisamos ter testes funcionais para isso? Além disso, se eu adicionar todas as visualizações a um painel e reabri-lo - isso é um teste suficiente? Obrigado!