Merge pull request #5137 from mzgoddard/concatenated-file-dependencies

Include module file and context dependencies in ConcatenatedModule

lib/optimize/ConcatenatedModule.js

@@ -154,6 +154,8 @@ class ConcatenatedModule extends Module {
 		this.dependencies = [];
 		this.dependenciesWarnings = [];
 		this.dependenciesErrors = [];
+		this.fileDependencies = [];
+		this.contextDependencies = [];
 		this.warnings = [];
 		this.errors = [];
 		this.assets = {};
@@ -165,6 +167,10 @@ class ConcatenatedModule extends Module {
 			m.dependenciesWarnings.forEach(depWarning => this.dependenciesWarnings.push(depWarning));
 			// populate dep errors
 			m.dependenciesErrors.forEach(depError => this.dependenciesErrors.push(depError));
+			// populate file dependencies
+			if(m.fileDependencies) m.fileDependencies.forEach(file => this.fileDependencies.push(file));
+			// populate context dependencies
+			if(m.contextDependencies) m.contextDependencies.forEach(context => this.contextDependencies.push(context));
 			// populate warnings
 			m.warnings.forEach(warning => this.warnings.push(warning));
 			// populate errors

test/watchCases/plugins/module-concatenation-plugin/0/bar/a.js

@@ -0,0 +1 @@
+export default 'This ';

test/watchCases/plugins/module-concatenation-plugin/0/bar/abc.js

@@ -0,0 +1,5 @@
+import a from './a';
+import b from './b';
+import c from './c';
+
+export default a + b + c;

test/watchCases/plugins/module-concatenation-plugin/0/bar/b.js

@@ -0,0 +1 @@
+export default 'is only';

test/watchCases/plugins/module-concatenation-plugin/0/bar/c.js

@@ -0,0 +1 @@
+export default ' a test';

test/watchCases/plugins/module-concatenation-plugin/0/foo/0.js

@@ -0,0 +1 @@
+module.exports = require('../bar/abc').default + '.0';

test/watchCases/plugins/module-concatenation-plugin/0/index.js

@@ -0,0 +1,13 @@
+it("should watch for changes", function() {
+	if(WATCH_STEP === '0') {
+		require("./foo/" + WATCH_STEP).should.be.eql('This is only a test.' + WATCH_STEP);
+	}
+	else if(WATCH_STEP === '1') {
+		require("./foo/" + WATCH_STEP).should.be.eql('This should be a test.' + WATCH_STEP);
+	}
+	else if(WATCH_STEP === '2') {
+		require("./foo/" + WATCH_STEP).should.be.eql('This should be working.' + WATCH_STEP);
+	}
+
+	STATS_JSON.modules.length.should.equal(4 + Number(WATCH_STEP));
+});

test/watchCases/plugins/module-concatenation-plugin/1/bar/b.js

@@ -0,0 +1 @@
+export default 'should be';

test/watchCases/plugins/module-concatenation-plugin/1/foo/1.js

@@ -0,0 +1 @@
+module.exports = require('../bar/abc').default + '.1';

test/watchCases/plugins/module-concatenation-plugin/2/bar/c.js

@@ -0,0 +1 @@
+export default ' working';

test/watchCases/plugins/module-concatenation-plugin/2/foo/2.js

@@ -0,0 +1 @@
+module.exports = require('../bar/abc').default + '.2';

test/watchCases/plugins/module-concatenation-plugin/webpack.config.js

@@ -0,0 +1,6 @@
+var webpack = require("../../../../");
+module.exports = {
+	plugins: [
+		new webpack.optimize.ModuleConcatenationPlugin()
+	]
+};