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

Unified Diff: LayoutTests/http/tests/eventsource/eventsource-status-code-states.html

Issue 347043002: Rework EventSource tests for better Worker test coverage. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Move out CORS tests Created 6 years, 6 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
Index: LayoutTests/http/tests/eventsource/eventsource-status-code-states.html
diff --git a/LayoutTests/http/tests/eventsource/eventsource-status-code-states.html b/LayoutTests/http/tests/eventsource/eventsource-status-code-states.html
index 206b1f87db870330af72d519552ee2a1a4886cf0..e09696f2dae7f837271a0d8944c894a3476775d7 100644
--- a/LayoutTests/http/tests/eventsource/eventsource-status-code-states.html
+++ b/LayoutTests/http/tests/eventsource/eventsource-status-code-states.html
@@ -1,78 +1,3 @@
-<html>
-<body>
-<p>Test EventSource states for different status codes. Should print a series of PASS messages followed by DONE.</p>
-<div id="result"></div>
-<script>
-function log(msg) {
- document.getElementById("result").innerHTML += msg + "<br>";
-}
-
-function arrayCompare(a1, a2) {
- if (a1.length != a2.length)
- return false;
- for (var i = 0; i < a1.length; i++)
- if (a1[i] != a2[i])
- return false;
- return true;
-}
-
-var stateNames = ["CONNECTING", "OPEN", "CLOSED"];
-for (var i in stateNames)
- eval("var " + stateNames[i] + " = " + i);
-
-var tests = [{"code": 200, "expectedStates": [CONNECTING, OPEN, OPEN, CONNECTING, CLOSED]},
- {"code": 204, "expectedStates": [CONNECTING,,, CLOSED, CLOSED]},
- {"code": 205, "expectedStates": [CONNECTING,,, CLOSED, CLOSED]},
- {"code": 202, "expectedStates": [CONNECTING,,, CLOSED, CLOSED]}, // other 2xx
- {"code": 301, "expectedStates": [CONNECTING, OPEN, OPEN, CONNECTING, CLOSED]},
- {"code": 302, "expectedStates": [CONNECTING, OPEN, OPEN, CONNECTING, CLOSED]},
- {"code": 303, "expectedStates": [CONNECTING, OPEN, OPEN, CONNECTING, CLOSED]},
- {"code": 307, "expectedStates": [CONNECTING, OPEN, OPEN, CONNECTING, CLOSED]},
- {"code": 404, "expectedStates": [CONNECTING,,, CLOSED, CLOSED]}]; // any other
-var count = 0;
-
-function runTest() {
- if (count >= tests.length) {
- log("DONE");
- if (window.testRunner)
- testRunner.notifyDone();
- return;
- }
-
- var states = [];
- var es = new EventSource("resources/status-codes.php?status-code=" + tests[count].code);
- states[0] = es.readyState;
-
- es.onopen = function () {
- states[1] = es.readyState;
- };
-
- es.onmessage = function (evt) {
- states[2] = es.readyState;
- };
-
- es.onerror = function () {
- states[3] = es.readyState;
- es.close();
- states[4] = es.readyState;
-
- var result = arrayCompare(states, tests[count].expectedStates) ? "PASS" : "FAIL";
- result += ": status code " + tests[count].code + " resulted in states ";
- for (var i in states)
- result += (i != 0 ? ", " : "") + stateNames[states[i]];
- log(result);
-
- setTimeout(runTest, 0);
- count++;
- };
-}
-
-if (window.testRunner) {
- testRunner.dumpAsText();
- testRunner.waitUntilDone();
-}
-
-runTest();
-</script>
-</body>
-</html>
+<!DOCTYPE HTML>
+<script src="/js-test-resources/js-test.js"></script>
+<script src="script-tests/eventsource-status-code-states.js"></script>

Powered by Google App Engine
This is Rietveld 408576698