Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(83)

Unified Diff: extensions/test/data/mojo_private_unittest.js

Issue 862793003: Expose requireAsync method and make it more robust (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Unit tests. Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..1c9d40f227ad07dcc4e56ac120def7ecfd35699d 100644
--- a/extensions/test/data/mojo_private_unittest.js
+++ b/extensions/test/data/mojo_private_unittest.js
@@ -31,4 +31,35 @@ unittestBindings.exportTests([
mojoPrivate.define('testModule', ['does not exist!'], test.fail);
test.succeed();
},
+
+ function testRequireAsync() {
+ console.log('requireAsync');
+ mojoPrivate.requireAsync('mojo/public/js/codec').then(
+ test.callbackPass(function(codec) {
+ test.assertEq('function', typeof codec.Message);
+ }));
+ },
+
+ function testRequireAsyncModuleDoesNotExist() {
+ console.log('requireAsync');
+ mojoPrivate.requireAsync('testModule').then(
+ test.fail,
+ test.succeed);
+ },
+
+ function testDefineAndRequire() {
+ mojoPrivate.define('testModule', ['dependency'],
+ test.callbackPass(function(module) {
+ test.assertEq(12345, module.result);
+ mojoPrivate.requireAsync('dependency').then(
+ test.succeed,
+ test.fail);
+ mojoPrivate.requireAsync('NOT_A_MODULE').then(
+ test.fail,
+ test.succeed);
+ }));
+ mojoPrivate.define('dependency', test.callbackPass(function() {
+ return {result: 12345};
+ }));
+ }
], test.runTests, exports);

Powered by Google App Engine
This is Rietveld 408576698