Add failing test case

evanw · Aug 20, 2020 · 1bf97e6 · 1bf97e6
1 parent 9f93961
commit 1bf97e6
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 0 deletions.
diff --git a/internal/bundler/bundler_default_test.go b/internal/bundler/bundler_default_test.go
@@ -79,6 +79,27 @@ func TestNestedCommonJS(t *testing.T) {
 	})
 }
 
+// This test makes sure that NewExpressions containing require() calls aren't
+// broken.
+func TestNewExpressionCommonJS(t *testing.T) {
+	default_suite.expectBundled(t, bundled{
+		files: map[string]string{
+			"/entry.js": `
+        new (require("./foo.js")).Foo();
+			`,
+			"/foo.js": `
+        class Foo {}
+				module.exports = {Foo};
+			`,
+		},
+		entryPaths: []string{"/entry.js"},
+		options: config.Options{
+			IsBundling:    true,
+			AbsOutputFile: "/out.js",
+		},
+	})
+}
+
 func TestCommonJSFromES6(t *testing.T) {
 	default_suite.expectBundled(t, bundled{
 		files: map[string]string{

diff --git a/internal/bundler/snapshots/snapshots_default.txt b/internal/bundler/snapshots/snapshots_default.txt
@@ -1203,6 +1203,19 @@ TestNestedScopeBug
   a();
 })();
 
+================================================================================
+TestNewExpressionCommonJS
+---------- /out.js ----------
+// /foo.js
+var require_foo = __commonJS((exports, module) => {
+  class Foo {
+  }
+  module.exports = {Foo};
+});
+
+// /entry.js
+new (require_foo()).Foo();
+
 ================================================================================
 TestNodeModules
 ---------- /Users/user/project/out.js ----------