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/sequential/test-crypto-timing-safe-equal.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/sequential/test-crypto-timing-safe-equal.js')
| -rw-r--r-- | test/sequential/test-crypto-timing-safe-equal.js | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/test/sequential/test-crypto-timing-safe-equal.js b/test/sequential/test-crypto-timing-safe-equal.js index 7a1f8d2993..2847af9ef8 100644 --- a/test/sequential/test-crypto-timing-safe-equal.js +++ b/test/sequential/test-crypto-timing-safe-equal.js @@ -1,12 +1,9 @@ 'use strict'; const common = require('../common'); -const assert = require('assert'); - -if (!common.hasCrypto) { +if (!common.hasCrypto) common.skip('missing crypto'); - return; -} +const assert = require('assert'); const crypto = require('crypto'); assert.strictEqual( |