tests: add tests in case of warnings from loader

packages/webpack-cli/lib/utils/Compiler.js

@@ -80,7 +80,7 @@ class Compiler {
             logger.error(err.stack || err);
             process.exit(1); // eslint-disable-line
         }
-        if (!outputOptions.watch && stats.hasErrors()) {
+        if (!outputOptions.watch && (stats.hasErrors() || stats.hasWarnings())) {
             process.exitCode = 1;
         }
         if (outputOptions.json) {

test/loader/warning-test/loader-warning.test.js

@@ -0,0 +1,12 @@
+'use strict';
+
+const { run } = require('../../utils/test-utils');
+
+describe('loader warning test', () => {
+    it(`should not ignore loader's warning and exit with a non zero exit code`, () => {
+        const { stdout, exitCode } = run(__dirname, [], false);
+
+        expect(stdout).toContain('This is a warning');
+        expect(exitCode).not.toEqual(0);
+    });
+});

test/loader/warning-test/my-loader.js

@@ -0,0 +1,5 @@
+module.exports = function loader(source) {
+    const { emitWarning } = this;
+    emitWarning('This is a warning');
+    return source;
+};

test/loader/warning-test/src/main.js

@@ -0,0 +1 @@
+console.log('loader warning test');

test/loader/warning-test/webpack.config.js

@@ -0,0 +1,30 @@
+const path = require('path');
+
+module.exports = {
+    mode: 'development',
+
+    entry: {
+        bundle: './src/main.js',
+    },
+
+    output: {
+        path: path.resolve(__dirname, 'dist'),
+        filename: '[name].js',
+    },
+
+    module: {
+        rules: [
+            {
+                test: /.(js|jsx)?$/,
+                loader: 'my-loader',
+                include: [path.resolve(__dirname, 'src')],
+                exclude: [/node_modules/],
+            },
+        ],
+    },
+    resolveLoader: {
+        alias: {
+            'my-loader': require.resolve('./my-loader'),
+        },
+    },
+};