Update scripts to reflect new WPT directory name

This commit is contained in:
Martin Robinson 2023-06-24 09:22:18 +02:00
parent 234d507234
commit f017169ae4
No known key found for this signature in database
GPG key ID: D56AA4FA55EFE6F8
21 changed files with 39 additions and 45 deletions

View file

@ -17,7 +17,7 @@
import os import os
test_root = os.path.join('tests', 'wpt', 'web-platform-tests') test_root = os.path.join('tests', 'wpt', 'tests')
meta_root = os.path.join('tests', 'wpt', 'meta') meta_root = os.path.join('tests', 'wpt', 'meta')
missing_dirs = [] missing_dirs = []
@ -37,7 +37,7 @@ for base_dir, dir_names, files in os.walk(meta_root):
missing_dirs += [meta_dir] missing_dirs += [meta_dir]
continue continue
# Turn tests/wpt/meta/foo into tests/wpt/web-platform-tests/foo. # Turn tests/wpt/meta/foo into tests/wpt/tests/foo.
test_dir = os.path.join(test_root, os.path.relpath(meta_dir, meta_root)) test_dir = os.path.join(test_root, os.path.relpath(meta_dir, meta_root))
if not os.path.exists(test_dir): if not os.path.exists(test_dir):
missing_dirs += [meta_dir] missing_dirs += [meta_dir]
@ -48,7 +48,7 @@ for base_dir, dir_names, files in os.walk(meta_root):
if fname in ['__dir__.ini', 'MANIFEST.json', 'mozilla-sync']: if fname in ['__dir__.ini', 'MANIFEST.json', 'mozilla-sync']:
continue continue
# Turn tests/wpt/meta/foo/bar.html.ini into tests/wpt/web-platform-tests/foo/bar.html. # Turn tests/wpt/meta/foo/bar.html.ini into tests/wpt/tests/foo/bar.html.
test_dir = os.path.join(test_root, os.path.relpath(base_dir, meta_root)) test_dir = os.path.join(test_root, os.path.relpath(base_dir, meta_root))
test_file = os.path.join(test_dir, fname) test_file = os.path.join(test_dir, fname)
if not os.path.exists(os.path.splitext(test_file)[0]): if not os.path.exists(os.path.splitext(test_file)[0]):

View file

@ -20,7 +20,7 @@
import os import os
test_root = os.path.join('tests', 'wpt', 'web-platform-tests') test_root = os.path.join('tests', 'wpt', 'tests')
meta_root = os.path.join('tests', 'wpt', 'meta') meta_root = os.path.join('tests', 'wpt', 'meta')
test_counts = {} test_counts = {}

View file

@ -136,20 +136,14 @@ def wpt_path(is_firefox, topdir, *paths):
def wptrunner_path(is_firefox, topdir, *paths): def wptrunner_path(is_firefox, topdir, *paths):
wpt_root = wpt_path(is_firefox, topdir) wpt_root = wpt_path(is_firefox, topdir)
if is_firefox: rel = os.path.join(wpt_root, "tests", "tools", "wptrunner")
rel = os.path.join(wpt_root, "tests", "tools", "wptrunner")
else:
rel = os.path.join(wpt_root, "web-platform-tests", "tools", "wptrunner")
return os.path.join(topdir, rel, *paths) return os.path.join(topdir, rel, *paths)
def wptserve_path(is_firefox, topdir, *paths): def wptserve_path(is_firefox, topdir, *paths):
wpt_root = wpt_path(is_firefox, topdir) wpt_root = wpt_path(is_firefox, topdir)
if is_firefox: rel = os.path.join(wpt_root, "tests", "tools", "wptserve")
rel = os.path.join(wpt_root, "tests", "tools", "wptserve")
else:
rel = os.path.join(wpt_root, "web-platform-tests", "tools", "wptserve")
return os.path.join(topdir, rel, *paths) return os.path.join(topdir, rel, *paths)

View file

