meilisearch/MeiliSearch

Fix the dump import tests on windows

Ouverte

#2 364 ouverte le 3 mai 2022

Voir sur GitHubGuide contributeur
 (3 commentaires) (1 réaction) (0 personne assignée)Rust (2 542 forks)batch import
contribution acceptedgood first issuehelp wantedmaintenanceversion upgradewindows

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

Windows throw errors when we try to read a dump in the CI (maybe that actually happens everywhere?). See the following action: https://github.com/meilisearch/meilisearch/runs/6215426608?check_suite_focus=true

This could be related to this issue in which case we can't do anything about it.

For now, we ignore all the tests when running on windows.

Guide contributeur

Direction de recherche
Le problème est que les tests d'importation de dump échouent sur Windows en raison d'erreurs de verrouillage de fichier (erreur 32). Actuellement, ces tests sont ignorés sur Windows. Pour résoudre ce problème, il faut enquêter sur le problème de verrouillage de fichier, éventuellement en veillant à ce que les fichiers soient correctement fermés ou en utilisant des fichiers temporaires. Examinez le code de test dans le dépôt MeiliSearch pour comprendre l'importation de dump. Envisagez de modifier les tests pour contourner le comportement de verrouillage de fichier de Windows ou d'utiliser une approche différente.
Stack technique
rust
Domaine
backendtesting
Type d'issue
Bug
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
GitRust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75