Switch test_sky back to apache

This CL reverts my previous change because it didn't work on the bots.

TBR=eseidel@chromium.org

Review URL: https://codereview.chromium.org/692333007
This commit is contained in:
Adam Barth 2014-11-04 17:30:25 -08:00
parent 73e759df2c
commit 46a9ce7254
6 changed files with 24 additions and 22 deletions

View File

@ -1,17 +1,17 @@
ERROR: Uncaught SyntaxError: Unexpected token <
SOURCE: http://127.0.0.1:8000/sky/tests/parser/script.sky:6
SOURCE: http://127.0.0.1:8000/parser/script.sky:6
ERROR: Uncaught SyntaxError: Unexpected token <
SOURCE: http://127.0.0.1:8000/sky/tests/parser/script.sky:14
SOURCE: http://127.0.0.1:8000/parser/script.sky:14
ERROR: Uncaught SyntaxError: Unexpected token <
SOURCE: http://127.0.0.1:8000/sky/tests/parser/script.sky:21
SOURCE: http://127.0.0.1:8000/parser/script.sky:21
ERROR: Uncaught ReferenceError: TEST is not defined
SOURCE: http://127.0.0.1:8000/sky/tests/parser/script.sky:27
SOURCE: http://127.0.0.1:8000/parser/script.sky:27
ERROR: Uncaught SyntaxError: Invalid regular expression: missing /
SOURCE: http://127.0.0.1:8000/sky/tests/parser/script.sky:34
SOURCE: http://127.0.0.1:8000/parser/script.sky:34
ERROR: Uncaught SyntaxError: Invalid regular expression: missing /
SOURCE: http://127.0.0.1:8000/sky/tests/parser/script.sky:38
SOURCE: http://127.0.0.1:8000/parser/script.sky:38
ERROR: Uncaught SyntaxError: Unexpected token }
SOURCE: http://127.0.0.1:8000/sky/tests/parser/script.sky:43
SOURCE: http://127.0.0.1:8000/parser/script.sky:43
<html>
<import src="../resources/dump-as-markup.sky"></import>
<body>

View File

@ -1,4 +1,4 @@
CONSOLE: LOG: url => http://127.0.0.1:8000/sky/tests/services/resources/pass.txt
CONSOLE: LOG: url => http://127.0.0.1:8000/services/resources/pass.txt
CONSOLE: LOG: status_line => HTTP/1.1 200 OK
CONSOLE: LOG: mime_type => text/plain
CONSOLE: LOG: read 31 bytes

View File

@ -23,7 +23,7 @@ describe('Mojo network_service', function() {
loader.URLLoader.stubClass,
loader.URLLoader.proxyClass);
var urlRequest = new loader.URLRequest();
urlRequest.url = "http://127.0.0.1:8000/sky/tests/services/resources/pass.txt";
urlRequest.url = "http://127.0.0.1:8000/services/resources/pass.txt";
urlRequest.method = "GET";
urlRequest.auto_follow_redirects = true;
var urlRequestPromise = urlLoaderConnection.remote.start(urlRequest);

View File

@ -17,7 +17,7 @@ describe('XMLHttpRequest', function() {
assert.ok(false, "Got error: " + JSON.stringify(error));
done();
};
xhr.open("GET", "http://127.0.0.1:8000/sky/tests/services/resources/pass.txt");
xhr.open("GET", "http://127.0.0.1:8000/services/resources/pass.txt");
xhr.send();
});
});

View File

@ -34,12 +34,11 @@ import difflib
import errno
import itertools
import logging
import math
import os
import operator
import optparse
import os
import math
import re
import subprocess
import sys
try:
@ -1099,12 +1098,11 @@ class Port(object):
Ports can stub this out if they don't need a web server to be running."""
assert not self._http_server, 'Already running an http server.'
self._http_server = subprocess.Popen([
self.path_to_script('sky_server'),
'-t', self.get_option('configuration'),
self.path_from_chromium_base(),
'8000',
])
server = apache_http.ApacheHTTP(self, self.results_directory(),
additional_dirs=additional_dirs,
number_of_servers=number_of_drivers)
server.start()
self._http_server = server
def start_websocket_server(self):
"""Start a web server. Raise an error if it can't start or is already running.
@ -1140,7 +1138,7 @@ class Port(object):
def stop_http_server(self):
"""Shut down the http server if it is running. Do nothing if it isn't."""
if self._http_server:
self._http_server.terminate()
self._http_server.stop()
self._http_server = None
def stop_websocket_server(self):

View File

@ -230,8 +230,12 @@ class Driver(object):
'/http/tests/security/mixedContent/https/test1.html') will be loaded
over HTTPS; all other tests over HTTP.
"""
assert self.is_http_test(test_name)
return "http://127.0.0.1:8000/sky/tests/" + test_name
if not self.is_http_test(test_name):
return path.abspath_to_uri(self._port.host.platform, self._port.abspath_for_test(test_name))
if "/https/" in test_name:
return "https://127.0.0.1:8443/" + test_name
return "http://127.0.0.1:8000/" + test_name
def uri_to_test(self, uri):
"""Return the base layout test name for a given URI.