1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: refactor test-crypto-pbkdf2

Browse Source 
* re-order require() and crypto check per test writing guide
* use common.mustNotCall() to confirm callback is not invoked

PR-URL: https://github.com/nodejs/node/pull/13975
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
This commit is contained in:
Rich Trott 2017-06-28 20:41:24 -07:00
parent 0d95a0b60a
commit 48660fbb9d
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/parallel/test-crypto-pbkdf2.js
View File

@ -1,11 +1,12 @@
'use strict'; 'use strict';
const common = require('../common'); const common = require('../common');
const assert = require('assert');
if (!common.hasCrypto) { if (!common.hasCrypto) {
common.skip('missing crypto'); common.skip('missing crypto');
return; return;
} }
const assert = require('assert');
const crypto = require('crypto'); const crypto = require('crypto');
// //
@ -98,7 +99,7 @@ assert.doesNotThrow(() => {
}); });
assert.throws(() => { assert.throws(() => {
crypto.pbkdf2('password', 'salt', 8, 8, common.noop); crypto.pbkdf2('password', 'salt', 8, 8, common.mustNotCall());
}, /^TypeError: The "digest" argument is required and must not be undefined$/); }, /^TypeError: The "digest" argument is required and must not be undefined$/);
assert.throws(() => { assert.throws(() => {