Kibana: Utiliser visualizeLoader pour les tableaux de bord

Créé le 24 nov. 2017 · 3Commentaires · Source: elastic/kibana

Je pense que nous devrions utiliser le chargeur de visualisation pour charger les visualisations sur le tableau de bord (c'est-à-dire le VisualizeEmbeddableFactory ).

Actuellement, nous exposons le chargeur comme une sorte d'API pour charger des visualisations, mais nous ne l'utilisons nous-mêmes nulle part. Je pense que son utilisation garantirait que nous le testons en profondeur et qu'il restera également à jour avec les changements que nous apportons.

Nous aurions également une bonne idée des extensions que nous devons apporter à l'API, que quelque chose comme les tableaux de bord peut fonctionner correctement avec elle. Enfin, nous pourrions nous débarrasser d'une autre partie d'Angular dans le tableau de bord, car le chargeur de visualisation expose du JavaScript pur.

Dashboard Vis Loader Visualizations chore

Source

timroes

Commentaire le plus utile

@bhavyarm idéalement, cela ne devrait changer AUCUNE fonctionnalité, car il s'agit simplement d'une refactorisation de code interne. Cela signifie que tout devrait toujours fonctionner comme avant et que tous les tests de tableau de bord devraient s'exécuter de la même manière. Ce changement ne devrait donc pas vous donner de nouveaux scénarios de test, s'il nous manque des tests de tableaux de bord fonctionnels, ce serait les mêmes qui manquaient déjà auparavant :-)

timroes le 8 janv. 2018

🎉1 😄1 👍1

Tous les 3 commentaires

@timroes avons-nous besoin de tests fonctionnels pour cela ? De plus, si j'ajoute chaque visualisation à un tableau de bord et que je le rouvre, ce test est-il suffisant ? Merci!