Meilisearch-laravel-scout: Agregar pruebas no simuladas
¡Hola, todos!
El simulacro actual del servidor MeiliSearch me preocupa un poco por nuestras pruebas. Prefiero que las pruebas se ejecuten en un servidor MeiliSearch real. El objetivo es asegurarse de que este paquete funcione en producción con la última versión de MeiliSearch.
De hecho, MeiliSearch y meilisearch-php aún no son estables, y cada modificación en el servidor MeiliSearch nos obliga a actualizar el simulacro. De lo contrario, nuestras pruebas no están actualizadas con las últimas modificaciones de MeiliSearch y nuestro paquete podría contener errores sin darnos cuenta.
No digo que debamos eliminar la parte simulada, pero al menos deberíamos tener la parte de prueba de integración contra un servidor MeiliSearch real, así que ¿por qué no mantener ambos? Excepto si cree que esto es demasiado mantenimiento, eliminemos la parte simulada.
Hemos hecho lo mismo en el paquete de paquetes de Symfony: https://github.com/meilisearch/meilisearch-symfony/blob/c764bcdb45f83941bc10d3b461ded5cf8d7f86b7/.github/workflows/tests.yml#L29 -L32
Y en el repositorio de SDK de meilisearch-php: https://github.com/meilisearch/meilisearch-php/blob/master/.github/workflows/tests.yml
¿Qué opinas @shokme? (solo tu opinión, no te pregunto si tienes tiempo para hacerlo claro 😁)
Y también todos, ¿cuál es su opinión al respecto?
curquiza
Todos 4 comentarios
Estoy totalmente de tu lado en esto @curquiza. Agregar el servicio meilisearch en las pruebas de CI sería una gran adición.
En el mismo punto, podríamos dividir las pruebas en una estructura como Unit tests y Feature tests. Y deja que las pruebas simuladas actuales sigan adelante.
Mis cambios en el n. ° 89 agregaron la salida de cobertura de prueba que muestra las partes que no están probadas en este momento. Esa parte podría ser el comienzo de las nuevas pruebas Feature
mmachatschek
en 1 feb. 2021
¡Bien, entonces hagamos esto! Esto haría que nuestro repositorio sea más seguro 💪
curquiza
en 2 feb. 2021
@curquiza ¿esto se puede cerrar? Qué opinas
mmachatschek
en 16 mar. 2021
Sí, lo siento, ¡el RP debería haberlo cerrado automáticamente! 😇
curquiza
en 18 mar. 2021
Temas relacionados
dmyers
·
14Comentarios
elfeffe
·
30Comentarios
marky291
·
6Comentarios
husonghua
·
5Comentarios
dreamcog
·
14Comentarios
Comentario más útil
Estoy totalmente de tu lado en esto @curquiza. Agregar el servicio meilisearch en las pruebas de CI sería una gran adición.
En el mismo punto, podríamos dividir las pruebas en una estructura como
Unittests yFeaturetests. Y deja que las pruebas simuladas actuales sigan adelante.Mis cambios en el n. ° 89 agregaron la salida de cobertura de prueba que muestra las partes que no están probadas en este momento. Esa parte podría ser el comienzo de las nuevas pruebas
Feature