IQSS / IQSS/dataverse-client-javascript

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

Offen
#186 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

bug GREI Re-arch Size: 10
Vorherrschende Sprache
TypeScript
Sterne
20
Forks
11
Ø Merge
2 T. 5 Std.
Gemergte PRs (30 T.)
1

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie mit der Branch-Coverage-Konfiguration des pre-commit-Hooks und untersuchen Sie die Klassen GetCollection und GetCollectionFacets zusammen mit ihren bestehenden Testfällen. Reproduzieren Sie die gemeldete Branch-Coverage von 0 %, ermitteln Sie, warum diese Branches nicht gezählt werden, und überprüfen Sie, dass der Hook erfolgreich durchläuft, nachdem die erforderliche Branch-Coverage auf 95 % wiederhergestellt wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
testing-qa, tooling
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.