feat: configtest

.eslintignore

@@ -10,3 +10,4 @@ test/typescript/webpack.config.ts
 test/config/error-commonjs/syntax-error.js
 test/config/error-mjs/syntax-error.mjs
 test/config/error-array/webpack.config.js
+test/configtest/syntax-error.config.js

.prettierignore

@@ -9,3 +9,4 @@ test/config/error-mjs/syntax-error.mjs
 packages/webpack-cli/__tests__/test-assets/.yo-rc.json
 test/build-errors/stats.json
 packages/**/lib
+test/configtest/syntax-error.config.js

packages/configtest/src/index.ts

@@ -1,4 +1,4 @@
-import { validate } from 'webpack';
+import webpack from 'webpack';
 
 class ConfigTestCommand {
     async apply(cli): Promise<void> {
@@ -16,15 +16,23 @@ class ConfigTestCommand {
             async (configPath: string) => {
                 const { options } = await cli.resolveConfig({ config: [configPath] });
 
+                const isValidationError = (error) => {
+                    // https://github.com/webpack/webpack/blob/master/lib/index.js#L267
+                    // https://github.com/webpack/webpack/blob/v4.44.2/lib/webpack.js#L90
+                    const ValidationError: any = webpack.ValidationError || webpack.WebpackOptionsValidationError;
+
+                    return error instanceof ValidationError;
+                };
+
                 //eslint-disable-next-line @typescript-eslint/no-explicit-any
-                const validationErrors: any = validate(options);
+                const error: any = webpack.validate(options);
 
-                if (validationErrors) {
-                    logger.error(validationErrors);
+                if (error) {
+                    logger.error(isValidationError(error) ? error.message : error);
                     process.exit(2);
                 }
 
-                logger.success('There are no validation errors in the given webpack configuration.');
+                logger.success('There are no errors in the given webpack configuration.');
             },
         );
     }

test/configtest/configtest.test.js

@@ -8,7 +8,7 @@ describe('basic info usage', () => {
 
         expect(exitCode).toBe(0);
         expect(stderr).toBeFalsy();
-        expect(stdout).toContain('There are no validation errors in the given webpack configuration.');
+        expect(stdout).toContain('There are no errors in the given webpack configuration.');
     });
 
     it('should throw validation error', () => {
@@ -20,6 +20,14 @@ describe('basic info usage', () => {
         expect(stdout).toBeFalsy();
     });
 
+    it('should throw syntax error', () => {
+        const { exitCode, stderr, stdout } = run(__dirname, ['configtest', './syntax-error.config.js'], false);
+
+        expect(exitCode).toBe(2);
+        expect(stderr).toContain(`SyntaxError: Unexpected token ';'`);
+        expect(stdout).toBeFalsy();
+    });
+
     it(`should validate the config with alias 't'`, () => {
         const { exitCode, stderr, stdout } = run(__dirname, ['t', './error.config.js'], false);
 

test/configtest/syntax-error.config.js

@@ -0,0 +1,5 @@
+module.exports = {
+    name: 'config-error',
+    mode: 'development',
+    target: 'node'; //SyntaxError: Unexpected token ';'
+};