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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/redirected-response.https.html

Issue 2878003003: Upstream srvc wrkr "redirected resp" test to WPT (Closed)
Patch Set: Upstream service worker "request" test to WPT Created 3 years, 7 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/serviceworker/redirected-response.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/redirected-response.https.html
diff --git a/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/redirected-response.https.html b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/redirected-response.https.html
new file mode 100644
index 0000000000000000000000000000000000000000..c20f819c17af97c45551b494827e8e11b586df11
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/redirected-response.https.html
@@ -0,0 +1,201 @@
+<!DOCTYPE html>
+<title>Service Worker: Redirected response</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/common/get-host-info.sub.js"></script>
+<script src="resources/test-helpers.sub.js"></script>
+<script>
falken 2017/05/17 05:09:12 I find this function pretty dense. Could we add so
mike3 2017/05/17 21:19:27 Acknowledged.
+function redirected_test(url,
+ fetch_option,
+ fetch_method,
+ cache,
+ expected_redirected,
+ expected_url_list) {
falken 2017/05/17 05:09:12 nit: Could we make this a dictionary of params ins
mike3 2017/05/17 21:19:27 Done.
+ return fetch_method(url, fetch_option).then(response => {
+ var cloned_response = response.clone();
+ assert_equals(
+ response.redirected, expected_redirected,
+ 'The redirected flag of response must match. URL: ' + url);
+ assert_equals(
+ cloned_response.redirected, expected_redirected,
+ 'The redirected flag of cloned response must match. URL: ' + url);
+ if (self.internals) {
falken 2017/05/17 05:09:12 We shouldn't put this part in the WPT. self.intern
mike3 2017/05/17 21:19:28 Acknowledged.
+ assert_array_equals(
+ self.internals.getInternalResponseURLList(response),
+ expected_url_list,
+ 'The URL list of response must match. URL: ' + url);
+ assert_array_equals(
+ self.internals.getInternalResponseURLList(cloned_response),
+ expected_url_list,
+ 'The URL list of cloned response must match. URL: ' + url);
+ }
+ return cache.put(url, response);
+ })
+ .then(_ => cache.match(url))
+ .then(response => {
+ assert_equals(response.redirected, expected_redirected,
+ 'The redirected flag of response in CacheStorage must match. URL: ' +
+ url);
+ if (self.internals) {
falken 2017/05/17 05:09:12 ditto
mike3 2017/05/17 21:19:27 Acknowledged.
+ assert_array_equals(
+ self.internals.getInternalResponseURLList(response),
+ expected_url_list,
+ 'The URL list of response in CacheStorage must match. URL: ' +
+ url);
+ }
+ });
+}
+
+var host_info = get_host_info();
+var REDIRECT_URL = host_info['HTTPS_ORIGIN'] +
+ '/service-workers/service-worker/resources/redirect.py?Redirect=';
+var TARGET_URL = host_info['HTTPS_ORIGIN'] +
+ '/service-workers/service-worker/resources/simple.txt';
+var REDIRECT_TO_TARGET_URL = REDIRECT_URL + encodeURIComponent(TARGET_URL);
+var frame;
+var cache;
+var setup;
+
+promise_test(t => {
+ var SCOPE = 'resources/blank.html?redirected-response';
+ var SCRIPT = 'resources/fetch-rewrite-worker.js';
+ var CACHE_NAME = 'service-workers/service-worker/redirected-response';
+ setup = service_worker_unregister_and_register(t, SCRIPT, SCOPE)
+ .then(registration => {
+ promise_test(function() {
+ return registration.unregister();
+ }, 'restore global state (service worker registration)');
+
+ return wait_for_state(t, registration.installing, 'activated');
+ })
+ .then(_ => self.caches.open(CACHE_NAME))
+ .then(c => {
+ cache = c;
+
+ promise_test(function() {
+ return self.caches.delete(CACHE_NAME);
+ }, 'restore global state (caches)');
+
+ return with_iframe(SCOPE);
+ })
+ .then(f => {
+ frame = f;
+
+ t.add_cleanup(function() {
mike3 2017/05/17 21:19:28 This change was a mistake: the `t` here is the "se
+ f.remove();
+ });
+ });
+ return setup;
+ }, 'initialize global state (service worker registration and caches)');
+
+// Tests without service workers.
+promise_test(t => setup
+ .then(() => redirected_test(TARGET_URL, {}, self.fetch, cache,
+ false /* expected_redirected */,
+ [TARGET_URL])),
+ 'mode: "follow", non-intercepted request, no server redirect');
+
+promise_test(t => setup
+ .then(() => redirected_test(REDIRECT_TO_TARGET_URL, {}, self.fetch, cache,
+ true /* expected_redirected */,
+ [REDIRECT_TO_TARGET_URL, TARGET_URL])),
+ 'mode: "follow", non-intercepted request');
+
+promise_test(t => setup
+ .then(() => redirected_test(REDIRECT_TO_TARGET_URL + '&manual',
+ {redirect: 'manual'}, self.fetch, cache,
+ false /* expected_redirected */,
+ [REDIRECT_TO_TARGET_URL + '&manual'])),
+ 'mode: "manual", non-intercepted request');
+
+promise_test(t => setup
+ .then(() => promise_rejects(
+ t, new TypeError(),
+ self.fetch(REDIRECT_TO_TARGET_URL + '&error',
+ {redirect:'error'}),
+ 'The redirect response from the server should be treated as' +
+ ' an error when the redirect flag of request was \'error\'.')),
+ 'mode: "error", non-intercepted request');
+
+promise_test(t => setup
+ .then(() => redirected_test('./?url=' + encodeURIComponent(TARGET_URL),
+ {},
+ frame.contentWindow.fetch,
+ cache,
+ false /* expected_redirected */,
+ [TARGET_URL])),
+ 'mode: "follow", no mode change, no server redirect');
+
+
+ // The service worker returns a redirected response.
+promise_test(t => setup
+ .then(() => redirected_test(
+ './?url=' + encodeURIComponent(REDIRECT_TO_TARGET_URL) +
+ '&original-redirect-mode=follow',
+ {redirect: 'follow'},
+ frame.contentWindow.fetch,
+ cache,
+ true /* expected_redirected */,
+ [REDIRECT_TO_TARGET_URL, TARGET_URL])),
+ 'mode: "follow", no mode change');
+
+promise_test(t => setup
+ .then(() => promise_rejects(
+ t, new TypeError(),
+ frame.contentWindow.fetch(
+ './?url=' + encodeURIComponent(REDIRECT_TO_TARGET_URL) +
+ '&original-redirect-mode=error&redirect-mode=follow',
+ {redirect: 'error'}),
+ 'The redirected response from the service worker should be ' +
+ 'treated as an error when the redirect flag of request was ' +
+ '\'error\'.')),
+ 'mode: "error", mode change: "follow"');
+
+promise_test(t => setup
+ .then(() => promise_rejects(
+ t, new TypeError(),
+ frame.contentWindow.fetch(
+ './?url=' + encodeURIComponent(REDIRECT_TO_TARGET_URL) +
+ '&original-redirect-mode=manual&redirect-mode=follow',
+ {redirect: 'manual'}),
+ 'The redirected response from the service worker should be ' +
+ 'treated as an error when the redirect flag of request was ' +
+ '\'manual\'.')),
+ 'mode: "manual", mode change: "follow"');
+
+// The service worker returns an opaqueredirect response.
+promise_test(t => setup
+ .then(() => promise_rejects(
+ t, new TypeError(),
+ frame.contentWindow.fetch(
+ './?url=' + encodeURIComponent(REDIRECT_TO_TARGET_URL) +
+ '&original-redirect-mode=follow&redirect-mode=manual',
+ {redirect: 'follow'}),
+ 'The opaqueredirect response from the service worker should ' +
+ 'be treated as an error when the redirect flag of request was' +
+ ' \'follow\'.')),
+ 'mode: "follow", mode change: "redirect"');
+
+promise_test(t => setup
+ .then(() => promise_rejects(
+ t, new TypeError(),
+ frame.contentWindow.fetch(
+ './?url=' + encodeURIComponent(REDIRECT_TO_TARGET_URL) +
+ '&original-redirect-mode=error&redirect-mode=manual',
+ {redirect: 'error'}),
+ 'The opaqueredirect response from the service worker should ' +
+ 'be treated as an error when the redirect flag of request was' +
+ ' \'error\'.')),
+ 'mode: "error", mode change: "manual"');
+
+promise_test(t => setup
+ .then(() => redirected_test(
+ './?url=' + encodeURIComponent(REDIRECT_TO_TARGET_URL) +
+ '&original-redirect-mode=manual&redirect-mode=manual',
+ {redirect: 'manual'},
+ frame.contentWindow.fetch,
+ cache,
+ false /* expected_redirected */,
+ [REDIRECT_TO_TARGET_URL])),
+ 'mode: "manual", no mode change');
+</script>
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/http/tests/serviceworker/redirected-response.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698