firefox-devtools/profiler

Test coverage for src/components/app

Ouverte

#733 ouverte le 24 janv. 2018

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)TypeScript (486 forks)auto 404
code maintenancehelp wantedtests

Métriques du dépôt

Stars
 (1 515 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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

Guide contributeur

Direction de recherche
Envisagez d'utiliser Enzyme pour le rendu superficiel des composants listés. Commencez par Root.js (le moins couvert) et écrivez des tests unitaires axés sur le comportement du composant et les props.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ReactTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55