lukechilds
/
node
mirror of https://github.com/lukechilds/node.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity
You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
11598 Commits
131 Branches
446 Tags
325 MiB
Tree: 6e78e5feaa
Branches Tags
${ item.name }
Create tag ${ searchTerm }
Create branch ${ searchTerm }
from '6e78e5feaa'
${ noResults }
node/test/parallel/test-event-emitter-set-max-...

11 lines
238 B
Raw Normal View History

test: enable linting for tests Enable linting for the test directory. A number of changes was made so all tests conform the current rules used by lib and src directories. The only exception for tests is that unreachable (dead) code is allowed. test-fs-non-number-arguments-throw had to be excluded from the changes because of a weird issue on Windows CI. PR-URL: https://github.com/nodejs/io.js/pull/1721 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
10 years ago
'use strict';
events: setMaxListeners() should not have side effects This object initialization has been unnecessary since 12cf730b. Ref #3803.
12 years ago
var common = require('../common');
var assert = require('assert');
var events = require('events');
test: enable linting for tests Enable linting for the test directory. A number of changes was made so all tests conform the current rules used by lib and src directories. The only exception for tests is that unreachable (dead) code is allowed. test-fs-non-number-arguments-throw had to be excluded from the changes because of a weird issue on Windows CI. PR-URL: https://github.com/nodejs/io.js/pull/1721 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
10 years ago
var e = new events.EventEmitter();
events: setMaxListeners() should not have side effects This object initialization has been unnecessary since 12cf730b. Ref #3803.
12 years ago
events: _events to object and undefined not null By making sure the _events is always an object there is one less check that needs to be performed by emit. Use undefined instead of null. typeof checks are a lot faster than isArray. There are a few places where the this._events check cannot be removed because it is possible for the user to call those methods after using utils.extend to create their own EventEmitter, but before it has actually been instantiated.
12 years ago
assert.deepEqual(e._events, {});
events: setMaxListeners() should not have side effects This object initialization has been unnecessary since 12cf730b. Ref #3803.
12 years ago
e.setMaxListeners(5);
events: _events to object and undefined not null By making sure the _events is always an object there is one less check that needs to be performed by emit. Use undefined instead of null. typeof checks are a lot faster than isArray. There are a few places where the this._events check cannot be removed because it is possible for the user to call those methods after using utils.extend to create their own EventEmitter, but before it has actually been instantiated.
12 years ago
assert.deepEqual(e._events, {});