facebookarchive/prepack

Create smoke-test for prepack-node

Open

#693 ouverte le 1 juin 2017

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

Métriques du dépôt

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

Description

A recent change to the code resulted in PrepackFile throwing an exception whenever it's invoked (see fix in #691 ), but that was not detected by any of our tests. We should add a basic smoke test for each of the exported functions from prepack-node (prepackStdin, prepackFile, and prepackFileSync) to allow for such clear-cut breaks to be automatically caught by testing.

Guide contributeur

Direction de recherche
Examinez le correctif dans la PR #691 pour comprendre le problème d'exception. Créez ensuite un fichier de test de fumée pour prepack node qui teste prepackStdin, prepackFile et prepackFileSync avec des entrées de base pour garantir qu'ils ne lèvent pas d'exceptions.
Stack technique
javascriptnodejs
Domaine
testing qa
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70