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

35.29%

Dependency count

1

-50%

Number of package files

21

23.53%

Lines of code

299

64.29%

Number of low quality alerts

1

-50%

Worsened metrics

Dev dependency count

5

Infinity%

Number of low supply chain risk alerts

3

50%

Dependency changes

• + Added

  hairballs@0.0.7
  (transitive)

• - Removed

  handlebars@^3.0.1

• - Removed

  ansi-regex@2.1.1
  (transitive)

• - Removed

  ansi-styles@2.2.1
  (transitive)

• - Removed

  argparse@1.0.10
  (transitive)

• - Removed

  asn1@0.2.6
  (transitive)

• - Removed

  assert-plus@0.2.01.0.0
  (transitive)

• - Removed

  asynckit@0.4.0
  (transitive)

• - Removed

  aws-sign2@0.6.0
  (transitive)

• - Removed

  aws4@1.13.2
  (transitive)

• - Removed

  bcrypt-pbkdf@1.0.2
  (transitive)

• - Removed

  boom@2.10.1
  (transitive)

• - Removed

  caseless@0.11.0
  (transitive)

• - Removed

  chalk@1.1.3
  (transitive)

• - Removed

  combined-stream@1.0.8
  (transitive)

• - Removed

  commander@2.20.3
  (transitive)

• - Removed

  core-util-is@1.0.2
  (transitive)

• - Removed

  coveralls@2.13.3
  (transitive)

• - Removed

  cryptiles@2.0.5
  (transitive)

• - Removed

  dashdash@1.14.1
  (transitive)

• - Removed

  delayed-stream@1.0.0
  (transitive)

• - Removed

  ecc-jsbn@0.1.2
  (transitive)

• - Removed

  escape-string-regexp@1.0.5
  (transitive)

• - Removed

  esprima@2.7.3
  (transitive)

• - Removed

  extend@3.0.2
  (transitive)

• - Removed

  extsprintf@1.3.0
  (transitive)

• - Removed

  forever-agent@0.6.1
  (transitive)

• - Removed

  form-data@2.1.4
  (transitive)

• - Removed

  generate-function@2.3.1
  (transitive)

• - Removed

  generate-object-property@1.2.0
  (transitive)

• - Removed

  getpass@0.1.7
  (transitive)

• - Removed

  hairballs@0.0.5
  (transitive)

• - Removed

  har-validator@2.0.6
  (transitive)

• - Removed

  has-ansi@2.0.0
  (transitive)

• - Removed

  hawk@3.1.3
  (transitive)

• - Removed

  hoek@2.16.3
  (transitive)

• - Removed

  http-signature@1.1.1
  (transitive)

• - Removed

  is-my-ip-valid@1.0.1
  (transitive)

• - Removed

  is-my-json-valid@2.20.6
  (transitive)

• - Removed

  is-property@1.0.2
  (transitive)

• - Removed

  is-typedarray@1.0.0
  (transitive)

• - Removed

  isstream@0.1.2
  (transitive)

• - Removed

  js-yaml@3.6.1
  (transitive)

• - Removed

  jsbn@0.1.1
  (transitive)

• - Removed

  json-schema@0.4.0
  (transitive)

• - Removed

  json-stringify-safe@5.0.1
  (transitive)

• - Removed

  jsonpointer@5.0.1
  (transitive)

• - Removed

  jsprim@1.4.2
  (transitive)

• - Removed

  lcov-parse@0.0.10
  (transitive)

• - Removed

  log-driver@1.2.5
  (transitive)

• - Removed

  mime-db@1.52.0
  (transitive)

• - Removed

  mime-types@2.1.35
  (transitive)

• - Removed

  minimist@1.2.0
  (transitive)

• - Removed

  mocha-lcov-reporter@0.0.2
  (transitive)

• - Removed

  oauth-sign@0.8.2
  (transitive)

• - Removed

  pinkie@2.0.4
  (transitive)

• - Removed

  pinkie-promise@2.0.1
  (transitive)

• - Removed

  punycode@1.4.1
  (transitive)

• - Removed

  qs@6.3.3
  (transitive)

• - Removed

  request@2.79.0
  (transitive)

• - Removed

  safer-buffer@2.1.2
  (transitive)

• - Removed

  sntp@1.0.9
  (transitive)

• - Removed

  sprintf-js@1.0.3
  (transitive)

• - Removed

  sshpk@1.18.0
  (transitive)

• - Removed

  stringstream@0.0.6
  (transitive)

• - Removed

  strip-ansi@3.0.1
  (transitive)

• - Removed

  supports-color@2.0.0
  (transitive)

• - Removed

  tough-cookie@2.3.4
  (transitive)

• - Removed

  tunnel-agent@0.4.3
  (transitive)

• - Removed

  tweetnacl@0.14.5
  (transitive)

• - Removed

  uuid@3.4.0
  (transitive)

• - Removed

  verror@1.10.0
  (transitive)

• - Removed

  xtend@4.0.2
  (transitive)

• Updated

  hairballs@0.0.7