Avoid registering too many event listeners in tests (#477)
Instead of muting the warning that Node.js displays about it.
diff --git a/spec/cli.spec.js b/spec/cli.spec.js
index 51bbb09..b614630 100644
--- a/spec/cli.spec.js
+++ b/spec/cli.spec.js
@@ -21,9 +21,6 @@
const telemetry = require('../src/telemetry');
const cli = rewire('../src/cli');
-// avoid node complaining of too many event listener added
-process.setMaxListeners(0);
-
describe('cordova cli', () => {
let logger;
@@ -33,8 +30,9 @@
// logging events registered as a result of the "--verbose" flag in
// CLI testing below would cause lots of logging messages printed out by other specs.
- // This is required so that fake events chaining works (events.on('log').on('verbose')...)
- spyOn(events, 'on').and.returnValue({ on () { return this; } });
+ // Prevent listeners from piling up
+ spyOn(process, 'on');
+ events.removeAllListeners();
// Spy and mute output
logger = jasmine.createSpyObj('logger', [
