Meilisearch-laravel-scout: Adicionar testes não simulados
Olá pessoal!
O mock atual do servidor MeiliSearch me deixa um pouco preocupado com nossos testes. Eu prefiro ter testes em execução em um servidor MeiliSearch real. O objetivo é ter certeza de que este pacote funciona em produção com a versão mais recente do MeiliSearch.
Na verdade, MeiliSearch e meilisearch-php ainda não são estáveis, e cada modificação no servidor MeiliSearch nos obriga a atualizar o mock. Caso contrário, nossos testes não estão atualizados com as últimas modificações do MeiliSearch e nosso pacote pode conter erros sem notá-los.
Não digo que devemos remover a parte simulada, mas devemos pelo menos ter a parte de teste de integração em um servidor MeiliSearch real, então por que não manter ambos? Exceto se você achar que isso é muita manutenção, vamos remover a peça simulada então.
Fizemos o mesmo no pacote bundler Symfony: https://github.com/meilisearch/meilisearch-symfony/blob/c764bcdb45f83941bc10d3b461ded5cf8d7f86b7/.github/workflows/tests.yml#L29 -L32
E no repositório do SDK meilisearch-php: https://github.com/meilisearch/meilisearch-php/blob/master/.github/workflows/tests.yml
O que você acha @shokme? (apenas a sua opinião, não estou perguntando se você tem tempo para fazer isso, claro 😁)
E também a todos, qual a sua opinião sobre isso?
curquiza
Todos 4 comentários
Estou totalmente do seu lado neste @curquiza. Adicionar o serviço meilisearch nos testes de CI seria um ótimo acréscimo.
No mesmo ponto, poderíamos dividir os testes em uma estrutura como Unit testes e Feature testes. E deixe os testes simulados atuais viverem.
Minhas alterações em # 89 adicionaram a saída de cobertura de teste que mostra as partes que não foram testadas no momento. Essa parte pode ser o início dos novos testes Feature
mmachatschek
em 1 fev. 2021
Ok, então vamos fazer isso! Isso tornaria nosso repositório mais seguro 💪
curquiza
em 2 fev. 2021
@curquiza isso pode ser fechado? O que você acha
mmachatschek
em 16 mar. 2021
Sim, desculpe, o PR deveria ter fechado automaticamente! 😇
curquiza
em 18 mar. 2021
Questões relacionadas
marky291
·
6Comentários
curquiza
·
17Comentários
milosevicn
·
5Comentários
Kladislav
·
8Comentários
husonghua
·
5Comentários
Comentários muito úteis
Estou totalmente do seu lado neste @curquiza. Adicionar o serviço meilisearch nos testes de CI seria um ótimo acréscimo.
No mesmo ponto, poderíamos dividir os testes em uma estrutura como
Unittestes eFeaturetestes. E deixe os testes simulados atuais viverem.Minhas alterações em # 89 adicionaram a saída de cobertura de teste que mostra as partes que não foram testadas no momento. Essa parte pode ser o início dos novos testes
Feature