From 8bf0bf952cba404d3c0b49f22783cf49ba1dbb8f Mon Sep 17 00:00:00 2001 From: mattjmorrison Date: Thu, 5 Dec 2013 22:44:10 -0600 Subject: Added ability to turn on debug logging via environment variable For the grunt-karma-coveralls project, I need to be able to turn on debug level logging but without using `process.argv`. This will allow me to do so. This addresses this issue: https://github.com/mattjmorrison/grunt-karma-coveralls/issues/2 and can be fixed with this commit: https://github.com/mattjmorrison/grunt-karma-coveralls/commit/4bd6e2b58647dda4a6335fa6077334d8021e23c0 --- lib/logger.js | 22 ++++++++++++++++------ 1 file changed, 16 insertions(+), 6 deletions(-) (limited to 'lib') diff --git a/lib/logger.js b/lib/logger.js index 985b67b..30a6c60 100644 --- a/lib/logger.js +++ b/lib/logger.js @@ -1,8 +1,18 @@ module.exports = function(){ - if (process.argv[2]) { - if (~['-v', '--verbose'].indexOf(process.argv[2])) { - return require('log-driver')({level : 'debug'}); - } - } - return require('log-driver')({level : 'warn'}); + return require('log-driver')({level : getLogLevel()}); }; + +function getLogLevel(){ + if (hasVerboseCommandLineOption() || hasDebugEnvVariable()) { + return 'debug'; + } + return 'warn'; +} + +function hasVerboseCommandLineOption(){ + return process.argv[2] && ~['-v', '--verbose'].indexOf(process.argv[2]); +} + +function hasDebugEnvVariable(){ + return process.env.NODE_COVERALLS_DEBUG == 1; +} -- cgit v1.2.3