IQSS / IQSS/dataverse-client-javascript

Fix branch test coverage in classes with 0% coverage and restore branch expected coverage back to 95%

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

Personne n'a encore pris cette issue.

bug GREI Re-arch Size: 10
Langage dominant
TypeScript
Étoiles
20
Forks
11
Merge moyen
2 j 5 h
PR mergées (30 j)
1

Description

The pre-commit hook was failing because it expected 95% branch coverage, but certain classes (GetCollection and GetCollectionFacets) were showing 0% branch coverage, even though we have test cases that should cover these branches.

To temporarily resolve this issue, we lowered the required branch coverage to 90% for the pre-commit hook to pass. However, we need to investigate the cause of this discrepancy, fix the issue, and restore the required branch coverage back to 95%

Screenshot 2024-09-03 at 12 22 19

Guide de contribution

Ouvrir le guide de contribution

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 la configuration de la couverture des branches du hook pre-commit et examinez les classes GetCollection et GetCollectionFacets ainsi que leurs cas de test existants. Reproduisez la couverture des branches de 0 % signalée, identifiez pourquoi ces branches ne sont pas comptabilisées et vérifiez que le hook passe après avoir rétabli la couverture des branches requise à 95 %.

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

Évaluation

Stack technique
typescript
Domaine
testing-qa, tooling
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

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