Meilisearch-laravel-scout: Добавить непроверенные тесты

Созданный на 1 февр. 2021 · 4Комментарии · Источник: meilisearch/meilisearch-laravel-scout

Всем привет!

Текущий макет сервера MeiliSearch заставляет меня немного беспокоиться о наших тестах. Я бы предпочел, чтобы тесты выполнялись на реальном сервере MeiliSearch. Цель состоит в том, чтобы убедиться, что этот пакет работает в производственной среде с последней версией MeiliSearch.
Действительно, MeiliSearch и meilisearch-php еще нестабильны, и каждая модификация на сервере MeiliSearch вынуждает нас обновлять макет. В противном случае наши тесты не соответствуют последним модификациям MeiliSearch, и наш пакет может содержать ошибки, не замечая их.

Я не говорю, что мы должны удалить фиктивную часть, но мы должны, по крайней мере, иметь тестовую часть интеграции с реальным сервером MeiliSearch, так почему бы не сохранить и то, и другое? Если вы не думаете, что это слишком много обслуживания, давайте удалим фиктивную часть.

Мы сделали то же самое в пакете сборщика Symfony: https://github.com/meilisearch/meilisearch-symfony/blob/c764bcdb45f83941bc10d3b461ded5cf8d7f86b7/.github/workflows/tests.yml#L29 -L32
И в репозитории SDK meilisearch-php: https://github.com/meilisearch/meilisearch-php/blob/master/.github/workflows/tests.yml

Как ты думаешь @shokme? (только ваше мнение, я не спрашиваю, есть ли у вас время конечно 😁)
И все, что вы думаете об этом?

Laravel integration

Источник

curquiza

👍2

Самый полезный комментарий

Я полностью на вашей стороне в этом @curquiza. Добавление сервиса meilisearch в тесты CI было бы отличным дополнением.

В этот же момент мы могли бы разделить тесты на такую структуру, как Unit tests и Feature tests. И пусть текущие имитационные тесты продолжаются.

Мои изменения в №89 добавили вывод тестового покрытия, в котором показаны части, которые на данный момент не протестированы. Эта часть может быть началом новых тестов Feature