CesiumGS/cesium

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

Open

#3757 aperta il 23 mar 2016

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (3324 fork)batch import
good first issuetest failure

Metriche repository

Star
 (11.758 star)
Metriche merge PR
 (Merge medio 15g 12h) (23 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Trova il file di test menzionato, comprendi il test corrente che effettua una richiesta XHR reale a un server esterno e sostituiscilo con una richiesta XHR mockata/spiata usando sinon o una libreria simile per evitare la dipendenza dalla rete.
Tech stack
javascript
Dominio
frontend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70