rtfeldman/seamless-immutable

Fix tests that relied on Lodash 4

Open

#179 aberto em 11 de dez. de 2016

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)JavaScript (221 forks)batch import
help wanted

Métricas do repositório

Stars
 (5.367 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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! ❤️

Guia do colaborador

Direção de pesquisa
Encontre todos os casos de teste marcados com `xit` no conjunto de testes e reescreva os para usar recursos do Lodash 3.10 em vez dos recursos do Lodash 4. Consulte a documentação do Lodash 3.10 para métodos equivalentes.
Pilha de tecnologia
javascript
Domain
testing
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
None
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75