meilisearch/MeiliSearch

Add a test to export chat settings in dumps

Open

#5 736 ouverte le 7 juil. 2025

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Rust (733 forks)batch import
good first issuemaintenance

Métriques du dépôt

Stars
 (20 887 stars)
Métriques de merge PR
 (Merge moyen 4j 3h) (37 PRs mergées en 30 j)

Description

Hey, I overall it's nice, I think we should add a test in https://github.com/meilisearch/meilisearch/blob/fef089c7b6a47961c64f3851cce034e3202e011e/crates/dump/src/lib.rs#L546

But if we want to land the PR for the rc.0 IMO we merge it as-is and do the test later / open an issue

Originally posted by @irevoire in https://github.com/meilisearch/meilisearch/pull/5731#pullrequestreview-2992732964

Guide contributeur

Direction de recherche
Examinez le fichier suggéré dans crates/dump/src/lib.rs autour de la ligne 546 pour comprendre l'implémentation actuelle de l'exportation des paramètres de chat. Consultez la PR liée #5731 pour le contexte de ce qui a été fusionné. Écrivez un test qui vérifie l'exportation des paramètres de chat dans les dumps, en suivant les modèles de test existants dans le même fichier ou crate.
Stack technique
rust
Domaine
backendtesting
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rustcargo testdump module structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80