fix: handle undefined and empty configuration export (#2930)

Author: rishabh3112

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

@@ -67,11 +67,11 @@ class WebpackCLI {
     }
 
     // For babel/typescript
-    if (result.default) {
-      result = result.default;
+    if (result && typeof result === "object" && "default" in result) {
+      result = result.default || {};
     }
 
-    return result;
+    return result || {};
   }
 
   loadJSONFile(pathToFile, handleError = true) {

test/build/config/no-code/no-code.test.js

@@ -0,0 +1,16 @@
+"use strict";
+const { resolve } = require("path");
+const { run } = require("../../../utils/test-utils");
+
+describe("config flag with no code", () => {
+  it("should not throw error with no configuration or index file", async () => {
+    const { exitCode, stderr, stdout } = await run(__dirname, [
+      "-c",
+      resolve(__dirname, "webpack.config.js"),
+    ]);
+
+    expect(exitCode).toBe(0);
+    expect(stderr).toBeFalsy();
+    expect(stdout).toBeTruthy();
+  });
+});

test/build/config/no-code/src/index.js

@@ -0,0 +1 @@
+console.log("Peeves");

test/build/config/no-code/webpack.config.js

@@ -0,0 +1 @@
+console.log("Tom Riddle");

test/build/config/undefined-default/src/index.js

@@ -0,0 +1,16 @@
+"use strict";
+const { resolve } = require("path");
+const { run } = require("../../../utils/test-utils");
+
+describe("config flag with undefined default export config file", () => {
+  it("should not throw error with no configuration or index file", async () => {
+    const { exitCode, stderr, stdout } = await run(__dirname, [
+      "-c",
+      resolve(__dirname, "webpack.config.js"),
+    ]);
+
+    expect(exitCode).toBe(0);
+    expect(stderr).toBeFalsy();
+    expect(stdout).toBeTruthy();
+  });
+});

test/build/config/undefined-default/undefined-default.test.js

@@ -0,0 +1 @@
+module.exports.default = undefined;

test/build/config/undefined-default/webpack.config.js

@@ -0,0 +1 @@
+console.log("Percy Weasley");

test/build/config/undefined/src/index.js

@@ -0,0 +1,16 @@
+"use strict";
+const { resolve } = require("path");
+const { run } = require("../../../utils/test-utils");
+
+describe("config flag with undefined export config file", () => {
+  it("should not throw error with no configuration or index file", async () => {
+    const { exitCode, stderr, stdout } = await run(__dirname, [
+      "-c",
+      resolve(__dirname, "webpack.config.js"),
+    ]);
+
+    expect(exitCode).toBe(0);
+    expect(stderr).toBeFalsy();
+    expect(stdout).toBeTruthy();
+  });
+});

test/build/config/undefined/undefined.test.js

@@ -0,0 +1 @@
+module.exports = undefined;