code-corps / code-corps/code-corps-ember
Improve testing coverage after PR
- Vorherrschende Sprache
- JavaScript
- Sterne
- 120
- Forks
- 75
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
# Problem
This [PR](https://github.com/code-corps/code-corps-ember/pull/1463#issuecomment-340201852) resulted in a substantial drop in test coverage (~1%) due to refactoring of imports. This seemingly shouldn't have an impact on testing coverage; however, the current thought is the imports from before this PR were picked up as covered by coveralls when in fact the actual logic was not covered. Here is an [example](https://coveralls.io/builds/13934987/source?filename=app%2Fmixins%2Fonboarding-controller.js).
Beitragsleitfaden
Rechercherichtung
Beginne mit dem Coveralls-Beispiel für app/mixins/onboarding-controller.js und vergleiche dessen Abdeckung mit den Änderungen in PR 1463, insbesondere der Import-Refaktorierung. Ermittle die Logik, die als abgedeckt gemeldet wird, ohne dass Tests dafür vorhanden sind, füge anschließend Abdeckung für dieses Verhalten hinzu und überprüfe, dass der Gesamtbericht nach der Refaktorierung nicht mehr abfällt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- testing
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100