Expose requireAsync method and make it more robust

extensions/test/data/mojo_private_unittest.js

Index: extensions/test/data/mojo_private_unittest.js
diff --git a/extensions/test/data/mojo_private_unittest.js b/extensions/test/data/mojo_private_unittest.js
index 2a90d6509240d2cda449aa21a6633784efe43a0b..9584e840710bf43e4925a7120cd1f7077603c41f 100644
--- a/extensions/test/data/mojo_private_unittest.js
+++ b/extensions/test/data/mojo_private_unittest.js
@@ -31,4 +31,25 @@ unittestBindings.exportTests([
     mojoPrivate.define('testModule', ['does not exist!'], test.fail);
     test.succeed();
   },
+
+  function testRequireAsync() {
+    console.log('requireAsync');

[mark a. foltz, 2015/01/27 21:58:53]

Is this necessary?

+    mojoPrivate.requireAsync('mojo/public/js/codec').then(
+        test.callbackPass(function(codec) {
+          test.assertEq('function', typeof codec.Message);
+        }));
+  },
+
+  function testDefineAndRequire() {
+    mojoPrivate.define('testModule', ['dependency'],
+                       test.callbackPass(function(module) {
+      test.assertEq(12345, module.result);
+    }));
+    mojoPrivate.define('dependency', test.callbackPass(function() {
+      return {result: 12345};
+    }));
+    mojoPrivate.requireAsync('dependency').then(
+        test.callbackPass(),
+        test.fail);
+  }
 ], test.runTests, exports);