meilisearch/MeiliSearch

Add tests for the snapshots

Ouverte

#2 992 ouverte le 27 oct. 2022

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Rust (2 542 forks)batch import
help wantedmaintenanceOLD

Métriques du dépôt

Stars
 (57 572 étoiles)
Métriques de merge PR
 (Merge moyen 11j 20h) (41 PRs mergées en 30 j)

Description

There currently are not enough tests for the snapshot feature. We should make sure that they work correctly when:

  • We can create a snapshot in the right folder with the whole set of files (tasks files that contain documents).
  • We make sure that the CLI options about snapshots work correctly.

Guide contributeur

Direction de recherche
Examinez l'implémentation de la fonctionnalité de snapshot dans index scheduler/src/batch.rs (lignes 556-645). Comprenez comment les snapshots sont créés et quels fichiers sont impliqués. Passez en revue les tests existants dans le dépôt pour suivre les modèles de test. Le problème mentionne les options CLI pour les snapshots, donc vérifiez également le code de gestion des arguments CLI pour vérifier que ces options fonctionnent correctement. Concentrez vous sur l'écriture de tests qui créent des snapshots et vérifient la structure et le contenu des fichiers.
Stack technique
rust
Domaine
backend
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustTesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75