rtfeldman/seamless-immutable

Fix tests that relied on Lodash 4

Aperta

#179 aperta il 11 dic 2016

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)JavaScript (221 fork)batch import
help wanted

Metriche repository

Star
 (5367 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

I mistakenly upgraded our tests to use Lodash 4, not realizing some of the features in Lodash 4 did not work in IE9-10. (seamless-immutable itself does not depend on Lodash; this was for tests only.)

We subsequently added tests that relied on those features, causing IE9-10 tests to fail. I have reverted to Lodash 3.10 and disabled those tests, so that the other tests can complete cross-browser testing as normal. However, now we have a few pending tests that need to be rewritten to use Lodash 3.10 features instead of the Lodash 4 features they currently use.

The pending tests are marked with xit in the test suite.

Help on this would be greatly appreciated! ❤️

Guida contributor

Direzione di ricerca
Trova tutti i casi di test contrassegnati con `xit` nella suite di test e riscrivili per utilizzare le funzionalità di Lodash 3.10 invece di quelle di Lodash 4. Fai riferimento alla documentazione di Lodash 3.10 per metodi equivalenti.
Tech stack
javascript
Dominio
testing
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
Nessuno
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75