Hallo alle zusammen!
Der aktuelle Mock des MeiliSearch-Servers macht mir bei unseren Tests etwas Sorgen. Ich würde lieber Tests gegen einen echten MeiliSearch-Server ausführen lassen. Das Ziel besteht darin, sicherzustellen, dass dieses Paket in der Produktion mit der neuesten Version von MeiliSearch funktioniert.
Tatsächlich sind MeiliSearch und meilisearch-php noch nicht stabil, und jede Änderung im MeiliSearch-Server zwingt uns dazu, den Mock zu aktualisieren. Ansonsten sind unsere Tests mit den letzten Änderungen von MeiliSearch nicht auf dem neuesten Stand und unser Paket könnte Fehler enthalten, ohne sie zu bemerken.
Ich sage nicht, dass wir den verspotteten Teil entfernen sollten, aber wir sollten zumindest den Integrationstestteil gegen einen echten MeiliSearch-Server haben, also warum nicht beide behalten? Außer wenn Sie der Meinung sind, dass dies zu viel Wartung ist, entfernen wir dann das Scheinteil.
Dasselbe haben wir im Symfony-Bundler-Paket gemacht: https://github.com/meilisearch/meilisearch-symfony/blob/c764bcdb45f83941bc10d3b461ded5cf8d7f86b7/.github/workflows/tests.yml#L29 -L32
Und im meilisearch-php SDK-Repository: https://github.com/meilisearch/meilisearch-php/blob/master/.github/workflows/tests.yml
Was denkst du @shokme? (nur deine Meinung, ich frage dich natürlich nicht, ob du dafür Zeit hast 😁)
Und auch alle, wie ist eure Meinung dazu?
Ich bin bei diesem @curquiza total auf deiner Seite. Die Aufnahme des meilisearch-Dienstes in die CI-Tests wäre eine großartige Ergänzung.
Gleichzeitig könnten wir die Tests in eine Struktur wie Unit
Tests und Feature
Tests aufteilen. Und lassen Sie die aktuellen Scheintests weiterleben.
Meine Änderungen in #89 haben die Ausgabe der Testabdeckung hinzugefügt, die die Teile anzeigt, die im Moment ungetestet sind. Dieser Teil könnte der Beginn der neuen Feature
Tests sein
Okay, dann machen wir das! Das würde unser Repo sicherer machen 💪
@curquiza kann das geschlossen werden? Was denken Sie
Ja, sorry, der PR hätte es automatisch schließen sollen! 😇
Hilfreichster Kommentar
Ich bin bei diesem @curquiza total auf deiner Seite. Die Aufnahme des meilisearch-Dienstes in die CI-Tests wäre eine großartige Ergänzung.
Gleichzeitig könnten wir die Tests in eine Struktur wie
Unit
Tests undFeature
Tests aufteilen. Und lassen Sie die aktuellen Scheintests weiterleben.Meine Änderungen in #89 haben die Ausgabe der Testabdeckung hinzugefügt, die die Teile anzeigt, die im Moment ungetestet sind. Dieser Teil könnte der Beginn der neuen
Feature
Tests sein