@ -42,7 +42,7 @@ from distutils.dir_util import copy_tree
SCRIPT_PATH = os.path.split(__file__)[0] SCRIPT_PATH = os.path.split(__file__)[0]
PROJECT_TOPLEVEL_PATH = os.path.abspath(os.path.join(SCRIPT_PATH, "..", "..")) PROJECT_TOPLEVEL_PATH = os.path.abspath(os.path.join(SCRIPT_PATH, "..", ".."))
WEB_PLATFORM_TESTS_PATH = os.path.join("tests", "wpt", "web-platform-tests") WEB_PLATFORM_TESTS_PATH = os.path.join("tests", "wpt", "tests")
SERVO_TESTS_PATH = os.path.join("tests", "wpt", "mozilla", "tests") SERVO_TESTS_PATH = os.path.join("tests", "wpt", "mozilla", "tests")
CLANGFMT_CPP_DIRS = ["support/hololens/"] CLANGFMT_CPP_DIRS = ["support/hololens/"]
@ -711,14 +711,14 @@ class WebPlatformTestsCreator(CommandBase):
if test_url is None: if test_url is None:
print("""Test path %s is not in wpt directories: print("""Test path %s is not in wpt directories:
tests/wpt/web-platform-tests for tests that may be shared tests/wpt/tests for tests that may be shared
tests/wpt/mozilla/tests for Servo-only tests""" % test_path) tests/wpt/mozilla/tests for Servo-only tests""" % test_path)
return 1 return 1
if reference_url is None: if reference_url is None:
print("""Reference path %s is not in wpt directories: print("""Reference path %s is not in wpt directories:
testing/web-platform/tests for tests that may be shared tests/wpt/tests for tests that may be shared
testing/web-platform/mozilla/tests for Servo-only tests""" % reference_path) tests/wpt/mozilla/tests for Servo-only tests""" % reference_path)
return 1 return 1
if os.path.exists(test_path) and not kwargs["overwrite"]: if os.path.exists(test_path) and not kwargs["overwrite"]:

View file

@ -25,7 +25,7 @@ from .licenseck import OLD_MPL, MPL, APACHE, COPYRIGHT, licenses_toml, licenses_
TOPDIR = os.path.abspath(os.path.dirname(sys.argv[0])) TOPDIR = os.path.abspath(os.path.dirname(sys.argv[0]))
WPT_PATH = os.path.join(".", "tests", "wpt") WPT_PATH = os.path.join(".", "tests", "wpt")
SUITES = ["web-platform-tests", os.path.join("mozilla", "tests")] SUITES = ["tests", os.path.join("mozilla", "tests")]
def wpt_path(*args): def wpt_path(*args):
@ -40,7 +40,7 @@ URL_REGEX = re.compile(br'https?://(?:[-\w.]|(?:%[\da-fA-F]{2}))+')
# Import wptmanifest only when we do have wpt in tree, i.e. we're not # Import wptmanifest only when we do have wpt in tree, i.e. we're not
# inside a Firefox checkout. # inside a Firefox checkout.
if os.path.isfile(WPT_MANIFEST_PATH): if os.path.isfile(WPT_MANIFEST_PATH):
sys.path.append(wpt_path("web-platform-tests", "tools", "wptrunner", "wptrunner")) sys.path.append(wpt_path("tests", "tools", "wptrunner", "wptrunner"))
from wptmanifest import parser, node from wptmanifest import parser, node
# Default configs # Default configs
@ -506,7 +506,7 @@ def check_manifest_dirs(config_file, print_text=True):
print('\rChecking the wpt manifest file...') print('\rChecking the wpt manifest file...')
p = parser.parse(lines) p = parser.parse(lines)
paths = rec_parse(wpt_path("web-platform-tests"), p) paths = rec_parse(wpt_path("tests"), p)
for idx, path in enumerate(paths): for idx, path in enumerate(paths):
if '_mozilla' in path or '_webgl' in path or '_webgpu' in path: if '_mozilla' in path or '_webgl' in path or '_webgpu' in path:
continue continue
@ -1152,7 +1152,7 @@ class WPTLint(LintRunner):
if self.stylo or self.no_wpt: if self.stylo or self.no_wpt:
return return
wpt_working_dir = os.path.abspath(os.path.join(WPT_PATH, "web-platform-tests")) wpt_working_dir = os.path.abspath(os.path.join(WPT_PATH, "tests"))
for suite in SUITES: for suite in SUITES:
files = list(self._get_wpt_files(suite)) files = list(self._get_wpt_files(suite))
if not files: if not files:

View file

@ -15,7 +15,7 @@ import mozlog.commandline
SCRIPT_PATH = os.path.abspath(os.path.dirname(__file__)) SCRIPT_PATH = os.path.abspath(os.path.dirname(__file__))
SERVO_ROOT = os.path.abspath(os.path.join(SCRIPT_PATH, "..", "..")) SERVO_ROOT = os.path.abspath(os.path.join(SCRIPT_PATH, "..", ".."))
WPT_PATH = os.path.join(SERVO_ROOT, "tests", "wpt") WPT_PATH = os.path.join(SERVO_ROOT, "tests", "wpt")
WPT_TOOLS_PATH = os.path.join(WPT_PATH, "web-platform-tests", "tools") WPT_TOOLS_PATH = os.path.join(WPT_PATH, "tests", "tools")
CERTS_PATH = os.path.join(WPT_TOOLS_PATH, "certs") CERTS_PATH = os.path.join(WPT_TOOLS_PATH, "certs")
sys.path.insert(0, WPT_TOOLS_PATH) sys.path.insert(0, WPT_TOOLS_PATH)

