Meilisearch-laravel-scout: モックされていないテストを追加する

作成日 2021年02月01日  ·  4コメント  ·  ソース: meilisearch/meilisearch-laravel-scout

こんにちは、みんな!

MeiliSearchサーバーの現在のモックでは、テストについて少し心配しています。 実際のMeiliSearchサーバーに対してテストを実行したいと思います。 目標は、このパッケージがMeiliSearchの最新バージョンに対して本番環境で機能することを確認することです。
実際、MeiliSearchとmeilisearch-phpはまだ安定しておらず、MeiliSearchサーバーを変更するたびに、モックを更新する必要があります。 そうしないと、MeiliSearchの最終変更でテストが最新にならず、パッケージに気付かずにエラーが含まれる可能性があります。

モック部分を削除する必要があるとは言いませんが、少なくとも実際のMeiliSearchサーバーに対して統合テスト部分を用意する必要があるので、両方を保持しないのはなぜですか。 メンテナンスが多すぎると思われる場合を除いて、モック部分を削除しましょう。

Symfonyバンドラーパッケージでも同じことを行いました: https
そして、meilisearch-php SDKリポジトリ: https

@shokmeはどう思いますか? (あなたの意見ですが、もちろんそれをする時間があるかどうかは尋ねていません😁)
そして皆さん、それについてどう思いますか?

Laravel integration

最も参考になるコメント

私はこの@curquizaで完全にあなたの味方です。 CIテストにmeilisearchサービスを追加すると、すばらしい追加になります。

同じ時点で、テストをUnitテストやFeatureテストのような構造に分割することができます。 そして、現在の模擬テストを存続させましょう。

#89での私の変更により、現時点でテストされていないパーツを示すテストカバレッジ出力が追加されました。 その部分は、新しいFeatureテストの始まりである可能性があります

全てのコメント4件

私はこの@curquizaで完全にあなたの味方です。 CIテストにmeilisearchサービスを追加すると、すばらしい追加になります。

同じ時点で、テストをUnitテストやFeatureテストのような構造に分割することができます。 そして、現在の模擬テストを存続させましょう。

#89での私の変更により、現時点でテストされていないパーツを示すテストカバレッジ出力が追加されました。 その部分は、新しいFeatureテストの始まりである可能性があります

では、これを実行しましょう。 これにより、リポジトリがより安全になります💪

@curquizaこれを閉じることはできますか? どう思いますか

はい、申し訳ありませんが、PRは自動的にそれを閉じるべきでした! 😇

このページは役に立ちましたか?
0 / 5 - 0 評価

関連する問題

milosevicn picture milosevicn  ·  5コメント

JexPY picture JexPY  ·  8コメント

dreamcog picture dreamcog  ·  14コメント

marky291 picture marky291  ·  6コメント

curquiza picture curquiza  ·  17コメント