code-corps / code-corps/code-corps-ember

Improve testing coverage after PR

Aberta
#1,464 0 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
JavaScript
Estrelas
120
Forks
75
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

# 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).

Guia de contribuição

Abrir o guia de contribuição

Direção de pesquisa

Comece com o exemplo do Coveralls para app/mixins/onboarding-controller.js e compare a cobertura dele com as alterações em PR 1463, especialmente a refatoração dos imports. Identifique a lógica que é reportada como coberta sem testes e, em seguida, adicione cobertura para esse comportamento e verifique se o relatório geral não diminui mais após a refatoração.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
javascript
Domínio
testing
Tipo de issue
Refatoração
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Precisa de esclarecimento
Facilidade para iniciantes
25/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.