Merge pull request #7059 from bastimeyer/bugfix/loaderplugin-recursive

Fix LoaderPlugin not resolving recursively

lib/dependencies/LoaderPlugin.js

@@ -47,7 +47,7 @@ class LoaderPlugin {
 							],
 							true,
 							"lm",
-							false,
+							true,
 							err => {
 								compilation.semaphore.acquire(() => {
 									if (err) return callback(err);

test/cases/loaders/issue-4959/a.js

@@ -0,0 +1 @@
+module.exports = "a";

test/cases/loaders/issue-4959/b.js

@@ -0,0 +1 @@
+module.exports = require("c");

test/cases/loaders/issue-4959/c.js

@@ -0,0 +1 @@
+module.exports = "c";

test/cases/loaders/issue-4959/index.js

@@ -0,0 +1,3 @@
+it("should resolve module dependencies recursively", function() {
+	require("!./loaders/index!a").should.be.eql("c");
+});

test/cases/loaders/issue-4959/loaders/index.js

@@ -0,0 +1,9 @@
+module.exports = function() {
+	var callback = this.async();
+	this.loadModule("b", function(error) {
+		if (error) {
+			return callback(error);
+		}
+		callback(null, "module.exports = require('b');");
+	});
+};

test/cases/loaders/issue-4959/package.json

@@ -0,0 +1,7 @@
+{
+	"browser": {
+		"a": "./a.js",
+		"b": "./b.js",
+		"c": "./c.js"
+	}
+}