microsoft/RecursiveExtractor

Improve tests to validate the correct contents of extracted files

Open

#104 aberto em 20 de dez. de 2022

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C# (36 forks)auto 404
enhancementgood first issuetest

Métricas do repositório

Stars
 (218 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Currently the tests primarily check that the correct number of files are returned from extraction commands. However, as the issue identified in #102 shows this can be insufficient in cases where the underlying library may not be threadsafe (for example). In such cases the tests can still succeed, finding the correct number of files, but their contents may be incomplete. We should develop additional tests that verify for each file type we support that the contents + sizes of each file are correct. Perhaps the easiest method would be with a hash of the contents of the extracted stream.

Guia do colaborador

Direção de pesquisa
Examine o conjunto de testes existente, identifique os testes que contam arquivos e adicione asserções que verifiquem a integridade do conteúdo dos arquivos usando hashes (por exemplo, MD5 ou SHA) para cada tipo de arquivo suportado.
Pilha de tecnologia
csharp
Domain
backendtesting
Tipo Issue
Teste
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.
Meio dia
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
C#Gittesting basics
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65