meilisearch/MeiliSearch

Fix the dump import tests on windows

Aberta

#2.364 aberto em 3 de mai. de 2022

Ver no GitHubGuia do colaborador
 (3 comentários) (1 reação) (0 responsável)Rust (2.542 forks)batch import
contribution acceptedgood first issuehelp wantedmaintenanceversion upgradewindows

Métricas do repositório

Stars
 (57.572 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
O problema é que os testes de importação de dump falham no Windows devido a erros de bloqueio de arquivo (erro 32). Atualmente, esses testes são ignorados no Windows. Para corrigir, investigue o problema de bloqueio de arquivo, possivelmente garantindo que os arquivos sejam fechados corretamente ou usando arquivos temporários. Examine o código de teste no repositório do MeiliSearch para entender a importação de dump. Considere modificar os testes para contornar o comportamento de bloqueio de arquivo do Windows ou usar uma abordagem diferente.
Pilha de tecnologia
rust
Domain
backendtesting
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitRust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75