LayoutTests/http/tests/fetch/chromium/error-messages.html - Issue 1216203003: Replace all sequential_promise_tests

Unified Diff: LayoutTests/http/tests/fetch/chromium/error-messages.html

Issue 1216203003: Replace all sequential_promise_tests (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@update-testharnessjs

Patch Set: Created 5 years, 5 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 | « LayoutTests/bluetooth/requestDevice.html ('k') | LayoutTests/http/tests/fetch/resources/fetch-access-control-util.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/http/tests/fetch/chromium/error-messages.html

diff --git a/LayoutTests/http/tests/fetch/chromium/error-messages.html b/LayoutTests/http/tests/fetch/chromium/error-messages.html

index 7c216c7e15e15ff31851734fb809215d1b54788a..8be3a36c9abb6af3a82916143142643fba9c43b1 100644

--- a/LayoutTests/http/tests/fetch/chromium/error-messages.html

+++ b/LayoutTests/http/tests/fetch/chromium/error-messages.html

@@ -1,15 +1,14 @@

<!doctype html>

-<script src="/resources/testharness-helpers.js"></script>

-sequential_promise_test(function(t) {

+promise_test(function(t) {

return fetch('../resources/doctype.html')

.then(function() {}, t.unreached_func('must resolve'));

}, 'fetch() with 200 should not output error messages');

-sequential_promise_test(function(t) {

+promise_test(function(t) {

return fetch('../resources/not-found.html')

.then(function() {}, t.unreached_func('must resolve'));

}, 'fetch() with 404 should not output error messages');

@@ -17,12 +16,12 @@ sequential_promise_test(function(t) {

// Two didFail() errors

// that have console error messages from outside modules/fetch,

// and thus we OMIT messages from modules/fetch.

-sequential_promise_test(function(t) {

+promise_test(function(t) {

return fetch('../resources/redirect-loop.php?Count=100')

.then(t.unreached_func('must reject'), function() {});

}, 'net::ERR_TOO_MANY_REDIRECTS should not output error messages');

-sequential_promise_test(function(t) {

+promise_test(function(t) {

return fetch('../resources/slow-failure.cgi')

.then(function() {}, t.unreached_func('must resolve'));

}, 'net::ERR_INVALID_CHUNKED_ENCODING should not output error messages');

@@ -30,7 +29,7 @@ sequential_promise_test(function(t) {

// A didFailAccessControlCheck() error

// that has NO console error messages from outside modules/fetch,

// and thus we OUTPUT a message from modules/fetch.

-sequential_promise_test(function(t) {

+promise_test(function(t) {

return fetch('http://localhost:8000/fetch/resources/doctype.html')

.then(t.unreached_func('must reject'), function() {});

}, 'A didFailAccessControlCheck() error should output an error message');

@@ -38,7 +37,7 @@ sequential_promise_test(function(t) {

// A didFailRedirectCheck() error

// that has NO console error messages from outside modules/fetch,

// and thus we OUTPUT a message from modules/fetch.

-sequential_promise_test(function(t) {

+promise_test(function(t) {

return fetch('http://localhost:8000/fetch/resources/redirect-loop.php?Count=100&ACAOrigin=*')

.then(t.unreached_func('must reject'), function() {});

}, 'A didFailRedirectCheck() error should output an error message');

« no previous file with comments | « LayoutTests/bluetooth/requestDevice.html ('k') | LayoutTests/http/tests/fetch/resources/fetch-access-control-util.js » ('j') | no next file with comments »