Merge pull request #159 from chuckdumont/work2

Fix issue with calling runtime require sans callback function

Author: Chuck Dumont

lib/DojoAMDMainTemplate.runtime.js

@@ -132,7 +132,7 @@ module.exports = {
 						errors.push({mid: mid, error: e});
 					}
 				});
-				if (errors.length === 0) {
+				if (callback && errors.length === 0) {
 					callback.apply(this, modules);
 				} else {
 					var error = new Error("findModules");

test/TestCases/dependencies/simple/asyncDep3.js

@@ -0,0 +1,3 @@
+define([], function() {
+	global.asyncDep3Promise.resolve();
+});

test/TestCases/dependencies/simple/index.js

@@ -42,6 +42,24 @@ define(["exports", "module", "./dep"], function(exports, module, dep) {
 		}
 	});
 
+	it("require sans callback", function(done) {
+		var resolver;
+		global.asyncDep3Promise = new Promise((resolve) => {
+			resolver = resolve;
+		});
+		global.asyncDep3Promise.resolve = resolver;
+		require(["asyncDep3"]);
+		global.asyncDep3Promise.then(() => {
+			try {
+				var dep3 = ['asyncDep3'];
+				require(dep3);
+			} catch (err) {
+				return done(err);
+			}
+			done();
+		});
+	});
+
 	it("runtime require failures", function(done) {
 		var notTrue;
 		if (notTrue) {