diff options
author | Ben Noordhuis <info@bnoordhuis.nl> | 2015-01-20 19:30:50 +0100 |
---|---|---|
committer | Ben Noordhuis <info@bnoordhuis.nl> | 2015-01-20 23:25:57 +0100 |
commit | ee9cd004d8a211871439fc77c0696b79c5d0e52d (patch) | |
tree | b7af0b11de5b56d31aeafe93174483fad480994d /test/parallel/test-event-emitter-get-max-listeners.js | |
parent | 77d68070dafe56b5593ad92759a57c64de6b4cf1 (diff) | |
download | android-node-v8-ee9cd004d8a211871439fc77c0696b79c5d0e52d.tar.gz android-node-v8-ee9cd004d8a211871439fc77c0696b79c5d0e52d.tar.bz2 android-node-v8-ee9cd004d8a211871439fc77c0696b79c5d0e52d.zip |
lib: fix TypeError with EventEmitter#on() abuse
Commit 2931348 added EventEmitter#getMaxListeners() but introduced a
regression when people abuse EventEmitter.prototype.on.call() to call
EventEmitter#on() on a non-EE object. Add a workaround for that.
Fixes: https://github.com/iojs/io.js/issues/523
PR-URL: https://github.com/iojs/io.js/pull/527
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Diffstat (limited to 'test/parallel/test-event-emitter-get-max-listeners.js')
-rw-r--r-- | test/parallel/test-event-emitter-get-max-listeners.js | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/test/parallel/test-event-emitter-get-max-listeners.js b/test/parallel/test-event-emitter-get-max-listeners.js index bcaa70bd4a..57430a4e9b 100644 --- a/test/parallel/test-event-emitter-get-max-listeners.js +++ b/test/parallel/test-event-emitter-get-max-listeners.js @@ -11,3 +11,8 @@ assert.strictEqual(emitter.getMaxListeners(), 0); emitter.setMaxListeners(3); assert.strictEqual(emitter.getMaxListeners(), 3); + +// https://github.com/iojs/io.js/issues/523 - second call should not throw. +var recv = {}; +EventEmitter.prototype.on.call(recv, 'event', function() {}); +EventEmitter.prototype.on.call(recv, 'event', function() {}); |