fix: warnings not being logged to console (#26)

* Fixed warnings not being logged to console

Warnings were not being logged unless there was an error in the css linting.

* Corrected number of warnings returned

Author: iblack10

lib/run-compilation.js

@@ -12,21 +12,22 @@ var linter = require('./linter');
  */
 module.exports = function runCompilation(options, compilation, done) {
   var errors = [];
+  var warnings = [];
 
   linter(options)
     .then(function (lint) {
-      if (lint.errored) {
-        errors = lint.results
-          .filter(function (f) {
-            return f.errored;
-          })
-          .map(function (f) {
-            return f.source; // send error instead
-          });
-
-        if (!options.quiet) {
-          console.log(chalk.yellow(options.formatter(lint.results)));
-        }
+      warnings = lint.results
+        .filter(function (f) {
+          return f.warnings && f.warnings.length;
+        });
+      errors = lint.results
+        .filter(function (f) {
+          return f.errored;
+        }).map(function (f) {
+          return f.source; // send error instead
+        });
+      if (!options.quiet) {
+        console.log(chalk.yellow(options.formatter(lint.results)));
       }
 
       if (options.failOnError && errors.length) {
@@ -46,8 +47,7 @@ module.exports = function runCompilation(options, compilation, done) {
 
   // eslint-disable-next-line no-unused-expressions
   compilation.plugin && compilation.plugin('compilation', function (compilation) {
-    errors.forEach(function (err) {
-      compilation.errors.push(err);
-    });
+    compilation.errors = compilation.errors.concat(errors);
+    compilation.warnings = compilation.warnings.concat(warnings);
   });
 };

test/.stylelintrc

@@ -11,7 +11,8 @@
     "block-opening-brace-newline-after": "always-multi-line",
     "block-opening-brace-space-after": "always-single-line",
     "block-opening-brace-space-before": "always",
-    "color-hex-case": "lower",
+    "color-hex-case": [ "lower", { "severity": "warning" }
+    ],
     "color-hex-length": "short",
     "color-no-invalid-hex": true,
     "comment-empty-line-before": [ "always", {

test/fixtures/test7/index.js

@@ -1,3 +1,3 @@
 require(getPath('./../../../node_modules/file-loader/index') + '!./test.scss');
 
-console.log('test6');
+console.log('test7');

test/fixtures/test8/index.js

@@ -0,0 +1,3 @@
+require(getPath('./../../../node_modules/file-loader/index') + '!./test.scss');
+
+console.log('test8');

test/fixtures/test8/test.scss

@@ -0,0 +1,3 @@
+body {
+  color: #FFF;
+}

test/index.js

@@ -29,7 +29,6 @@ describe('stylelint-webpack-plugin', function () {
     return pack(assign({}, baseConfig, config))
       .then(function (stats) {
         expect(stats.compilation.errors).to.have.length(0);
-        expect(stats.compilation.warnings).to.have.length(0);
       });
   });
 
@@ -73,4 +72,22 @@ describe('stylelint-webpack-plugin', function () {
         expect(stats.compilation.errors).to.have.length(2);
       });
   });
+
+  it('sends warnings properly', function () {
+    var config = {
+      context: './test/fixtures/test8',
+      entry: './index',
+      plugins: [
+        new StyleLintPlugin({
+          quiet: true,
+          configFile: configFilePath
+        })
+      ]
+    };
+
+    return pack(assign({}, baseConfig, config))
+      .then(function (stats) {
+        expect(stats.compilation.warnings).to.have.length(1);
+      });
+  });
 });