Skip to content

Commit

Permalink
test: refactor test-crypto-random
Browse files Browse the repository at this point in the history
* specify constructor for assert.throws()
* load additional modules only if crypto check passes
* normalize some potentially confusing indentation
* provided actual first and expected second in assertions

PR-URL: #10232
Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
  • Loading branch information
Trott authored and evanlucas committed Dec 14, 2016
1 parent 2f9c8d9 commit a97f264
Showing 1 changed file with 7 additions and 16 deletions.
23 changes: 7 additions & 16 deletions test/parallel/test-crypto-random.js
Original file line number Diff line number Diff line change
@@ -1,37 +1,28 @@
'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');

crypto.DEFAULT_ENCODING = 'buffer';

// bump, we register a lot of exit listeners
process.setMaxListeners(256);

[crypto.randomBytes,
crypto.pseudoRandomBytes
].forEach(function(f) {
[-1,
undefined,
null,
false,
true,
{},
[]
].forEach(function(value) {
assert.throws(function() { f(value); });
assert.throws(function() { f(value, function() {}); });
[crypto.randomBytes, crypto.pseudoRandomBytes].forEach(function(f) {
[-1, undefined, null, false, true, {}, []].forEach(function(value) {
assert.throws(function() { f(value); }, TypeError);
assert.throws(function() { f(value, function() {}); }, TypeError);
});

[0, 1, 2, 4, 16, 256, 1024].forEach(function(len) {
f(len, common.mustCall(function(ex, buf) {
assert.strictEqual(null, ex);
assert.strictEqual(len, buf.length);
assert.strictEqual(ex, null);
assert.strictEqual(buf.length, len);
assert.ok(Buffer.isBuffer(buf));
}));
});
Expand Down

0 comments on commit a97f264

Please sign in to comment.