facebookarchive/prepack

Create smoke-test for prepack-node

Open

#693 aperta il 1 giu 2017

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (520 fork)batch import
help wantedtest needed

Metriche repository

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

Descrizione

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.

Guida contributor

Direzione di ricerca
Esamina la correzione nella PR #691 per comprendere il problema dell'eccezione. Quindi crea un file di smoke test per prepack node che testi prepackStdin, prepackFile e prepackFileSync con input di base per garantire che non generino eccezioni.
Tech stack
javascriptnodejs
Dominio
testing qa
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70