firefox-devtools/profiler

Test coverage for src/components/app

Aberta

#733 aberto em 24 de jan. de 2018

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (0 responsável)TypeScript (486 forks)auto 404
code maintenancehelp wantedtests

Métricas do repositório

Stars
 (1.515 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

In trying to find our worst offenders for test coverage, src/components/app/ stands out as being particularly bad. We might need to take on enzyme as a dependency and shallow rendering to properly deal with some of these components.

Files ordered by lines uncovered:

  • Root.js (77 lines uncovered)
  • ProfileSharing.js (67 lines uncovered)
  • ProfileViewer.js (40 lines uncovered)
  • UrlManager.js (32 lines uncovered)
  • Home.js (27 lines uncovered)
  • SymbolicationStatusOverlay.js (23 lines uncovered)
  • TabBar.js (17 lines uncovered)
  • ProfileFilterNavigator.js (9 lines uncovered)
  • FooterLinks.js (3 lines uncovered)
  • Initializing.js (2 lines uncovered)

┆Issue is synchronized with this Jira Task

Guia do colaborador

Direção de pesquisa
Investigue o uso do Enzyme para renderização superficial dos componentes listados. Comece com Root.js (o mais descoberto) e escreva testes unitários focando no comportamento do componente e nas props.
Pilha de tecnologia
typescriptreact
Domain
frontend
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ReactTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55