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 <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
  • Loading branch information
Trott authored and MylesBorins committed Jan 24, 2017
1 parent cd2b32e commit d8bc456
Showing 1 changed file with 11 additions and 32 deletions.
43 changes: 11 additions & 32 deletions test/parallel/test-crypto-random.js
Original file line number Diff line number Diff line change
@@ -1,55 +1,34 @@
'use strict';
var common = require('../common');
var assert = require('assert');
const common = require('../common');

if (!common.hasCrypto) {
common.skip('missing crypto');
return;
}
var crypto = require('crypto');

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, checkCall(function(ex, buf) {
assert.equal(null, ex);
assert.equal(len, buf.length);
f(len, common.mustCall(function(ex, buf) {
assert.strictEqual(ex, null);
assert.strictEqual(buf.length, len);
assert.ok(Buffer.isBuffer(buf));
}));
});
});

// assert that the callback is indeed called
function checkCall(cb, desc) {
var called_ = false;

process.on('exit', function() {
assert.equal(true, called_, desc || ('callback not called: ' + cb));
});

return function() {
return called_ = true, cb.apply(cb, Array.prototype.slice.call(arguments));
};
}

// #5126, "FATAL ERROR: v8::Object::SetIndexedPropertiesToExternalArrayData()
// length exceeds max acceptable value"
assert.throws(function() {
Expand Down

0 comments on commit d8bc456

Please sign in to comment.