Kibana: Utilice visualizeLoader para paneles

Creado en 24 nov. 2017  ·  3Comentarios  ·  Fuente: elastic/kibana

Creo que deberíamos usar el cargador de visualización para cargar visualizaciones en el tablero (es decir, el VisualizeEmbeddableFactory ).

Actualmente, exponemos el cargador como una especie de API para cargar visualizaciones, pero no lo usamos nosotros mismos en ninguna parte. Creo que usarlo aseguraría que lo probaremos a fondo y también se mantendrá actualizado con los cambios que hagamos.

También tendríamos una buena idea de qué extensiones necesitamos hacer para la API, que algo como los paneles de control pueden funcionar correctamente con ella. Por último, pero no menos importante, podríamos deshacernos de otra parte de Angular en el Tablero, ya que el cargador de visualización expone JavaScript puro.

Dashboard Vis Loader Visualizations chore

Comentario más útil

@bhavyarm idealmente esto no debería cambiar NINGUNA funcionalidad, ya que es solo una refactorización de código interno. Lo que significa que todo debería funcionar igual que antes, y todas las pruebas de panel deberían ejecutarse igual. Por lo tanto, este cambio no debería brindarle nuevos escenarios de prueba, si nos faltan las pruebas funcionales del tablero, sería lo mismo que ya faltaba de antemano :-)

Todos 3 comentarios

@timroes ¿necesitamos tener pruebas funcionales para esto? Además, si agrego todas las visualizaciones a un tablero y lo vuelvo a abrir, ¿eso es suficiente prueba? ¡Gracias!

@bhavyarm idealmente esto no debería cambiar NINGUNA funcionalidad, ya que es solo una refactorización de código interno. Lo que significa que todo debería funcionar igual que antes, y todas las pruebas de panel deberían ejecutarse igual. Por lo tanto, este cambio no debería brindarle nuevos escenarios de prueba, si nos faltan las pruebas funcionales del tablero, sería lo mismo que ya faltaba de antemano :-)

Excelente. Gracias @timroes

¿Fue útil esta página
0 / 5 - 0 calificaciones

Temas relacionados

passkey1510 picture passkey1510  ·  96Comentarios

Alex-Ikanow picture Alex-Ikanow  ·  364Comentarios

hvisage picture hvisage  ·  170Comentarios

cff3 picture cff3  ·  83Comentarios

pkubat picture pkubat  ·  75Comentarios