Merge pull request #1 from emartech/custom-error

feat(logger): add customError function

Author: sonicoder86

src/logger/logger.js

@@ -31,12 +31,16 @@ class Logger {
     return this._enabled;
   }
 
+  customError(severity, action, error, data = {}) {
+    logMethodFactory(severity).bind(this)(action, Object.assign(this._getErrorDetails(error), data));
+  }
+
   fromError(action, error, data = {}) {
-    this.error(action, Object.assign(this._getErrorDetails(error), data));
+    this.customError('error', action, error, data);
   }
 
   warnFromError(action, error, data = {}) {
-    this.warn(action, Object.assign(this._getErrorDetails(error), data));
+    this.customError('warn', action, error, data);
   }
 
   timer() {

src/logger/logger.spec.js

@@ -121,6 +121,61 @@ describe('Logger', function() {
     expect(logArguments.error_data.length).to.eql(3004);
   });
 
+  describe('#customError', function() {
+    it('should log error as the given severity with action', function() {
+      const error = new Error('failed');
+      error.data = { test: 'data' };
+
+      logger.customError('info', 'hi', error, { details: 'here' });
+
+      const logArguments = JSON.parse(console.log.args[0]);
+      expect(logArguments.name).to.eql('mongo');
+      expect(logArguments.action).to.eql('hi');
+      expect(logArguments.level).to.eql(30);
+      expect(logArguments.details).to.eql('here');
+
+      expect(logArguments.error_name).to.eql(error.name);
+      expect(logArguments.error_stack).to.eql(error.stack);
+      expect(logArguments.error_message).to.eql(error.message);
+      expect(logArguments.error_data).to.eql(JSON.stringify(error.data));
+    });
+
+    it('should not log error data when it is undefined', function() {
+      const error = new Error('failed');
+
+      logger.customError('warn', 'hi', error, { details: 'here' });
+
+      const logArguments = JSON.parse(console.log.args[0]);
+      expect(logArguments.name).to.eql('mongo');
+      expect(logArguments.action).to.eql('hi');
+      expect(logArguments.level).to.eql(40);
+      expect(logArguments.details).to.eql('here');
+
+      expect(logArguments.error_name).to.eql(error.name);
+      expect(logArguments.error_stack).to.eql(error.stack);
+      expect(logArguments.error_message).to.eql(error.message);
+      expect(logArguments).to.not.have.any.keys('error_data');
+    });
+
+    it('should log only 3000 character of data', function() {
+      const error = new Error('failed');
+      error.data = 'exactlyTen'.repeat(400);
+
+      logger.customError('error', 'hi', error, { details: 'here' });
+
+      const logArguments = JSON.parse(console.log.args[0]);
+      expect(logArguments.name).to.eql('mongo');
+      expect(logArguments.action).to.eql('hi');
+      expect(logArguments.level).to.eql(50);
+      expect(logArguments.details).to.eql('here');
+
+      expect(logArguments.error_name).to.eql(error.name);
+      expect(logArguments.error_stack).to.eql(error.stack);
+      expect(logArguments.error_message).to.eql(error.message);
+      expect(logArguments.error_data.length).to.eql(3004);
+    });
+  });
+
   describe('#configure', function() {
     it('should change format method', function() {
       const formattedOutput = '{"my":"method"}';