facebookarchive/prepack

Enhance serializer tests with automated heap traversals

Open

#1 852 ouverte le 3 mai 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (520 forks)batch import
enhancementhelp wantedserializertest needed

Métriques du dépôt

Stars
 (14 268 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Currently, the serializer tests only checks a heap projection: whatever the inspect function computes.

This is great for debugging issues, as each serializer test tends to be minimal and test a separate feature.

However, we are ignoring additional potential. What could be done is effectively synthesizing an alternative inspect function implementation that returns the result of a systematic, deterministic heap traversal.

Challenges:

  1. Ideally, we'd like to detect any changes to built-ins as well as changes rooted in the global object.
  2. Prepack does not preserve everything, e.g. function names and function source code change. So this should be abstracted by the traversal.
  3. Functions are interesting in general: How can we detect that their behavior didn't change? To start with, we can just ignore that and focus on the immediately accessible heap. Maybe some annotations are needed on how to invoke functions to get interesting outputs, which in turn can be traversed.

Guide contributeur

Direction de recherche
Étudiez les tests actuels du sérialiseur dans le dépôt prepack, en particulier la fonction inspect et comment elle calcule les projections du tas. Étudiez les défis listés dans l'issue : détecter les modifications apportées aux objets intégrés et globaux, gérer les fonctions qui ne sont pas préservées par prepack, et définir un parcours systématique du tas. Étant donné le statut archivé, cette issue est plus adaptée à des fins d'apprentissage ou comme exercice de conception.
Stack technique
javascript
Domaine
backendbuild system
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Investigation requise
Prérequis
JavaScripttesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30