cssinjs/jss

Throw Error instead of warning when running tests

Open

#891 geöffnet am 14. Okt. 2018

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (434 Forks)batch import
help wantedideaimportant

Repository-Metriken

Stars
 (7.000 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

I see in the logs many warnings, which I think shouldn't be there, which means we are ignoring problems without noticing in the CI.

I am wondering if we should replace the warning when running tests with a function that throws errors so that our tests can't ignore them.

As a bonus we won't ned to rewire in so many places, but instead use expect for catching.

Contributor Guide

Research-Richtung
Untersuchen Sie den in Tests verwendeten Warnmechanismus, identifizieren Sie alle Instanzen, in denen Warnungen aufgerufen werden, und ersetzen Sie sie durch ein Throw Muster. Stellen Sie sicher, dass Tests, die Warnungen erwarten, try/catch verwenden.
Tech Stack
javascript
Domain
testing
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
GitJavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65