test: refactor test-crypto-pbkdf2

* re-order require() and crypto check per test writing guide * use common.mustNotCall() to confirm callback is not invoked PR-URL: nodejs#13975 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Daniel Bevenius <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Refael Ackermann <[email protected]>
Trott · Jul 2, 2017 · 48660fb · 48660fb
1 parent 0d95a0b
commit 48660fb
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test/parallel/test-crypto-pbkdf2.js b/test/parallel/test-crypto-pbkdf2.js
@@ -1,11 +1,12 @@
 'use strict';
 const common = require('../common');
-const assert = require('assert');
 
 if (!common.hasCrypto) {
   common.skip('missing crypto');
   return;
 }
+
+const assert = require('assert');
 const crypto = require('crypto');
 
 //
@@ -98,7 +99,7 @@ assert.doesNotThrow(() => {
 });
 
 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$/);
 
 assert.throws(() => {