Skip to content

Commit 891394c

Browse files
authored
Update WPT to 818762709 (#3667)
1 parent 8a5ca61 commit 891394c

File tree

4 files changed

+52
-9
lines changed

4 files changed

+52
-9
lines changed

build/deps/build_deps.jsonc

+1-1
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@
241241
"repo": "workerd-tools",
242242
"file_regex": "wpt-.*.tar.gz",
243243
"build_file": "//:build/BUILD.wpt",
244-
"freeze_version": "wpt-af83b77e2"
244+
"freeze_version": "wpt-818762709"
245245
}
246246
]
247247
}

build/deps/gen/dep_wpt.bzl

+4-4
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,10 @@
22

33
load("@//:build/http.bzl", "http_archive")
44

5-
TAG_NAME = "wpt-af83b77e2"
6-
URL = "https://github.com/cloudflare/workerd-tools/releases/download/wpt-af83b77e2/wpt-af83b77e2.tar.gz"
7-
STRIP_PREFIX = "wpt-af83b77e2"
8-
SHA256 = "ce7f09bc8ab6604362897c484a1ee044afd7254741e3cecd7cf609d93d779d51"
5+
TAG_NAME = "wpt-818762709"
6+
URL = "https://github.com/cloudflare/workerd-tools/releases/download/wpt-818762709/wpt-818762709.tar.gz"
7+
STRIP_PREFIX = "wpt-818762709"
8+
SHA256 = "c93305fb0cbd84e20fa329c2ab2e417dbb2456ba1fe521ee35e07f5f4b191341"
99
TYPE = "tgz"
1010

1111
def dep_wpt():

src/wpt/url-test.ts

+38-4
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,14 @@
55
import { type TestRunnerConfig } from 'harness/harness';
66

77
export default {
8-
'IdnaTestV2.window.js': {},
8+
'IdnaTestV2-removed.window.js': {},
9+
'IdnaTestV2.window.js': {
10+
comment:
11+
'WPT recently updated this test to check for Unicode 16 compliance',
12+
// We now have a huge number of failures
13+
// https://github.com/web-platform-tests/wpt/commit/d0cd7c05f70f6928234e90ddff90b39ef9c1eebc
14+
skipAllTests: true,
15+
},
916
'historical.any.js': {},
1017
'idlharness.any.js': {
1118
comment: 'Does not contain any relevant tests',
@@ -28,17 +35,38 @@ export default {
2835
},
2936
'toascii.window.js': {
3037
comment:
31-
'Replacer disables tests involving document.createElement. Expected failures are due to Unicode 15.1',
38+
'Replacer disables tests involving document.createElement. Expected failures are due to Unicode 15.1 and Unicode 16',
3239
expectedFailures: [
40+
// Unicode 15.1
3341
// Taken from https://github.com/nodejs/node/blob/5ab7c4c5b01e7579fd436000232f0f0484289d44/test/wpt/status/url.json#L13
3442
'\uD87E\uDC68.com (using URL)',
3543
'\uD87E\uDC68.com (using URL.host)',
3644
'\uD87E\uDC68.com (using URL.hostname)',
45+
// Unicode 16
46+
// https://github.com/web-platform-tests/wpt/commit/12de160159122dc4d0e3ee2d756444b249063e8a
47+
'look\u180Eout.net (using URL)',
48+
'look\u180Eout.net (using URL.host)',
49+
'look\u180Eout.net (using URL.hostname)',
50+
'look\u206Bout.net (using URL)',
51+
'look\u206Bout.net (using URL.host)',
52+
'look\u206Bout.net (using URL.hostname)',
53+
'\u04C0.com (using URL)',
54+
'\u04C0.com (using URL.host)',
55+
'\u04C0.com (using URL.hostname)',
56+
'\u2183.com (using URL)',
57+
'\u2183.com (using URL.host)',
58+
'\u2183.com (using URL.hostname)',
3759
],
3860
replace: (code): string =>
3961
code.replace(/\["url", "a", "area"\]/, '[ "url" ]'),
4062
},
41-
'url-constructor.any.js': {},
63+
'url-constructor.any.js': {
64+
comment: 'We are expected to encode ^ as %5E',
65+
expectedFailures: [
66+
'Parsing: <foo://host/ !"$%&\'()*+,-./:;<=>@[\\]^_`{|}~> without base',
67+
'Parsing: <wss://host/ !"$%&\'()*+,-./:;<=>@[\\]^_`{|}~> without base',
68+
],
69+
},
4270
'url-origin.any.js': {},
4371
'url-searchparams.any.js': {},
4472
'url-setters-a-area.window.js': {
@@ -47,7 +75,13 @@ export default {
4775
skipAllTests: true,
4876
},
4977
'url-setters-stripping.any.js': {},
50-
'url-setters.any.js': {},
78+
'url-setters.any.js': {
79+
comment: 'We are expected to encode ^ as %5E',
80+
expectedFailures: [
81+
"URL: Setting <a:/>.pathname = '\x00\x01\t\n" +
82+
"\r\x1F !\"#$%&'()*+,-./09:;<=>?@AZ[\\]^_`az{|}~\x7F\x80\x81Éé' UTF-8 percent encoding with the default encode set. Tabs and newlines are removed.",
83+
],
84+
},
5185
'url-statics-canparse.any.js': {},
5286
'url-statics-parse.any.js': {},
5387
'url-tojson.any.js': {},

src/wpt/urlpattern-test.ts

+9
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,15 @@ export default {
155155
'Pattern: [{"pathname":"/([[a-z]--a])"}] Inputs: [{"pathname":"/z"}]',
156156
'Pattern: [{"pathname":"/([\\\\d&&[0-1]])"}] Inputs: [{"pathname":"/0"}]',
157157
'Pattern: [{"pathname":"/([\\\\d&&[0-1]])"}] Inputs: [{"pathname":"/3"}]',
158+
'Pattern: ["http://🚲.com/"] Inputs: ["http://🚲.com/"]',
159+
'Pattern: [{"pathname":"\\ud83d \\udeb2"}] Inputs: []',
160+
'Pattern: [{"pathname":"test/:a𐑐b"}] Inputs: [{"pathname":"test/foo"}]',
161+
'Pattern: [{"hostname":"bad#hostname"}] Inputs: undefined',
162+
'Pattern: [{"hostname":"bad%hostname"}] Inputs: undefined',
163+
'Pattern: [{"hostname":"bad/hostname"}] Inputs: undefined',
164+
'Pattern: [{"hostname":"bad\\nhostname"}] Inputs: undefined',
165+
'Pattern: [{"hostname":"bad\\rhostname"}] Inputs: undefined',
166+
'Pattern: [{"hostname":"bad\\thostname"}] Inputs: undefined',
158167
],
159168
},
160169
'urlpattern.https.any.js': {

0 commit comments

Comments
 (0)