Skip to content

Commit

Permalink
make it possible for user to touch defaultEmitter
Browse files Browse the repository at this point in the history
  • Loading branch information
Tomohito Nakayama committed Feb 21, 2016
1 parent c705e2d commit 91dc233
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 16 deletions.
17 changes: 2 additions & 15 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -551,21 +551,8 @@ function DefaultEmitter () {};
util.inherits( DefaultEmitter, EventEmitter);
let defaultEmitter = new DefaultEmitter();

function replaceErrorListenerOfDefaultEmitter(errorListener){
defaultEmitter.removeAllListeners('error');
defaultEmitter.on(
'error',
errorListener
);
};

//Initialization of module.
replaceErrorListenerOfDefaultEmitter( defaultErrorListener );
defaultEmitter.on('error',defaultErrorListener);

//User can replace error listener.
module.exports.replaceErrorListenerOfDefaultEmitter =
function( userErrorListener ){
replaceErrorListenerOfDefaultEmitter( userErrorListener );
};
module.exports.defaultEmitter = defaultEmitter;

module.exports.JsonFiledError = JsonFiledError;
4 changes: 3 additions & 1 deletion test/unit/testError.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,9 @@ describe('Error ', function () {
it('should be handled in error listener', function (done) {

/*
jf.replaceErrorListenerOfDefaultEmitter(
jf.defaultEmitter.removeAllListeners('error');
jf.defaultEmitter.on(
'error',
function(err){
console.error(err);
done();
Expand Down

0 comments on commit 91dc233

Please sign in to comment.