facebookarchive/prepack

Enable fixed point checking in serializer tests for all cases

Open

#835 geöffnet am 25. Juli 2017

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (520 Forks)batch import
design neededhelp wantedserializer

Repository-Metriken

Stars
 (14.268 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Suchen Sie nach dem TODO in src/test runner.js und ändern Sie die verzögerte Initialisierungslogik im Serialisierer, um die Festpunktprüfung zu aktivieren, wenn anyDelayedValues vorhanden sind.
Tech Stack
javascript
Domain
tooling
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptGit
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
55