diff options
| author | Arpad Borsos <[email protected]> | 2013-06-02 14:20:12 +0200 |
|---|---|---|
| committer | Arpad Borsos <[email protected]> | 2013-06-02 14:20:12 +0200 |
| commit | 6187392ca1c1bfcbb4d8ced51de35ed2a04bc8a3 (patch) | |
| tree | b0caaa1d5c21933629eb00db58a2ec4bcc8d8929 /lib | |
| parent | a04d30f7473e93c6390f99090caf5d8a27213dc2 (diff) | |
| download | node-coveralls-6187392ca1c1bfcbb4d8ced51de35ed2a04bc8a3.tar.xz node-coveralls-6187392ca1c1bfcbb4d8ced51de35ed2a04bc8a3.zip | |
Fixes 7: be quiet by default
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/convertLcovToCoveralls.js | 12 | ||||
| -rw-r--r-- | lib/handleInput.js | 22 |
2 files changed, 17 insertions, 17 deletions
diff --git a/lib/convertLcovToCoveralls.js b/lib/convertLcovToCoveralls.js index 4657f10..6290d0c 100644 --- a/lib/convertLcovToCoveralls.js +++ b/lib/convertLcovToCoveralls.js @@ -25,15 +25,17 @@ var convertLcovFileObject = function(file, filepath){ coverage : coverage }; }; -var convertLcovToCoveralls = function(input, filepath, repo_token, cb){ - console.log("in: ", filepath); +var convertLcovToCoveralls = function(input, options, cb){ + var repo_token = options.repo_token, + filepath = options.filepath || ''; + options.verbose && console.log("in: ", filepath); if (filepath[0] !== '/'){ filepath = path.join(process.cwd(), filepath); } lcovParse(input, function(err, parsed){ - if (err){ - console.log("error from lcovParse: ", err); - console.log("input: ", input); + if (err){ + console.error("error from lcovParse: ", err); + console.error("input: ", input); return cb(err); } var postJson = { diff --git a/lib/handleInput.js b/lib/handleInput.js index d081e94..be85c4a 100644 --- a/lib/handleInput.js +++ b/lib/handleInput.js @@ -3,31 +3,29 @@ var path = require('path'); var YAML = require('libyaml'); var sendToCoveralls = require('../index').sendToCoveralls; var convertLcovToCoveralls = require('../index').convertLcovToCoveralls; -var repo_token; -var handleInput = function(input){ - console.log(input); - var libDir = process.argv[2] || ''; +var handleInput = function(input, options){ + options.verbose && console.log(input); if (process.env.COVERALLS_REPO_TOKEN) { - repo_token = process.env.COVERALLS_REPO_TOKEN; + options.repo_token = process.env.COVERALLS_REPO_TOKEN; } else { var yml = path.join(process.cwd(), '.coveralls.yml'); try { if (fs.statSync(yml).isFile()) { - repo_token = YAML.readFileSync(yml)[0].repo_token; + options.repo_token = YAML.readFileSync(yml)[0].repo_token; } } catch(ex){ - console.log("Repo token could not be determined. Continuing without it."); + console.warn("Repo token could not be determined. Continuing without it."); } } - convertLcovToCoveralls(input, libDir, repo_token, function(err, postData){ + convertLcovToCoveralls(input, options, function(err, postData){ if (err){ - console.log("error from convertLcovToCoveralls"); + console.error("error from convertLcovToCoveralls"); throw err; } - console.log("sending this to coveralls.io: ", postData); + options.verbose && console.info("sending this to coveralls.io: ", postData); sendToCoveralls(postData, function(err, response, body){ if (err){ throw err; @@ -35,8 +33,8 @@ var handleInput = function(input){ if (response.statusCode >= 400){ throw "Bad response: " + response.statusCode + " " + body; } - console.log(response.statusCode); - console.log(body); + options.verbose && console.log(response.statusCode); + options.verbose && console.log(body); }); }); |
