IQSS / IQSS/dataverse-client-javascript

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

Abierto
#186 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

bug GREI Re-arch Size: 10
Lenguaje dominante
TypeScript
Estrellas
20
Forks
11
Merge medio
2 d 5 h
PR fusionados (30 d)
1

Descripción

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

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comience con la configuración de cobertura de ramas del hook pre-commit e inspeccione las clases GetCollection y GetCollectionFacets junto con sus casos de prueba existentes. Reproduzca la cobertura de ramas del 0 % informada, identifique por qué esas ramas no se cuentan y verifique que el hook se ejecute correctamente después de restaurar la cobertura de ramas requerida al 95 %.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
typescript
Área
testing-qa, tooling
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.