code-corps / code-corps/code-corps-ember
Improve testing coverage after PR
- Lenguaje dominante
- JavaScript
- Estrellas
- 120
- Forks
- 75
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
# 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).
Guía de contribución
Línea de trabajo
Comienza con el ejemplo de Coveralls para app/mixins/onboarding-controller.js y compara su cobertura con los cambios de PR 1463, especialmente la refactorización de imports. Identifica la lógica que se informa como cubierta sin pruebas y, después, añade cobertura para ese comportamiento y verifica que el informe general ya no disminuya tras la refactorización.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- testing
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100