Allow to pass a language other than Javascript as param

Author: pedrorijo91

bin/codacy-coverage.js

@@ -20,6 +20,7 @@
         .option('-f, --format [value]', 'Coverage input format')
         .option('-t, --token [value]', 'Codacy Project API Token')
         .option('-c, --commit [value]', 'Commit SHA hash')
+        .option('-l, --language [value', 'Project Language')
         .option('-e, --endpoint [value]', 'Codacy API Endpoint')
         .option('-p, --prefix [value]', 'Project path prefix')
         .option('-v, --verbose', 'Display verbose output')
@@ -31,8 +32,8 @@
         debug: program.debug
     });
 
-    loggerImpl.info(util.format('Started with: token [%j], commitId [%j], endpoint [%j], format [%j], path prefix [%j], verbose [%j], debug [%j]',
-        program.token, program.commit, program.endpoint, program.format, program.prefix, program.verbose, program.debug));
+    loggerImpl.info(util.format('Started with: token [%j], commitId [%j], language [%j], endpoint [%j], format [%j], path prefix [%j], verbose [%j], debug [%j]',
+        program.token, program.commit, program.language, program.endpoint, program.format, program.prefix, program.verbose, program.debug));
 
     process.stdin.on('end', function () {
         loggerImpl.trace('Received file through stdin');

lib/handleInput.js

@@ -7,6 +7,7 @@
         var commit = opts.commit;
         var format = opts.format || 'lcov';
         var pathPrefix = opts.prefix || '';
+        var language = opts.language || 'javascript';
         var loggerImpl;
 
         loggerImpl = logger({
@@ -18,8 +19,8 @@
             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));
+        loggerImpl.info(util.format('Handling input for: token [%j], commitId [%j], language [%j], endpoint [%j], format [%j], path prefix [%j], verbose [%j], debug [%j]',
+            token, commit, language, 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) {
@@ -28,7 +29,7 @@
             loggerImpl.debug('Sending coverage');
             return reporter({
                 endpoint: opts.endpoint
-            }).sendCoverage(token, commitId, parsedCoverage);
+            }).sendCoverage(token, commitId, language, parsedCoverage);
         });
     };
 }(require('./coverageParser'), require('./reporter'), require('./getGitData'), require('./logger'), require('bluebird'), require('util')));

lib/reporter.js

@@ -26,11 +26,11 @@
             throw optionsValid.error;
         }
 
-        var endpoint = (options.endpoint || 'https://api.codacy.com') + '/2.0/coverage/:commitId/javascript';
+        var endpoint = (options.endpoint || 'https://api.codacy.com') + '/2.0/coverage/:commitId/:language';
         logger.debug('Setting up reporter communicating to: ' + endpoint);
 
         return {
-            sendCoverage: function sendCoverage(token, commitId, data) {
+            sendCoverage: function sendCoverage(token, commitId, language, data) {
                 return new Promise(function (resolve, reject) {
                     logger.trace(util.format('Sending Coverage for token [%s] and commitId [%s]', token, commitId));
                     var tokenValid = Joi.validate(token, tokenValidation);
@@ -45,7 +45,7 @@
                         return reject(validationErr);
                     }
 
-                    var url = endpoint.replace(':commitId', commitId);
+                    var url = endpoint.replace(':commitId', commitId).replace(':language', language);
                     logger.trace(util.format('Sending POST to %s', url));
 
                     return request({

test/cli.js

@@ -61,8 +61,8 @@
                     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]/);
+                expect(res).to.match(/Started with: token \["1234"], commitId \["4321"], language \[undefined], endpoint \["something"], format \["lcov"], path prefix \["asdf\/"], verbose \[true], debug \[true]/);
+                expect(res).to.match(/Handling input for: token \["1234"], commitId \["4321"], language \["javascript"], endpoint \["something"], format \["lcov"], path prefix \["asdf\/"], verbose \[true], debug \[true]/);
                 done();
             });
         });

test/reporter.js

@@ -53,7 +53,7 @@
             sampleCoverageData.fileReports[0].coverage['3'] = 0;
 
             return expect(reporter({})
-                .sendCoverage('1234', '4321', sampleCoverageData))
+                .sendCoverage('1234', '4321', 'javascript', sampleCoverageData))
                 .to.eventually.be.rejectedWith(Error, 'child "fileReports" fails because ["fileReports" does not contain 1 required value(s)]');
         });
         it('shouldn\'t be able to create a reporter with invalid options', function () {
@@ -65,23 +65,23 @@
             return helper.setupMockEndpoint('1234', '4321', bodyValidator)
                 .then(function () {
                     return expect(reporter({})
-                        .sendCoverage('1234', '4321', sampleCoverageData))
+                        .sendCoverage('1234', '4321', 'javascript', sampleCoverageData))
                         .to.eventually.be.fulfilled();
                 });
         });
         it('should receive error when non-200 status code', function () {
             return helper.setupMockEndpoint('1234', '4321', bodyValidator, 204)
                 .then(function () {
                     return expect(reporter({})
-                        .sendCoverage('1234', '4321', sampleCoverageData))
+                        .sendCoverage('1234', '4321', 'javascript', sampleCoverageData))
                         .to.eventually.be.rejectedWith(Error, 'Expected Status Code of 200, but got [204]');
                 });
         });
         it('should receive error when 400 level status code', function () {
             return helper.setupMockEndpoint('1234', '4321', bodyValidator, 418)
                 .then(function () {
                     return expect(reporter({})
-                        .sendCoverage('1234', '4321', sampleCoverageData))
+                        .sendCoverage('1234', '4321', 'javascript', sampleCoverageData))
                         .to.eventually.be.rejectedWith(Error, 'Expected Successful Status Code, but got [418]');
                 });
         });