Meilisearch-laravel-scout: Ajouter des tests non simulés

Créé le 1 févr. 2021 · 4Commentaires · Source: meilisearch/meilisearch-laravel-scout

Bonjour à tous!

La maquette actuelle du serveur MeiliSearch m'inquiète un peu pour nos tests. Je préférerais que les tests soient exécutés sur un vrai serveur MeiliSearch. L'objectif est de s'assurer que ce package fonctionne en production avec la dernière version de MeiliSearch.
En effet, MeiliSearch et meilisearch-php ne sont pas encore stables, et chaque modification du serveur MeiliSearch nous oblige à mettre à jour le mock. Sinon, nos tests ne sont pas à jour avec les dernières modifications de MeiliSearch et notre package pourrait contenir des erreurs sans qu'on s'en aperçoive.

Je ne dis pas que nous devrions supprimer la partie simulée, mais nous devrions au moins avoir la partie test d'intégration contre un vrai serveur MeiliSearch, alors pourquoi ne pas garder les deux ? Sauf si vous pensez que c'est trop d'entretien, retirons alors la partie fictive.

Nous avons fait la même chose dans le package du bundle Symfony : https://github.com/meilisearch/meilisearch-symfony/blob/c764bcdb45f83941bc10d3b461ded5cf8d7f86b7/.github/workflows/tests.yml#L29 -L32
Et dans le référentiel du SDK meilisearch-php : https://github.com/meilisearch/meilisearch-php/blob/master/.github/workflows/tests.yml

Qu'en penses-tu @shokme ? (juste votre avis, je ne vous demande pas si vous avez le temps de le faire bien sûr )
Et tout le monde aussi, qu'en pensez-vous ?

Laravel integration

Source

curquiza

👍2

Commentaire le plus utile

Je suis totalement de ton côté sur ce @curquiza. L'ajout du service meilisearch dans les tests CI serait un excellent ajout.

Au même moment, nous pourrions diviser les tests en une structure comme Unit tests et Feature tests. Et laissez vivre les tests simulés actuels.

Mes modifications dans #89 ont ajouté la sortie de couverture de test qui montre les pièces qui ne sont pas testées pour le moment. Cette partie pourrait être le début des nouveaux tests Feature