LayoutTests/http/tests/push_messaging/register-document.html - Issue 783983003: Push API: move PushManager from Navigator to ServiceWorkerRegistration [switchover 6/6]

Unified Diff: LayoutTests/http/tests/push_messaging/register-document.html

Issue 783983003: Push API: move PushManager from Navigator to ServiceWorkerRegistration [switchover 6/6] (Closed) Base URL: svn://svn.chromium.org/blink/trunk

Patch Set: Created 6 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

« LayoutTests/http/tests/push_messaging/has-permission-document.html ('K') | « LayoutTests/http/tests/push_messaging/has-permission-document.html ('k') | LayoutTests/http/tests/push_messaging/resources/empty-worker.js » ('j') | LayoutTests/http/tests/push_messaging/resources/push_manifest.json » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/http/tests/push_messaging/register-document.html

diff --git a/LayoutTests/http/tests/push_messaging/register-document.html b/LayoutTests/http/tests/push_messaging/register-document.html

new file mode 100644

index 0000000000000000000000000000000000000000..e6cd1c4e90c9ab9879cc9a88b5356cf09f6e8fe8

--- /dev/null

+++ b/LayoutTests/http/tests/push_messaging/register-document.html

@@ -0,0 +1,42 @@

+<!DOCTYPE html>

+<html>

+<head>

+<title>Push API: register in document context</title>

+<link rel="manifest" href="resources/push_manifest.json">

+<script src="../resources/testharness.js"></script>

+<script src="../resources/testharnessreport.js"></script>

+<script src="../serviceworker/resources/test-helpers.js"></script>

+</head>

+<body>

+<script>

+async_test(function(test) {

Peter Beverloo 2014/12/08 21:40:50 This is a bit of a weird test. Also, "register in

Michael van Ouwerkerk 2014/12/09 18:47:45 It was mixed up because multiple async tests on th

It was mixed up because multiple async tests on the same page don't work correctly with service workers and permissions management. I've split the following into separate test pages now: * register success * register failure due to permission denied * register failure due to worker not being active yet The relevant API surfaces are verified in there as well, doing that separately would duplicate an awful lot of code. It does seem fine to wait with (4) until unregister() and getRegistration() have landed, so we can avoid unnecessary TestRunner methods.

+ assert_true(!!window.testRunner, 'This test requires the TestRunner.');

+ var url = 'resources/empty-worker.js';

+ var scope = 'resources/scope/register';

+ var origin = 'http://127.0.0.1:8000';

+ var swRegistration;

+ service_worker_unregister_and_register(test, url, scope)

+ .then(function(serviceWorkerRegistration) {

+ swRegistration = serviceWorkerRegistration;

+ testRunner.setPushMessagingPermission(origin, true);

+ return swRegistration.pushManager.register();

+ })

+ .then(function(pushRegistration) {

+ assert_equals(pushRegistration.pushEndpoint, 'https://example.com/LayoutTestEndpoint');

+ assert_equals(pushRegistration.pushRegistrationId, 'layoutTestRegistrationId_0');

+ testRunner.resetPushMessagingRegistrations();

+ testRunner.setPushMessagingPermission(origin, false);

+ return swRegistration.pushManager.register();

+ })

+ .then(function(pushRegistration) {

+ assert_unreached('Registration must not succeed after permission is denied.');

+ }, function(e) {

+ assert_equals(e.name, 'AbortError');

+ assert_equals(e.message, 'Registration failed - permission denied');

+ test.done();

+ })

+ .catch(unreached_rejection(test));

+}, 'Push API: register in document context');

+</script>

+</body>

+</html>