Browse Source

test: refactor domain tests

* Check that noop callback is or isn't invoked as appropriate using
  common.mustCall() and common.mustNotCall()
* Fix typo in array literal

PR-URL: https://github.com/nodejs/node/pull/13480
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
v6.x
Rich Trott 8 years ago
committed by Myles Borins
parent
commit
3bdf7bf9e9
No known key found for this signature in database GPG Key ID: 933B01F40B5CA946
  1. 6
      test/parallel/test-domain-crypto.js
  2. 4
      test/parallel/test-domain-timers.js
  3. 4
      test/parallel/test-domain.js

6
test/parallel/test-domain-crypto.js

@ -17,7 +17,7 @@ global.domain = require('domain');
// should not throw a 'TypeError: undefined is not a function' exception
crypto.randomBytes(8);
crypto.randomBytes(8, common.noop);
crypto.randomBytes(8, common.mustCall());
crypto.pseudoRandomBytes(8);
crypto.pseudoRandomBytes(8, common.noop);
crypto.pbkdf2('password', 'salt', 8, 8, common.noop);
crypto.pseudoRandomBytes(8, common.mustCall());
crypto.pbkdf2('password', 'salt', 8, 8, 'sha1', common.mustCall());

4
test/parallel/test-domain-timers.js

@ -3,8 +3,6 @@ const common = require('../common');
const domain = require('domain');
const assert = require('assert');
let timeout;
const timeoutd = domain.create();
timeoutd.on('error', common.mustCall(function(e) {
@ -32,4 +30,4 @@ immediated.run(function() {
});
});
timeout = setTimeout(common.noop, 10 * 1000);
const timeout = setTimeout(common.mustNotCall(), 10 * 1000);

4
test/parallel/test-domain.js

@ -1,7 +1,7 @@
'use strict';
// Simple tests of most basic domain functionality.
const common = require('../common');
require('../common');
const assert = require('assert');
const domain = require('domain');
const events = require('events');
@ -238,7 +238,7 @@ let fst = fs.createReadStream('stream for nonexistent file');
d.add(fst);
expectCaught++;
[42, null, , false, common.noop, 'string'].forEach(function(something) {
[42, null, undefined, false, () => {}, 'string'].forEach(function(something) {
const d = new domain.Domain();
d.run(function() {
process.nextTick(function() {

Loading…
Cancel
Save