diff options
| author | Vse Mozhet Byt <vsemozhetbyt@gmail.com> | 2017-07-01 02:29:09 +0300 |
|---|---|---|
| committer | Jeremiah Senkpiel <fishrock123@rocketmail.com> | 2017-07-19 15:02:06 -0400 |
| commit | bc99efc9739d252d5307c78fd40fc80307cd6d98 (patch) | |
| tree | ff660b831d97728a8410a588228ca8f452da623d /test/parallel/test-http-full-response.js | |
| parent | 898fe6135b49585013f35e2a3e1aa68834d8e8b6 (diff) | |
| download | node-new-bc99efc9739d252d5307c78fd40fc80307cd6d98.tar.gz | |
test: simplify test skipping
* Make common.skip() exit.
Also add common.printSkipMessage() for partial skips.
* Don't make needless things before skip
PR-URL: https://github.com/nodejs/node/pull/14021
Fixes: https://github.com/nodejs/node/issues/14016
Reviewed-By: Refael Ackermann <refack@gmail.com>
Diffstat (limited to 'test/parallel/test-http-full-response.js')
| -rw-r--r-- | test/parallel/test-http-full-response.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/parallel/test-http-full-response.js b/test/parallel/test-http-full-response.js index dc52a5bc2f..4221343209 100644 --- a/test/parallel/test-http-full-response.js +++ b/test/parallel/test-http-full-response.js @@ -43,7 +43,7 @@ function runAb(opts, callback) { exec(command, function(err, stdout, stderr) { if (err) { if (/ab|apr/i.test(stderr)) { - common.skip(`problem spawning \`ab\`.\n${stderr}`); + common.printSkipMessage(`problem spawning \`ab\`.\n${stderr}`); process.reallyExit(0); } process.exit(); |