Meilisearch-laravel-scout: Tambahkan tes yang tidak diejek

Dibuat pada 1 Feb 2021 · 4Komentar · Sumber: meilisearch/meilisearch-laravel-scout

Halo semuanya!

Momen server MeiliSearch saat ini membuat saya sedikit khawatir tentang pengujian kami. Saya lebih suka menjalankan tes terhadap server MeiliSearch yang sebenarnya. Tujuannya adalah untuk memastikan paket ini bekerja dalam produksi dengan versi terbaru MeiliSearch.
Memang, MeiliSearch dan meilisearch-php belum stabil, dan setiap modifikasi di server MeiliSearch memaksa kami untuk memperbarui tiruannya. Jika tidak, pengujian kami tidak mutakhir dengan modifikasi terakhir MeiliSearch dan paket kami dapat berisi kesalahan tanpa menyadarinya.

Saya tidak mengatakan kita harus menghapus bagian yang diejek, tetapi setidaknya kita harus memiliki bagian uji integrasi terhadap server MeiliSearch yang sebenarnya, jadi mengapa tidak menyimpan keduanya? Kecuali jika menurut Anda ini terlalu banyak perawatan, mari kita hapus bagian tiruannya.

Kami telah melakukan hal yang sama dalam paket bundler Symfony: https://github.com/meilisearch/meilisearch-symfony/blob/c764bcdb45f83941bc10d3b461ded5cf8d7f86b7/.github/workflows/tests.yml#L29 -L32
Dan di repo SDK meilisearch-php: https://github.com/meilisearch/meilisearch-php/blob/master/.github/workflows/tests.yml

Bagaimana menurutmu @shokme? (hanya pendapatmu saja, aku tidak bertanya apakah kamu punya waktu untuk melakukannya tentunya 😁)
Dan juga semua orang, apa pendapat Anda tentang itu?

Laravel integration

Sumber

curquiza

👍2

Komentar yang paling membantu

Saya benar-benar di pihak Anda di @curquiza ini. Menambahkan layanan meilisearch dalam tes CI akan menjadi tambahan yang bagus.

Pada saat yang sama, kita dapat membagi pengujian menjadi struktur seperti pengujian Unit dan pengujian Feature . Dan biarkan tes mengejek saat ini hidup bersama.

Perubahan saya di #89 menambahkan output cakupan pengujian yang menunjukkan bagian-bagian yang belum diuji saat ini. Bagian itu bisa menjadi awal dari tes Feature

mmachatschek pada 1 Feb 2021

👍3

Semua 4 komentar

Saya benar-benar di pihak Anda di @curquiza ini. Menambahkan layanan meilisearch dalam tes CI akan menjadi tambahan yang bagus.

Pada saat yang sama, kita dapat membagi pengujian menjadi struktur seperti pengujian Unit dan pengujian Feature . Dan biarkan tes mengejek saat ini hidup bersama.

Perubahan saya di #89 menambahkan output cakupan pengujian yang menunjukkan bagian-bagian yang belum diuji saat ini. Bagian itu bisa menjadi awal dari tes Feature

mmachatschek pada 1 Feb 2021

👍3

Oke, kalau begitu mari kita lakukan ini! Ini akan membuat repo kita lebih aman

curquiza pada 2 Feb 2021

@curquiza ini bisa ditutup? Bagaimana menurut anda

mmachatschek pada 16 Mar 2021

Ya maaf, PR seharusnya menutupnya secara otomatis! 😇

curquiza pada 18 Mar 2021

👍1

Apakah halaman ini membantu?

0 / 5 - 0 peringkat