From: Aleksander Ɓukasz Date: Mon, 30 Dec 2013 12:09:55 +0000 (+0100) Subject: Simple logging api inspired by Python logging module X-Git-Url: https://git.mdrn.pl/fnpeditor.git/commitdiff_plain/a4e92f85f83046cef29d4ce0f71cea6250ce8df6?ds=sidebyside Simple logging api inspired by Python logging module --- diff --git a/src/fnpjs/logging/handlers.js b/src/fnpjs/logging/handlers.js new file mode 100644 index 0000000..c15d17f --- /dev/null +++ b/src/fnpjs/logging/handlers.js @@ -0,0 +1,8 @@ +define(function() { + +'use strict'; + + +return {}; + +}); \ No newline at end of file diff --git a/src/fnpjs/logging/logging.js b/src/fnpjs/logging/logging.js new file mode 100644 index 0000000..e68302d --- /dev/null +++ b/src/fnpjs/logging/logging.js @@ -0,0 +1,81 @@ +define(function(require) { + +'use strict'; + +var _ = require('libs/underscore'), + handlers = require('fnpjs/logging/handlers'), + config = {}, + levels = ['debug', 'info', 'warning', 'error', 'critical']; + + +var Logger = function(name) { + this.name = name; + Object.defineProperty(this, 'config', { + get: function() { + return _.extend({ + propagate: true, + level: 'warn', + handlers: [] + }, config.loggers[name] || {}); + } + }); +}; + +_.extend(Logger.prototype, { + log: function(level, message, data) { + if(levels.indexOf(level) !== -1 && levels.indexOf(level) >= levels.indexOf(this.config.level)) { + this.config.handlers.forEach(function(handlerName) { + var handlerConfig = config.handlers[handlerName], + handler = handlerConfig.handler, + handlerLevel = handlerConfig.level || 'info'; + + if(typeof handler === 'string') { + handler = handlers[handlerConfig.handler]; + } + if(!handler) { + throw new Error('Unknown handler: ' + handlerName); + } + + if(levels.indexOf(handlerLevel) !== -1 && levels.indexOf(level) >= levels.indexOf(handlerLevel)) { + handler(message, level, data); + } + }); + } + if(this.config.propagate && this.name) { + var logger = new Logger(this.name.split('.').slice(0, -1).join('.')); + logger.log(level, message, data); + } + }, + exception: function(e) { + this.log('error', e.toString(), {exception: e}); + } +}); + +levels.forEach(function(level) { + Logger.prototype[level] = function(message, data) { + return this.log(level, message, data); + }; +}); + + +var api = { + getLogger: function(name) { + return new Logger(name); + }, + setConfig: function(_config) { + config = _.extend({ + handlers: [], + loggers: [] + } ,_config); + }, + clearConfig: function() { + this.setConfig({}); + } +}; + +api.clearConfig(); + + +return api; + +}); diff --git a/src/fnpjs/logging/logging.test.js b/src/fnpjs/logging/logging.test.js new file mode 100644 index 0000000..5876f5a --- /dev/null +++ b/src/fnpjs/logging/logging.test.js @@ -0,0 +1,128 @@ +define(function(require) { + +'use strict'; +/* global describe, it, beforeEach */ +var chai = require('libs/chai'), + logging = require('./logging.js'), + expect = chai.expect; + + +// Global log object for defining expectations +var log = { + _msgs: [], + clear: function() { + this._msgs = []; + }, + append: function(msg) { + this._msgs.push(msg); + }, + contains: function(msg) { + return this._msgs.indexOf(msg) !== -1; + }, + getLast: function() { + return this._msgs.length ? this._msgs[this._msgs.length] : undefined; + }, + isEmpty: function() { + return this._msgs.length === 0; + }, + getMessages: function() { + return this._msgs; + } +}; + +// Loggin handler that just writes to the global logger object +var testLoggingHandler = function(msg) { + log.append(msg); +}; + +describe('Logging', function() { + + + beforeEach(function() { + log.clear(); + logging.clearConfig(); + }); + + var setConfig = function(loggerLevel, handlerLevel) { + logging.setConfig({ + handlers: { + testHandler: { + handler: testLoggingHandler, + level: handlerLevel + } + }, + loggers: { + '': { + level: loggerLevel, + handlers: ['testHandler'] + } + } + }); + }; + + it('works with sample config', function() { + setConfig('debug', 'debug'); + var logger = logging.getLogger('some.name'); + logger.debug('debug msg'); + expect(log.contains('debug msg')).to.equal(true); + }); + + it('filters level on loggers', function() { + setConfig('info', 'debug'); + var logger = logging.getLogger('some.name'); + logger.debug('debug msg'); + expect(log.isEmpty()).to.equal(true, 'debug message filtered out'); + logger.info('info msg'); + expect(log.contains('info msg')).to.equal(true, 'info message passed'); + }); + + it('filters level on handlers', function() { + setConfig('debug', 'info'); + var logger = logging.getLogger('some.name'); + logger.debug('debug msg'); + expect(log.isEmpty()).to.equal(true, 'debug message filtered out'); + logger.info('info msg'); + expect(log.contains('info msg')).to.equal(true, 'info message passed'); + }); + + it('propagates message to upper logger depending on the propagate flag', function() { + var config = { + handlers: { + testHandler: { + handler: testLoggingHandler, + level: 'debug' + } + }, + loggers: { + '': { + level: 'debug', + handlers: ['testHandler'] + }, + 'logger1': { + level: 'debug', + handlers: ['testHandler'] + } + } + }, + logger; + + config.loggers.logger1.propagate = false; + logging.setConfig(config); + + logger = logging.getLogger('logger1'); + + logger.debug('msg1'); + expect(log.contains('msg1')).to.equal(true, 'first message logged'); + expect(log.getMessages().length === 1).to.equal(true, 'logger didn\'t propagate its message'); + + log.clear(); + config.loggers.logger1.propagate = true; + logging.setConfig(config); + + logger.debug('msg2'); + expect(log.contains('msg2')).to.equal(true, 'second message logged'); + expect(log.getMessages().length === 2).to.equal(true, 'second message propagated to upper logger'); + }); +}); + +});