microsoft / microsoft/RecursiveExtractor

Improve tests to validate the correct contents of extracted files

Ouverte
#104 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

enhancement good first issue test
Langage dominant
C#
Étoiles
220
Forks
35
Merge moyen
20 j 1 h
PR mergées (30 j)
1

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par localiser les tests existants pour les commandes d’extraction et examinez l’issue #102 pour le scénario d’échec. Étendez la couverture à chaque type de fichier pris en charge afin que les tests vérifient le contenu et la taille des fichiers extraits, éventuellement en calculant le hash de chaque flux extrait. La tâche est terminée lorsque les tests détectent les contenus incomplets, même si le nombre de fichiers attendu est renvoyé.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
csharp
Domaine
testing-qa
Type d'issue
Refactorisation
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.