test: add edge case coverage for cjs require destructuring tree shaking (#20596)

aryanraj45 · web-flow · commit f5889547eff3 · 2026-03-06T16:09:23.000+03:00
diff --git a/test/cases/cjs-tree-shaking/require-destructuring/index.js b/test/cases/cjs-tree-shaking/require-destructuring/index.js
@@ -10,3 +10,28 @@ it("should support require context destructuring assignment", () => {
 	expect(a).toBe("a/a");
 	expect(usedExports).toEqual(["a", "usedExports"]);
 });
+
+it("should static analyze aliased require destructuring", () => {
+	const { a: renamedA, usedExports } = require("./module");
+	expect(renamedA).toBe("a");
+	expect(usedExports).toEqual(["a", "usedExports"]);
+});
+
+it("should support require context aliased destructuring assignment", () => {
+	const file = "a";
+	const { a: renamedA, usedExports } = require(`./dir/${file}.js`);
+	expect(renamedA).toBe("a/a");
+	expect(usedExports).toEqual(["a", "usedExports"]);
+});
+
+it("should static analyze require destructuring with default values", () => {
+	const { a = "fallback", usedExports } = require("./module");
+	expect(a).toBe("a");
+	expect(usedExports).toEqual(["a", "usedExports"]);
+});
+
+it("should bail on rest element in require destructuring", () => {
+	const { usedExports, ...rest } = require("./module-rest");
+	expect(usedExports).toBe(true);
+	expect(rest).toEqual({ a: "a", b: "b" });
+});
diff --git a/test/cases/cjs-tree-shaking/require-destructuring/module-rest.js b/test/cases/cjs-tree-shaking/require-destructuring/module-rest.js
@@ -0,0 +1,3 @@
+exports.a = "a";
+exports.b = "b";
+exports.usedExports = __webpack_exports_info__.usedExports;

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+exports.a = "a";`
	`2`	`+exports.b = "b";`
	`3`	`+exports.usedExports = __webpack_exports_info__.usedExports;`