Skip to content

Commit 782c583

Browse files
committed
Assign the right thing to 'delay', and use it consistently
1 parent 3b4eeb4 commit 782c583

2 files changed

Lines changed: 4 additions & 3 deletions

File tree

lib/sockets.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ var Readable = Stream.Readable || require('readable-stream/readable');
88
var Writable = Stream.Writable || require('readable-stream/writable');
99
var Duplex = Stream.Duplex || require('readable-stream/duplex');
1010

11-
var delay = global.setImmediate || nextTick;
11+
var delay = global.setImmediate || process.nextTick;
1212

1313
// A default value to substitute when we don't get a callback
1414
function ignore() {}
@@ -113,7 +113,7 @@ function Socket(setup, options) {
113113

114114
function closeAndInvalidate(event, err) {
115115
this.readable = this.writable = false;
116-
setImmediate(this.emit.bind(this, event, err));
116+
delay(this.emit.bind(this, event, err));
117117
}
118118

119119
var close = closeAndInvalidate.bind(this, 'close');

test/tests.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ var assert = require('assert');
22

33
var createContext = require('../index').createContext;
44

5+
var delay = global.setImmediate || process.nextTick;
56

67
var PARAMS = process.env['AMQP_PARAMS'];
78
if (PARAMS) {
@@ -45,7 +46,7 @@ function testWithContext(test) {
4546
return function(done) { // mocha looks at the number of arguments
4647
withContext(function(ctx) {
4748
var closeAndDone = function(maybeErr) {
48-
setImmediate(ctx.close.bind(ctx));
49+
delay(ctx.close.bind(ctx));
4950
done(maybeErr);
5051
};
5152
ctx.on('ready', function() { return test(closeAndDone, ctx); });

0 commit comments

Comments
 (0)