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

Side by Side Diff: LayoutTests/http/tests/serviceworker/resources/worker-test-harness.js

Issue 430993002: [ServiceWorker] CacheStorage tests. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 3 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « LayoutTests/http/tests/serviceworker/resources/test-helpers.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * worker-test-harness should be considered a temporary polyfill around 2 * worker-test-harness should be considered a temporary polyfill around
3 * testharness.js for supporting Service Worker based tests. It should not be 3 * testharness.js for supporting Service Worker based tests. It should not be
4 * necessary once the test harness is able to drive worker based tests natively. 4 * necessary once the test harness is able to drive worker based tests natively.
5 * See https://github.com/w3c/testharness.js/pull/82 for status of effort to 5 * See https://github.com/w3c/testharness.js/pull/82 for status of effort to
6 * update upstream testharness.js. Once the upstreaming is complete, tests that 6 * update upstream testharness.js. Once the upstreaming is complete, tests that
7 * reference worker-test-harness should be updated to directly import 7 * reference worker-test-harness should be updated to directly import
8 * testharness.js. 8 * testharness.js.
9 */ 9 */
10 10
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 // boilerplate for testing with promises. 77 // boilerplate for testing with promises.
78 function promise_test(func, name, properties) { 78 function promise_test(func, name, properties) {
79 properties = properties || {}; 79 properties = properties || {};
80 var test = async_test(name, properties); 80 var test = async_test(name, properties);
81 Promise.resolve(test.step(func, test, test)) 81 Promise.resolve(test.step(func, test, test))
82 .then(function() { test.done(); }) 82 .then(function() { test.done(); })
83 .catch(test.step_func(function(value) { 83 .catch(test.step_func(function(value) {
84 throw value; 84 throw value;
85 })); 85 }));
86 } 86 }
87
88 // Returns a promise that fulfills after the provided |promise| is fulfilled.
89 // The |test| succeeds only if |promise| throws an exception matching |code|.
90 // The optional |description| describes the test being performed.
91 // E.g.:
92 // assert_promise_throws(
93 // new Promise(...), // something that should throw an exception.
94 // 'NotFoundError',
95 // 'Should throw NotFoundError.');
96 function assert_promise_throws(promise, code, description) {
jsbell 2014/09/11 22:27:30 Can we rename this to assert_promise_rejects since
asanka 2014/09/24 03:28:55 Done.
97 return promise.then(
98 function() {
99 throw 'assert_promise_throws: ' + description + ' Promise did not throw.';
100 },
101 function(e) {
102 assert_throws(code, function() { throw e; }, description);
103 });
104 }
OLDNEW
« no previous file with comments | « LayoutTests/http/tests/serviceworker/resources/test-helpers.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698