firefox-devtools/profiler

Test coverage for src/components/app

Offen

#733 geöffnet am 24.01.2018

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (486 Forks)auto 404
code maintenancehelp wantedtests

Repository-Metriken

Stars
 (1.515 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die Verwendung von Enzyme für das flache Rendern der aufgelisteten Komponenten. Beginnen Sie mit Root.js (am wenigsten abgedeckt) und schreiben Sie Unit Tests, die sich auf das Komponentenverhalten und Props konzentrieren.
Tech Stack
typescriptreact
Domain
frontend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
ReactTypeScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55