GeezerVison is showing github.com — test: update WPT for url to 67880a4eb83ca9aa732eec4b35a1971ff5bf37ff · nodejs/node@c52bc5d
File treeExpand file treeCollapse file tree Open diff view settings Expand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/README.md +1-1Lines changed: 1 addition & 1 deletion - Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|
@@ -28,7 +28,7 @@ Last update: | 28 | 28 | - resource-timing: https://github.com/web-platform-tests/wpt/tree/22d38586d0/resource-timing | 29 | 29 | - resources: https://github.com/web-platform-tests/wpt/tree/1e140d63ec/resources | 30 | 30 | - streams: https://github.com/web-platform-tests/wpt/tree/2bd26e124c/streams | 31 | | -- url: https://github.com/web-platform-tests/wpt/tree/6a39784534/url | | 31 | +- url: https://github.com/web-platform-tests/wpt/tree/67880a4eb8/url | 32 | 32 | - user-timing: https://github.com/web-platform-tests/wpt/tree/5ae85bf826/user-timing | 33 | 33 | - wasm/jsapi: https://github.com/web-platform-tests/wpt/tree/cde25e7e3c/wasm/jsapi | 34 | 34 | - wasm/webapi: https://github.com/web-platform-tests/wpt/tree/fd1b23eeaa/wasm/webapi | |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/META.yml -1Lines changed: 0 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -3,5 +3,4 @@ suggested_reviewers: | 3 | 3 | - mikewest | 4 | 4 | - domenic | 5 | 5 | - annevk | 6 | | - - GPHemsley | 7 | 6 | - TimothyGu |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/README.md +7-5Lines changed: 7 additions & 5 deletions - Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|
| 1 | | -## urltestdata.json | | 1 | +## urltestdata.json / urltestdata-javascript-only.json | 2 | 2 |
| 3 | | -`resources/urltestdata.json` contains URL parsing tests suitable for any URL parser implementation. | | 3 | +[`resources/urltestdata.json`](resources/urltestdata.json) contains URL parsing tests suitable for any URL parser implementation. | | 4 | +[`resources/urltestdata-javascript-only.json`](resources/urltestdata-javascript-only.json) contains URL parsing tests specifically meant | | 5 | +for JavaScript's `URL()` class as well as other languages accepting non-scalar-value strings. | 4 | 6 |
| 5 | | -It's used as a source of tests by `a-element.html`, `failure.html`, `url-constructor.any.js`, and | 6 | | -other test files in this directory. | | 7 | +These files are used as a source of tests by `a-element.html`, `failure.html`, `url-constructor.any.js`, | | 8 | +and other test files in this directory. | 7 | 9 |
| 8 | | -The format of `resources/urltestdata.json` is a JSON array of comments as strings and test cases as | | 10 | +Both files share the same format. They consist of a JSON array of comments as strings and test cases as | 9 | 11 | objects. The keys for each test case are: | 10 | 12 |
| 11 | 13 | * `input`: a string to be parsed as URL. | |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/a-element-origin-xhtml.xhtml +5-1Lines changed: 5 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
| 12 | 12 | <script src="resources/a-element-origin.js"></script> | 13 | 13 | </body> | 14 | 14 | </html> | 15 | | -<!-- Other dependencies: resources/urltestdata.json --> | | 15 | +<!-- | | 16 | + Other dependencies: | | 17 | + * resources/urltestdata.json, | | 18 | + * resources/urltestdata-javascript-only.json, | | 19 | +--> |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/a-element-origin.html +5-1Lines changed: 5 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
| 5 | 5 | <base id=base> | 6 | 6 | <div id=log></div> | 7 | 7 | <script src=resources/a-element-origin.js></script> | 8 | | -<!-- Other dependencies: resources/urltestdata.json --> | | 8 | +<!-- | | 9 | + Other dependencies: | | 10 | + * resources/urltestdata.json, | | 11 | + * resources/urltestdata-javascript-only.json, | | 12 | +--> |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/a-element-xhtml.xhtml +5-1Lines changed: 5 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
| 17 | 17 | <script src="resources/a-element.js"></script> | 18 | 18 | </body> | 19 | 19 | </html> | 20 | | -<!-- Other dependencies: resources/urltestdata.json --> | | 20 | +<!-- | | 21 | + Other dependencies: | | 22 | + * resources/urltestdata.json, | | 23 | + * resources/urltestdata-javascript-only.json, | | 24 | +--> |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/a-element.html +5-1Lines changed: 5 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
| 10 | 10 | <base id=base> | 11 | 11 | <div id=log></div> | 12 | 12 | <script src=resources/a-element.js></script> | 13 | | -<!-- Other dependencies: resources/urltestdata.json --> | | 13 | +<!-- | | 14 | + Other dependencies: | | 15 | + * resources/urltestdata.json, | | 16 | + * resources/urltestdata-javascript-only.json, | | 17 | +--> | 14 | 18 |
| 15 | 19 |
| 16 | 20 | <a id="multline-entity" download="multline-entity.txt" href="data:text/plain;charset=utf-8,first%20line second%20line"> Link with embedded \n is parsed correctly </a> | |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/failure.html +4-1Lines changed: 4 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
| 6 | 6 | <script src=/resources/testharnessreport.js></script> | 7 | 7 | <div id=log></div> | 8 | 8 | <script> | 9 | | -promise_test(() => fetch("resources/urltestdata.json").then(res => res.json()).then(runTests), "Loading data…") | | 9 | +promise_test(() => Promise.all([ | | 10 | + fetch("resources/urltestdata.json").then(res => res.json()), | | 11 | + fetch("resources/urltestdata-javascript-only.json").then(res => res.json()), | | 12 | +]).then((tests) => tests.flat()).then(runTests), "Loading data…"); | 10 | 13 |
| 11 | 14 | function runTests(testData) { | 12 | 15 | for (const test of testData) { | |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/idlharness-shadowrealm.window.js -2Lines changed: 0 additions & 2 deletions Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/url/idlharness.any.js +1Lines changed: 1 addition & 0 deletions | Original file line number | Diff line number | Diff line change |
|---|
| 1 | 1 | // META: script=/resources/WebIDLParser.js | 2 | 2 | // META: script=/resources/idlharness.js | | 3 | +// META: global=window,dedicatedworker,shadowrealm-in-window | 3 | 4 |
| 4 | 5 | idl_test( | 5 | 6 | ['url'], | |
|