Fixed warnings not being logged to console

lib/run-compilation.js

@@ -12,21 +12,27 @@ 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
-          });
+      warnings = lint.results
+        .filter(function (f) {
+          return f.warnings;
+        })
+        .map(function (f) {
+          return f.source; // send error instead
+        });
+      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.quiet) {
+        console.log(chalk.yellow(options.formatter(lint.results)));
       }
 
       if (options.failOnError && errors.length) {
@@ -49,5 +55,8 @@ module.exports = function runCompilation(options, compilation, done) {
     errors.forEach(function (err) {
       compilation.errors.push(err);
     });
+    warnings.forEach(function (warning) {
+      compilation.warnings.push(warning);
+    });
   });
 };

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/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

@@ -40,7 +40,6 @@ describe('stylelint-webpack-plugin', function () {
     pack(assign({}, baseConfig, config), function (err, stats) {
       expect(err).to.not.exist;
       expect(stats.compilation.errors).to.have.length(0);
-      expect(stats.compilation.warnings).to.have.length(0);
       done(err);
     });
   });
@@ -118,6 +117,25 @@ describe('stylelint-webpack-plugin', function () {
     });
   });
 
+  it('sends warnings properly', function (done) {
+    var config = {
+      context: './test/fixtures/test8',
+      entry: './index',
+      plugins: [
+        new StyleLintPlugin({
+          quiet: true,
+          configFile: configFilePath
+        })
+      ]
+    };
+
+    pack(assign({}, baseConfig, config), function (err, stats) {
+      expect(err).to.not.exist;
+      expect(stats.compilation.warnings).to.have.length(1);
+      done(err);
+    });
+  });
+
   // it('should work with multiple context', function(done) {
   //   var config = {
   //     context: './test/fixtures/test5',