Browse Source

test: future-proof simple/test-event-emitter-memory-leak

Run the garbage collector before running the actual test. It doesn't
matter now but if in the future something in node.js core creates a lot
of reclaimable garbage, that will break the test's expectation.
v0.10.16-release
Ben Noordhuis 12 years ago
parent
commit
9e1eb361e8
  1. 4
      test/simple/test-event-emitter-memory-leak.js

4
test/simple/test-event-emitter-memory-leak.js

@ -28,6 +28,8 @@ var assert = require('assert');
var events = require('events');
assert.equal(typeof gc, 'function', 'Run this test with --expose-gc');
gc();
var before = process.memoryUsage().heapUsed;
var e = new events.EventEmitter();
@ -36,7 +38,7 @@ for (var i = 0; i < 2.5e5; ++i) {
e.on(name, assert.fail);
e.removeListener(name, assert.fail);
}
while (gc());
gc();
var after = process.memoryUsage().heapUsed;
assert(after - before < 1024*1024, 'EventEmitter leaks event names.');

Loading…
Cancel
Save