Meilisearch-laravel-scout: Nicht verspottete Tests hinzufügen

Erstellt am 1. Feb. 2021  ·  4Kommentare  ·  Quelle: meilisearch/meilisearch-laravel-scout

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?

Laravel integration

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 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

Alle 4 Kommentare

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! 😇

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen

Verwandte Themen

milosevicn picture milosevicn  ·  5Kommentare

desaintflorent picture desaintflorent  ·  8Kommentare

elfeffe picture elfeffe  ·  30Kommentare

marky291 picture marky291  ·  6Kommentare

dreamcog picture dreamcog  ·  14Kommentare