diff options
| author | mattjmorrison <[email protected]> | 2013-12-05 22:44:10 -0600 |
|---|---|---|
| committer | mattjmorrison <[email protected]> | 2013-12-05 22:44:10 -0600 |
| commit | 8bf0bf952cba404d3c0b49f22783cf49ba1dbb8f (patch) | |
| tree | 43e4a2777fa2dfb2d8d77f2c32e9c408c6ad9074 /lib | |
| parent | be9cf789b3db0158b8364a48475e5671d45f16e8 (diff) | |
| download | node-coveralls-8bf0bf952cba404d3c0b49f22783cf49ba1dbb8f.tar.xz node-coveralls-8bf0bf952cba404d3c0b49f22783cf49ba1dbb8f.zip | |
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
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/logger.js | 22 |
1 files changed, 16 insertions, 6 deletions
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; +} |
