scss-lint-html-reporter - npm Package Compare versions

Comparing version 0.2.1 to 0.2.2

package.json

 {
   "name": "scss-lint-html-reporter",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "SCSS Lint HTML Reporter with optional TeamCity integration",
@@ -8,3 +8,4 @@ "main": "./src/index.js",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "istanbul cover _mocha",
+    "coveralls": "istanbul cover ./node_modules/.bin/_mocha --report lcovonly -- -R spec && cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js"
   },
@@ -33,5 +34,11 @@ "repository": {
   "dependencies": {
-    "hairballs": "0.0.5",
-    "handlebars": "^3.0.1"
+    "hairballs": "0.0.7"
+  },
+  "devDependencies": {
+    "chai": "^2.2.0",
+    "coveralls": "^2.11.2",
+    "istanbul": "^0.3.13",
+    "mocha": "^2.2.4",
+    "mocha-lcov-reporter": "0.0.2"
   }
 }

src/index.js

@@ -1,3 +0,1 @@
-#!/usr/bin/env node
-
 var fs = require('fs');
@@ -10,8 +8,8 @@
 module.exports = function(input) {
-  var lintReporter = new LintReporter(input);
-  var data = lintReporter.runReport();
+  var lintReporter = new LintReporter();
+  var data = lintReporter.runReport(input);
 
   if (lintReporter.generateHtml) {
     fs.writeFile(lintReporter.outputPath, templateUtils.applyTemplates(data), function(err) {
-      if (err) {
+      if(err) {
         throw err;
@@ -18,0 +16,0 @@ }

src/js/lint-reporter.js

@@ -11,3 +11,3 @@ /**
 
-function LintReporter(jsonOutput) {
+function LintReporter() {
 
@@ -82,8 +82,5 @@ /**
 
-    var teamCityLogger = new TeamCityLogger('SCSS Lint');
-    teamCityLogger.reportStart();
-
     for (var fileName in data) {
 
-      teamCityLogger.testStart(fileName);
+      this.teamCityLogger.testStart(fileName);
 
@@ -99,6 +96,6 @@ var file = { path: fileName, errors: 0, warnings: 0, messages: [], errorList: [] };
       if (file.errorList.length) {
-        teamCityLogger.testFailed(fileName, file.errorList);
+        this.teamCityLogger.testFailed(fileName, file.errorList);
       }
 
-      teamCityLogger.testEnd(fileName);
+      this.teamCityLogger.testEnd(fileName);
       hairballs.updateFileSummary(file);
@@ -112,9 +109,2 @@
     }
-
-    teamCityLogger.reportEnd();
-
-    // output team city report to the console
-    if (this.useTeamCityReport) {
-      console.log(teamCityLogger.reportOutput.join('\n'));
-    }
   };
@@ -128,11 +118,16 @@
   this.fixJSON = function(jsonString) {
-    if (!jsonString) {
-      return '';
-    }
+    try {
+      jsonString = jsonString
+        .replace(/\n/g, '')
+        .replace(/\r/g, '');
 
-    jsonString = jsonString
-      .replace(/\n/g, '')
-      .replace(/\r/g, '');
+      var jsonObject = JSON.parse(jsonString);
 
-    return JSON.parse(jsonString);
+      return jsonObject;
+
+    } catch(err) {
+      this.teamCityLogger.logMessage('SCSS Lint Error', 'An error occured parsing the lint results', 'ERROR');
+
+      console.log('An error occured parsing the JSON: ' + err);
+    }
   };
@@ -168,6 +163,10 @@
    */
-  this.runReport = function() {
+  this.runReport = function(jsonOutput) {
+
+    this.teamCityLogger.reportStart();
+
+    this.checkParameters();
+
     var data = this.fixJSON(jsonOutput);
 
-    this.checkParameters();
     this.summarizeData(data);
@@ -180,2 +179,9 @@
 
+    this.teamCityLogger.reportEnd();
+
+    // output team city report to the console
+    if (this.useTeamCityReport) {
+      console.log(this.teamCityLogger.reportOutput.join('\n'));
+    }
+
     return {
@@ -198,2 +204,3 @@ fileSummary: hairballs.fileSummary,
   this.outputPath = 'scss-lint-report.html';
+  this.teamCityLogger = new TeamCityLogger('SCSS Lint');
   this.ruleUrl = 'https://github.com/brigade/scss-lint/blob/master/lib/scss_lint/linter/README.md#';
@@ -200,0 +207,0 @@ }

Fixed alerts

No tests

Quality

Package does not have any tests. This is a strong signal of a poorly maintained or low quality package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

15975

increased by35.29%

Dependency count

1

decreased by-50%

Number of package files

21

increased by23.53%

Lines of code

299

increased by64.29%

Number of low quality alerts

1

decreased by-50%

Worsened metrics

Dev dependency count

5

increased byInfinity%

Number of low supply chain risk alerts

2

increased by100%

Dependency changes

• + Addedhairballs@0.0.7(transitive)

• - Removedhandlebars@^3.0.1

• - Removedansi-regex@2.1.1(transitive)

• - Removedansi-styles@2.2.1(transitive)

• - Removedargparse@1.0.10(transitive)

• - Removedasn1@0.2.6(transitive)

• - Removedassert-plus@0.2.01.0.0(transitive)

• - Removedasynckit@0.4.0(transitive)

• - Removedaws-sign2@0.6.0(transitive)

• - Removedaws4@1.13.2(transitive)

• - Removedbcrypt-pbkdf@1.0.2(transitive)

• - Removedboom@2.10.1(transitive)

• - Removedcaseless@0.11.0(transitive)

• - Removedchalk@1.1.3(transitive)

• - Removedcombined-stream@1.0.8(transitive)

• - Removedcommander@2.20.3(transitive)

• - Removedcore-util-is@1.0.2(transitive)

• - Removedcoveralls@2.13.3(transitive)

• - Removedcryptiles@2.0.5(transitive)

• - Removeddashdash@1.14.1(transitive)

• - Removeddelayed-stream@1.0.0(transitive)

• - Removedecc-jsbn@0.1.2(transitive)

• - Removedescape-string-regexp@1.0.5(transitive)

• - Removedesprima@2.7.3(transitive)

• - Removedextend@3.0.2(transitive)

• - Removedextsprintf@1.3.0(transitive)

• - Removedforever-agent@0.6.1(transitive)

• - Removedform-data@2.1.4(transitive)

• - Removedgenerate-function@2.3.1(transitive)

• - Removedgenerate-object-property@1.2.0(transitive)

• - Removedgetpass@0.1.7(transitive)

• - Removedhairballs@0.0.5(transitive)

• - Removedhar-validator@2.0.6(transitive)

• - Removedhas-ansi@2.0.0(transitive)

• - Removedhawk@3.1.3(transitive)

• - Removedhoek@2.16.3(transitive)

• - Removedhttp-signature@1.1.1(transitive)

• - Removedis-my-ip-valid@1.0.1(transitive)

• - Removedis-my-json-valid@2.20.6(transitive)

• - Removedis-property@1.0.2(transitive)

• - Removedis-typedarray@1.0.0(transitive)

• - Removedisstream@0.1.2(transitive)

• - Removedjs-yaml@3.6.1(transitive)

• - Removedjsbn@0.1.1(transitive)

• - Removedjson-schema@0.4.0(transitive)

• - Removedjson-stringify-safe@5.0.1(transitive)

• - Removedjsonpointer@5.0.1(transitive)

• - Removedjsprim@1.4.2(transitive)

• - Removedlcov-parse@0.0.10(transitive)

• - Removedlog-driver@1.2.5(transitive)

• - Removedmime-db@1.52.0(transitive)

• - Removedmime-types@2.1.35(transitive)

• - Removedminimist@1.2.0(transitive)

• - Removedmocha-lcov-reporter@0.0.2(transitive)

• - Removedoauth-sign@0.8.2(transitive)

• - Removedpinkie@2.0.4(transitive)

• - Removedpinkie-promise@2.0.1(transitive)

• - Removedpunycode@1.4.1(transitive)

• - Removedqs@6.3.3(transitive)

• - Removedrequest@2.79.0(transitive)

• - Removedsafer-buffer@2.1.2(transitive)

• - Removedsntp@1.0.9(transitive)

• - Removedsprintf-js@1.0.3(transitive)

• - Removedsshpk@1.18.0(transitive)

• - Removedstringstream@0.0.6(transitive)

• - Removedstrip-ansi@3.0.1(transitive)

• - Removedsupports-color@2.0.0(transitive)

• - Removedtough-cookie@2.3.4(transitive)

• - Removedtunnel-agent@0.4.3(transitive)

• - Removedtweetnacl@0.14.5(transitive)

• - Removeduuid@3.4.0(transitive)

• - Removedverror@1.10.0(transitive)

• - Removedxtend@4.0.2(transitive)

• Updatedhairballs@0.0.7