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

Unified Diff: third_party/WebKit/LayoutTests/http/tests/budget/get-cost-in-service-worker.html

Issue 2600733002: Convert budget LayoutTests to use promise_test. (Closed)
Patch Set: Created 4 years 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: third_party/WebKit/LayoutTests/http/tests/budget/get-cost-in-service-worker.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/budget/get-cost-in-service-worker.html b/third_party/WebKit/LayoutTests/http/tests/budget/get-cost-in-service-worker.html
index 2dbdfbc894b4822bad677e6dc08e194677686aea..46afb6f4b342542cd336fc14705f76aaeac3b6b3 100644
--- a/third_party/WebKit/LayoutTests/http/tests/budget/get-cost-in-service-worker.html
+++ b/third_party/WebKit/LayoutTests/http/tests/budget/get-cost-in-service-worker.html
@@ -11,17 +11,16 @@
</head>
<body>
<script>
- async_test(function(test) {
- var script = 'resources/instrumentation-service-worker.js';
- var scope = 'resources/scope/' + location.pathname;
- var port;
+ promise_test(function(test) {
+ const script = 'resources/instrumentation-service-worker.js';
+ const scope = 'resources/scope/' + location.pathname;
+ let port;
- budgetServiceMock.then(mock => {
+ return budgetServiceMock.then(mock => {
mock.setCost("silent-push", TEST_BUDGET_COST);
- getActiveServiceWorkerWithMessagePort(test, script, scope)
+ return getActiveServiceWorkerWithMessagePort(test, script, scope)
.then(function(workerInfo) {
port = workerInfo.port;
- port.postMessage({ command: 'getCost' });
port.addEventListener('message', function(event) {
if (typeof event.data != 'object' || !event.data.command)
@@ -31,8 +30,9 @@
assert_true(event.data.success,
'getCost should succeed. Error message: ' + event.data.errorMessage);
assert_equals(event.data.cost, TEST_BUDGET_COST);
- test.done();
});
+
+ return sendCommand(port, { command: 'getCost' });
})
.catch(unreached_rejection(test));
});

Powered by Google App Engine
This is Rietveld 408576698