CesiumGS/cesium

Replace "can support 2xx HTTP status (other than 200)" test

Open

#3.757 geöffnet am 23. März 2016

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (3.324 Forks)batch import
good first issuetest failure

Repository-Metriken

Stars
 (11.758 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 15T 12h) (23 gemergte PRs in 30 T)

Beschreibung

The test can support 2xx HTTP status (other than 200) occasionally fails because it's relying on an external server that seems to have spotty availability. This causes CI to someone fail for no good reason. We should update this test to spy/mock the XHR request so that it doesn't need to access the network at all.

Contributor Guide

Research-Richtung
Finde die genannte Testdatei, verstehe den aktuellen Test, der eine echte XHR Anfrage an einen externen Server stellt, und ersetze ihn durch eine gemockte/gespyte XHR Anfrage mit sinon oder einer ähnlichen Bibliothek, um Netzwerkabhängigkeiten zu vermeiden.
Tech Stack
javascript
Domain
frontend
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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.
70