Merge pull request webpack#5160 from asmundg/fix-amd-deprecation-warning

sokra · web-flow · commit 53944e8c144f · 2017-07-01T16:54:35.000+02:00
Replace deprecated Chunk.modules usage
diff --git a/lib/AmdMainTemplatePlugin.js b/lib/AmdMainTemplatePlugin.js
@@ -17,7 +17,7 @@ class AmdMainTemplatePlugin {
 		const mainTemplate = compilation.mainTemplate;
 
 		compilation.templatesPlugin("render-with-entry", (source, chunk, hash) => {
-			const externals = chunk.modules.filter((m) => m.external);
+			const externals = chunk.getModules().filter((m) => m.external);
 			const externalsDepsArray = JSON.stringify(externals.map((m) =>
 				typeof m.request === "object" ? m.request.amd : m.request
 			));
diff --git a/test/AmdMainTemplatePlugin.test.js b/test/AmdMainTemplatePlugin.test.js
@@ -67,7 +67,7 @@ describe("AmdMainTemplatePlugin", () => {
 				describe("with name", () => {
 					beforeEach(() => {
 						env.chunk = {
-							modules: env.modulesListWithExternals
+							getModules: () => env.modulesListWithExternals
 						};
 						env.eventBinding = setupPluginAndGetEventBinding("foo");
 					});
@@ -82,7 +82,7 @@ describe("AmdMainTemplatePlugin", () => {
 				describe("with external dependencies", () => {
 					beforeEach(() => {
 						env.chunk = {
-							modules: env.modulesListWithExternals
+							getModules: () => env.modulesListWithExternals
 						};
 						env.eventBinding = setupPluginAndGetEventBinding();
 					});
@@ -101,7 +101,7 @@ describe("AmdMainTemplatePlugin", () => {
 						};
 						const noExternals = env.modulesListWithExternals.map((module) => Object.assign(module, externalFlag));
 						env.chunk = {
-							modules: noExternals
+							getModules: () => env.modulesListWithExternals
 						};
 						env.eventBinding = setupPluginAndGetEventBinding();
 					});