View file

@ -9,7 +9,7 @@
# pylint: disable=missing-docstring # pylint: disable=missing-docstring
UPSTREAMABLE_PATH = "tests/wpt/web-platform-tests/" UPSTREAMABLE_PATH = "tests/wpt/tests/"
NO_SYNC_SIGNAL = "[no-wpt-sync]" NO_SYNC_SIGNAL = "[no-wpt-sync]"
OPENED_NEW_UPSTREAM_PR = ( OPENED_NEW_UPSTREAM_PR = (

View file

@ -34,7 +34,7 @@ def create_parser():
def update(check_clean=True, rebuild=False, **kwargs): def update(check_clean=True, rebuild=False, **kwargs):
logger = wptlogging.setup(kwargs, {"mach": sys.stdout}) logger = wptlogging.setup(kwargs, {"mach": sys.stdout})
kwargs = {"config": os.path.join(WPT_PATH, "config.ini"), kwargs = {"config": os.path.join(WPT_PATH, "config.ini"),
"manifest_path": os.path.join(WPT_PATH, "metadata"), "manifest_path": os.path.join(WPT_PATH, "meta"),
"tests_root": None, "tests_root": None,
"metadata_root": None} "metadata_root": None}

View file

@ -1,7 +1,7 @@
diff --git a/tests/wpt/web-platform-tests/fetch/api/redirect/redirect-location.html b/tests/wpt/web-platform-tests/fetch/api/redirect/redirect-location.html diff --git a/tests/wpt/tests/fetch/api/redirect/redirect-location.html b/tests/wpt/tests/fetch/api/redirect/redirect-location.html
index ac35dea54c47..135ad21d15ab 100644 index ac35dea54c47..135ad21d15ab 100644
--- a/tests/wpt/web-platform-tests/fetch/api/redirect/redirect-location.html --- a/tests/wpt/tests/fetch/api/redirect/redirect-location.html
+++ b/tests/wpt/web-platform-tests/fetch/api/redirect/redirect-location.html +++ b/tests/wpt/tests/fetch/api/redirect/redirect-location.html
@@ -13,4 +13,5 @@ @@ -13,4 +13,5 @@
<script src="redirect-location.js"></script> <script src="redirect-location.js"></script>
</body> </body>

View file

@ -1,7 +1,7 @@
diff --git a/tests/wpt/web-platform-tests/non-utf8-file.txt b/tests/wpt/web-platform-tests/non-utf8-file.txt diff --git a/tests/wpt/tests/non-utf8-file.txt b/tests/wpt/tests/non-utf8-file.txt
new file mode 100644 new file mode 100644
index 0000000..5a992e0 index 0000000..5a992e0
--- /dev/null --- /dev/null
+++ b/tests/wpt/web-platform-tests/non-utf8-file.txt +++ b/tests/wpt/tests/non-utf8-file.txt
@@ -0,0 +1 @@ @@ -0,0 +1 @@
+foo bér ÿ " +foo bér <20> "

View file

@ -1,7 +1,7 @@
diff --git a/tests/wpt/web-platform-tests/css/css-test.html b/tests/wpt/web-platform-tests/css/css-test.html diff --git a/tests/wpt/tests/css/css-test.html b/tests/wpt/tests/css/css-test.html
index cffb6eb..6100a49 100644 index cffb6eb..6100a49 100644
--- a/tests/wpt/web-platform-tests/css/out-of-sync-test.html --- a/tests/wpt/tests/css/out-of-sync-test.html
+++ b/tests/wpt/web-platform-tests/css/out-of-sync-test.html +++ b/tests/wpt/tests/css/out-of-sync-test.html
@@ -1,3 +1,3 @@ @@ -1,3 +1,3 @@
<html> <html>
-<h1>css test!</h1> -<h1>css test!</h1>

View file

@ -1,4 +1,4 @@
diff --git a/tests/wpt/mozilla/tests/mozilla/mozilla-test.html b/tests/wpt/web-platform-tests/mozilla-test.html diff --git a/tests/wpt/mozilla/tests/mozilla/mozilla-test.html b/tests/wpt/tests/mozilla-test.html
similarity index 100% similarity index 100%
rename from tests/wpt/mozilla/tests/mozilla/mozilla-test.html rename from tests/wpt/mozilla/tests/mozilla/mozilla-test.html
rename to tests/wpt/web-platform-tests/mozilla-test.html rename to tests/wpt/tests/mozilla-test.html

View file

@ -1,4 +1,4 @@
diff --git a/tests/wpt/web-platform-tests/test.html b/tests/wpt/test.html diff --git a/tests/wpt/tests/test.html b/tests/wpt/test.html
similarity index 100% similarity index 100%
rename from tests/wpt/web-platform-tests/test.html rename from tests/wpt/tests/test.html
rename to tests/wpt/test.html rename to tests/wpt/test.html

View file

@ -1,7 +1,7 @@
diff --git a/tests/wpt/web-platform-tests/css/css-test.html b/tests/wpt/web-platform-tests/css/css-test.html diff --git a/tests/wpt/tests/css/css-test.html b/tests/wpt/tests/css/css-test.html
index cffb6eb..6100a49 100644 index cffb6eb..6100a49 100644
--- a/tests/wpt/web-platform-tests/css/css-test.html --- a/tests/wpt/tests/css/css-test.html
+++ b/tests/wpt/web-platform-tests/css/css-test.html +++ b/tests/wpt/tests/css/css-test.html
@@ -1,3 +1,3 @@ @@ -1,3 +1,3 @@
<html> <html>
-<h1>css test!</h1> -<h1>css test!</h1>

View file

@ -133,7 +133,7 @@ directories = [
"./support/android/apk", "./support/android/apk",
"./support/hololens", "./support/hololens",
"./tests/wpt/harness", "./tests/wpt/harness",
"./tests/wpt/web-platform-tests", "./tests/wpt/tests",
"./tests/wpt/mozilla/tests/mozilla/referrer-policy", "./tests/wpt/mozilla/tests/mozilla/referrer-policy",
"./tests/wpt/mozilla/tests/webgl", "./tests/wpt/mozilla/tests/webgl",
"./python/tidy/tests", "./python/tidy/tests",

View file

@ -9,7 +9,7 @@ In particular, this folder contains:
* `config.ini`: some configuration for the web-platform-tests. * `config.ini`: some configuration for the web-platform-tests.
* `include.ini`: the subset of web-platform-tests we currently run. * `include.ini`: the subset of web-platform-tests we currently run.
* `web-platform-tests`: copy of the web-platform-tests. * `tests`: copy of the web-platform-tests.
* `meta`: expected failures for the web-platform-tests we run. * `meta`: expected failures for the web-platform-tests we run.
* `mozilla`: web-platform-tests that cannot be upstreamed. * `mozilla`: web-platform-tests that cannot be upstreamed.
@ -24,7 +24,7 @@ stdout.
A subset of tests may be run by providing positional arguments to the A subset of tests may be run by providing positional arguments to the
mach command, either as filesystem paths or as test urls e.g. mach command, either as filesystem paths or as test urls e.g.
./mach test-wpt tests/wpt/web-platform-tests/dom/historical.html ./mach test-wpt tests/wpt/tests/dom/historical.html
to run the dom/historical.html test, or to run the dom/historical.html test, or
@ -59,8 +59,8 @@ can only run one WPT server at a time. To fix this:
1. Follow the steps in [**Running the tests manually**](#running-the-tests-manually) 1. Follow the steps in [**Running the tests manually**](#running-the-tests-manually)
2. Add a `break` to [start_servers in serve.py](https://github.com/servo/servo/blob/ce92b7bfbd5855aac18cb4f8a8ec59048041712e/tests/wpt/web-platform-tests/tools/serve/serve.py#L745-L783) as follows: 2. Add a `break` to [start_servers in serve.py](https://github.com/servo/servo/blob/ce92b7bfbd5855aac18cb4f8a8ec59048041712e/tests/wpt/web-platform-tests/tools/serve/serve.py#L745-L783) as follows:
``` ```
--- a/tests/wpt/web-platform-tests/tools/serve/serve.py --- a/tests/wpt/tests/tools/serve/serve.py
+++ b/tests/wpt/web-platform-tests/tools/serve/serve.py +++ b/tests/wpt/tests/tools/serve/serve.py
@@ -746,6 +746,7 @@ def start_servers(logger, host, ports, paths, routes, bind_address, config, @@ -746,6 +746,7 @@ def start_servers(logger, host, ports, paths, routes, bind_address, config,
mp_context, log_handlers, **kwargs): mp_context, log_handlers, **kwargs):
servers = defaultdict(list) servers = defaultdict(list)

View file

@ -12,7 +12,7 @@ sync_path = sync
run-info = . run-info = .
[manifest:upstream] [manifest:upstream]
tests = web-platform-tests tests = tests
metadata = meta metadata = meta
url-base = / url-base = /