facebookarchive/prepack

Create smoke-test for prepack-node

Open

#693 geöffnet am 1. Juni 2017

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (520 Forks)batch import
help wantedtest needed

Repository-Metriken

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

Beschreibung

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.

Contributor Guide

Research-Richtung
Überprüfen Sie den Fix in PR #691, um das Ausnahmeproblem zu verstehen. Erstellen Sie dann eine Smoke Test Datei für prepack node, die prepackStdin, prepackFile und prepackFileSync mit grundlegenden Eingaben testet, um sicherzustellen, dass sie keine Ausnahmen auslösen.
Tech Stack
javascriptnodejs
Domain
testing qa
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70