facebookarchive/prepack

Enable fixed point checking in serializer tests for all cases

Open

#835 aperta il 25 lug 2017

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
design neededhelp wantedserializer

Metriche repository

Star
 (14.268 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Cerca il TODO in src/test runner.js e modifica la logica di inizializzazione ritardata nel serializzatore per abilitare il controllo del punto fisso quando sono presenti anyDelayedValues.
Tech stack
javascript
Dominio
tooling
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55