Added test to make sure that options are being properly conveyed to both the CLI and handle input pieces.

Author: DavidTPate

lib/handleInput.js

@@ -1,4 +1,4 @@
-(function (parser, reporter, getGitData, logger, Promise) {
+(function (parser, reporter, getGitData, logger, Promise, util) {
     'use strict';
     module.exports = function (input, opts) {
         opts = opts || {};
@@ -18,6 +18,9 @@
             return Promise.reject(new Error('Token is required'));
         }
 
+        loggerImpl.info(util.format('Handling input for: token [%j], commitId [%j], endpoint [%j], format [%j], path prefix [%j], verbose [%j], debug [%j]',
+            token, commit, opts.endpoint, format, pathPrefix, opts.verbose, opts.debug));
+
         // Parse the coverage data for the given format and retrieve the commit id if we don't have it.
         return Promise.all([parser.getParser(format).parse(pathPrefix, input), getGitData.getCommitId(commit)]).spread(function (parsedCoverage, commitId) {
             // Now that we've parse the coverage data to the correct format, send it to Codacy.
@@ -28,4 +31,4 @@
             }).sendCoverage(token, commitId, parsedCoverage);
         });
     };
-}(require('./coverageParser'), require('./reporter'), require('./getGitData'), require('./logger'), require('bluebird')));
+}(require('./coverageParser'), require('./reporter'), require('./getGitData'), require('./logger'), require('bluebird'), require('util')));

package.json

@@ -1,6 +1,6 @@
 {
   "name": "codacy-coverage",
-  "version": "1.0.4",
+  "version": "1.1.0",
   "description": "Code Coverage reporter for Codacy.com",
   "keywords": [
     "codacy",

test/cli.js

@@ -50,11 +50,21 @@
                     }
 
                     expect(res).to.match(/Status Code \[404\] - Error \[{"error":"not found"}\]/);
-                    console.log(res);
                     //nock.done(); //TODO: Need to figure out how to use nock here. Since it's a separate process, it's not tied together.
                     done();
                 });
             });
         });
+        it('should be able to set options', function (done) {
+            exec('cat ./test/mock/no-lines.info | node ./bin/codacy-coverage.js --debug --verbose --token 1234 --commit 4321 --prefix asdf/ --endpoint something --format lcov', function (err, res) {
+                if (err) {
+                    return done(err);
+                }
+
+                expect(res).to.match(/Started with: token \["1234"], commitId \["4321"], endpoint \["something"], format \["lcov"], path prefix \["asdf\/"], verbose \[true], debug \[true]/);
+                expect(res).to.match(/Handling input for: token \["1234"], commitId \["4321"], endpoint \["something"], format \["lcov"], path prefix \["asdf\/"], verbose \[true], debug \[true]/);
+                done();
+            });
+        });
     });
 }(require('child_process').exec, require('joi'), require('../'), require('./helper')));