facebookarchive/prepack

Enable fixed point checking in serializer tests for all cases

Open

#835 aberto em 25 de jul. de 2017

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (520 forks)batch import
design neededhelp wantedserializer

Métricas do repositório

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

Description

Currently, fixed point checking is disabled in the serializer test runner (src/test-runner.js) when there are anyDelayedValues.

Search for a TODO with the text Make delayed initializations logic more sophisticated in order to still reach a fixed point..

The delayed initialization logic implemented in the serializer needs to be tweaked to avoid this significant weakening of the serializer tests.

Guia do colaborador

Direção de pesquisa
Procure o TODO em src/test runner.js e modifique a lógica de inicialização atrasada no serializador para habilitar a verificação de ponto fixo quando houver anyDelayedValues.
Pilha de tecnologia
javascript
Domain
tooling
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
55