Properly render dynamic imports when imported module is empty (#2714)

src/Chunk.ts

@@ -382,7 +382,7 @@ export default class Chunk {
 			for (const { node, resolution } of module.dynamicImports) {
 				if (!resolution) continue;
 				if (resolution instanceof Module) {
-					if (resolution.isIncluded() && resolution.chunk !== this) {
+					if (!resolution.chunk.isEmpty && resolution.chunk !== this) {
 						const resolutionChunk = resolution.facadeChunk || resolution.chunk;
 						let relPath = normalize(relative(dirname(this.id), resolutionChunk.id));
 						if (!relPath.startsWith('../')) relPath = './' + relPath;

test/chunking-form/samples/dynamic-import-only-reexports/_config.js

@@ -0,0 +1,6 @@
+module.exports = {
+	description: 'handles dynamic imports when the imported module only reexports from other modules',
+	options: {
+		input: ['main']
+	}
+};

test/chunking-form/samples/dynamic-import-only-reexports/_expected/amd/generated-chunk.js

@@ -0,0 +1,7 @@
+define(['exports'], function (exports) { 'use strict';
+
+	const value = 42;
+
+	exports.value = value;
+
+});

test/chunking-form/samples/dynamic-import-only-reexports/_expected/amd/main.js

@@ -0,0 +1,5 @@
+define(['require'], function (require) { 'use strict';
+
+	new Promise(function (resolve, reject) { require(['./generated-chunk.js'], resolve, reject) }).then(({ value }) => console.log(value));
+
+});

test/chunking-form/samples/dynamic-import-only-reexports/_expected/cjs/generated-chunk.js

@@ -0,0 +1,5 @@
+'use strict';
+
+const value = 42;
+
+exports.value = value;

test/chunking-form/samples/dynamic-import-only-reexports/_expected/cjs/main.js

@@ -0,0 +1,3 @@
+'use strict';
+
+Promise.resolve(require('./generated-chunk.js')).then(({ value }) => console.log(value));

test/chunking-form/samples/dynamic-import-only-reexports/_expected/es/generated-chunk.js

@@ -0,0 +1,3 @@
+const value = 42;
+
+export { value };

test/chunking-form/samples/dynamic-import-only-reexports/_expected/es/main.js

@@ -0,0 +1 @@
+import('./generated-chunk.js').then(({ value }) => console.log(value));

test/chunking-form/samples/dynamic-import-only-reexports/_expected/system/generated-chunk.js

@@ -0,0 +1,10 @@
+System.register([], function (exports, module) {
+	'use strict';
+	return {
+		execute: function () {
+
+			const value = exports('value', 42);
+
+		}
+	};
+});

test/chunking-form/samples/dynamic-import-only-reexports/_expected/system/main.js

@@ -0,0 +1,10 @@
+System.register([], function (exports, module) {
+	'use strict';
+	return {
+		execute: function () {
+
+			module.import('./generated-chunk.js').then(({ value }) => console.log(value));
+
+		}
+	};
+});

test/chunking-form/samples/dynamic-import-only-reexports/dep.js

@@ -0,0 +1 @@
+export const value = 42;

test/chunking-form/samples/dynamic-import-only-reexports/dynamic.js

@@ -0,0 +1 @@
+export {value} from './dep';

test/chunking-form/samples/dynamic-import-only-reexports/main.js

@@ -0,0 +1 @@
+import('./dynamic').then(({ value }) => console.log(value));