facebookarchive/prepack

Create smoke-test for prepack-node

Open

#693 opened on 2017年6月1日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)JavaScript (14,268 stars) (520 forks)batch import
help wantedtest needed

説明

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.

コントリビューターガイド

技術スタック
javascriptnodejs
領域
testing
Issue 種別
test
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
なし
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
20
調査方針
The issue requests adding smoke tests for the three exported functions of prepack node: prepackStdin, prepackFile, and prepackFileSync. A previous fix (#691) was not caught by tests, so these tests are needed. The code is likely in a prepack node directory. The contributor should set up a testing environment and write basic tests that call each function to ensure they don't throw exceptions.
Create smoke-test for prepack-node · facebookarchive/prepack#693 | Good First Issue