Merge pull request #45 from codacy/FixJSCoverageReporting

Fix js coverage reporting

lib/impl/lcov.js

@@ -50,9 +50,9 @@
 
                         //TODO: Convert to reduce function
                         stats.lines.details.forEach(function (detail) {
-                            // If a line is not sent to the service then it is considered to be 0, so no need to be redundant in the payload.
+                            // Codacy needs the 0s to know failed coverage data
                             // We also can't have a negative number of hits on a line, so exclude those.
-                            if (detail.hit >= 1) {
+                            if (detail.hit >= 0) {
                                 fileStats.coverage[detail.line] = detail.hit;
                             }
                         });

lib/reporter.js

@@ -11,7 +11,7 @@
         fileReports: Joi.array().required().items(Joi.object({
             filename: Joi.string().required().min(1),
             total: Joi.number().integer().required().min(0).max(100),
-            coverage: Joi.object().pattern(/\d/, Joi.number().integer().min(1))
+            coverage: Joi.object().pattern(/\d/, Joi.number().integer().min(0))
         }).required())
     }).example({total: 50, fileReports: [{filename: 'filename', total: 10, coverage: {1: 1, 2: 3}}]});
 
@@ -21,14 +21,15 @@
         ts: 'typescript',
         tsx: 'typescript',
         coffee: 'coffeescript'
-    }
+    };
 
     function sendByLanguage(endpoint, commitId, token, data) {
         var reportsByLanguage = lodash.groupBy(data.fileReports, function(elem) {
             return languageMap[lodash.head(lodash.takeRight(elem.filename.split('.'), 1))] || 'javascript';
         });
 
         var languageResponses = lodash.map(reportsByLanguage, function(fileReports, language) {
+
             var weighedCoverage = lodash.reduce(fileReports, function(accom, elem) {
                 return accom + (elem.total * Object.keys(elem.coverage).length);
             }, 0);