Make sure tree-shaken dynamic imports do not lead to the creation of

namespace objects when inlined

src/Chunk.ts

@@ -248,10 +248,12 @@ export default class Chunk {
 			const declaration = module.imports[importName];
 			this.traceAndInitializeImport(declaration.name, declaration.module);
 		}
-		for (const { resolution } of module.dynamicImports) {
-			this.hasDynamicImport = true;
-			if (resolution instanceof Module && resolution.chunk === this)
-				resolution.getOrCreateNamespace().include();
+		for (const { node, resolution } of module.dynamicImports) {
+			if (node.included) {
+				this.hasDynamicImport = true;
+				if (resolution instanceof Module && resolution.chunk === this)
+					resolution.getOrCreateNamespace().include();
+			}
 		}
 	}
 

test/chunking-form/samples/dynamic-import-tree-shaking-1/_expected/amd/entryB.js

@@ -1,4 +1,4 @@
-define(['require'], function (require) { 'use strict';
+define(function () { 'use strict';
 
 	console.log('main2');
 

test/chunking-form/samples/dynamic-import-tree-shaking-2/_expected/amd/main1.js

@@ -1,4 +1,4 @@
-define(['require'], function (require) { 'use strict';
+define(function () { 'use strict';
 
 	console.log('main1');
 

test/form/samples/default-identifier-deshadowing/dep.js

@@ -1,4 +1,4 @@
 export default function a() {
   a = someGlobal;
   return a();
-}
+}

test/form/samples/inlined-treeshaken-dynamic-import/_config.js

@@ -0,0 +1,6 @@
+module.exports = {
+	description: 'completely removes tree-shaken dynamic imports ',
+	options: {
+		inlineDynamicImports: true
+	}
+};

test/form/samples/inlined-treeshaken-dynamic-import/_expected/amd.js

@@ -0,0 +1,5 @@
+define(function () { 'use strict';
+
+	console.log('main');
+
+});

test/form/samples/inlined-treeshaken-dynamic-import/_expected/cjs.js

@@ -0,0 +1,3 @@
+'use strict';
+
+console.log('main');

test/form/samples/inlined-treeshaken-dynamic-import/_expected/es.js

@@ -0,0 +1 @@
+console.log('main');

test/form/samples/inlined-treeshaken-dynamic-import/_expected/iife.js

@@ -0,0 +1,6 @@
+(function () {
+	'use strict';
+
+	console.log('main');
+
+}());

test/form/samples/inlined-treeshaken-dynamic-import/_expected/system.js

@@ -0,0 +1,10 @@
+System.register([], function (exports, module) {
+	'use strict';
+	return {
+		execute: function () {
+
+			console.log('main');
+
+		}
+	};
+});

test/form/samples/inlined-treeshaken-dynamic-import/_expected/umd.js

@@ -0,0 +1,9 @@
+(function (global, factory) {
+	typeof exports === 'object' && typeof module !== 'undefined' ? factory() :
+	typeof define === 'function' && define.amd ? define(factory) :
+	(factory());
+}(this, (function () { 'use strict';
+
+	console.log('main');
+
+})));

test/form/samples/inlined-treeshaken-dynamic-import/dynamic.js

@@ -0,0 +1,2 @@
+console.log('dynamic');
+export var dynamic = 42;

test/form/samples/inlined-treeshaken-dynamic-import/main.js

@@ -0,0 +1,2 @@
+console.log('main');
+const getFoo = () => import('./dynamic.js');