Skip to content

Commit

Permalink
Merge pull request #55 from nlaplante/master
Browse files Browse the repository at this point in the history
Add logSilent option to suppress logs
  • Loading branch information
fergiemcdowall committed Nov 18, 2014
2 parents 36ca9ff + 57d372b commit fbd1efb
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 8 deletions.
21 changes: 13 additions & 8 deletions lib/search-index.js
Expand Up @@ -25,28 +25,35 @@ var indexes = null, indexesMultiply = null;

var defaults = {
indexPath: 'si',
logLevel: 'info'
logLevel: 'info',
logSilent: false
};

var SearchIndex = module.exports = function (options) {
var logLevel = 'info';
var indexPath = 'si';
var logSilent = false;

if (options) {
if (options.logLevel)
if (options.logLevel)
logLevel = options.logLevel;
if (options.indexPath)
if (options.indexPath)
indexPath = options.indexPath;
if (options.logSilent)
logSilent = options.logSilent;
}
indexes = level(indexPath, {valueEncoding: 'json'}),
indexesMultiply = levelMultiply(indexes);

searchIndexLogger = new (winston.Logger)({
transports: [
new (winston.transports.Console)({ level: logLevel }),
// new (winston.transports.File)({ filename: 'somefile.log' })
new (winston.transports.Console)({
level: logLevel,
silent: logSilent
})
]
});

calibrater.getTotalDocs(indexes, function(totalDocs) {
searcher.setTotalDocs(totalDocs);
});
Expand Down Expand Up @@ -161,5 +168,3 @@ SearchIndex.calibrate = function(callback) {
callback(msg);
});
};


12 changes: 12 additions & 0 deletions test/spec/6config-spec.js
Expand Up @@ -17,4 +17,16 @@ describe('configuration', function () {
return fs.existsSync('si2');
}, 5000);
});

it('should accept logSilent in configuration', function () {
var si;

runs(function () {
si = require('../../lib/search-index.js')({ logSilent: true });
});

waitsFor(function() {
return searchIndexLogger.transports.console.silent;
}, 5000);
});
});

0 comments on commit fbd1efb

Please sign in to comment.