Merge pull request #14 from codacy/api-2.0

api v2

Author: Rodrigo Fernandes

lib/reporter.js

@@ -2,7 +2,7 @@
     'use strict';
 
     var optionsValidation = Joi.object({
-        endpoint: Joi.string().min(1).optional().example('https://codacy.com/coverage/:token/:commitId')
+        endpoint: Joi.string().min(1).optional().example('https://codacy.com/api/2.0/coverage/:commitId/:language')
     });
     var tokenValidation = Joi.string().required().min(1).example('1234567890');//TODO: Revisit this validation to see if we can better validate the values
     var commitIdValidation = Joi.string().required().min(1).example('1234567890'); //TODO: Revisit this validation to see if we can better validate the values
@@ -26,7 +26,7 @@
             throw optionsValid.error;
         }
 
-        var endpoint = options.endpoint || 'https://www.codacy.com/api/coverage/:token/:commitId';
+        var endpoint = options.endpoint || 'https://www.codacy.com/api/2.0/coverage/:commitId/javascript';
         logger.debug('Setting up reporter communicating to: ' + endpoint);
 
         return {
@@ -45,13 +45,18 @@
                         return reject(validationErr);
                     }
 
-                    var url = endpoint.replace(':token', token).replace(':commitId', commitId);
+                    var url = endpoint.replace(':commitId', commitId);
                     logger.trace(util.format('Sending POST to %s', url));
 
                     return request({
                         url: url,
                         method: 'POST',
                         json: data,
+                        headers: {
+                            // jscs:disable
+                            'project_token': token
+                            // jscs:enable
+                        },
                         resolveWithFullResponse: true
                     }).then(function (res) {
                         if (res.statusCode !== 200) {

package.json

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

test/handleInput.js

@@ -40,7 +40,7 @@
             return helper.setupMockEndpoint('1234', '4321', bodyValidator)
                 .then(function () {
                     return expect(request({
-                        url: 'https://www.codacy.com/api/coverage/1234/4321',
+                        url: 'https://www.codacy.com/api/2.0/coverage/4321/javascript',
                         method: 'POST',
                         json: sampleCoverageData,
                         resolveWithFullResponse: true

test/helper.js

@@ -17,7 +17,7 @@
             expect(bodyValidator).to.be.ok();
 
             return resolve(nock('https://www.codacy.com')
-                .post('/api/coverage/' + token + '/' + commitId, function (body) {
+                .post('/api/2.0/coverage/' + commitId + '/javascript', function (body) {
                     var result = bodyValidator.validate(body);
                     return result.error ? false : true;
                 })

test/reporter.js

@@ -39,7 +39,7 @@
             return helper.setupMockEndpoint('1234', '4321', bodyValidator)
                 .then(function () {
                     return expect(request({
-                        url: 'https://www.codacy.com/api/coverage/1234/4321',
+                        url: 'https://www.codacy.com/api/2.0/coverage/4321/javascript',
                         method: 'POST',
                         json: sampleCoverageData,
                         resolveWithFullResponse: true