Skip to content

Commit

Permalink
Bug 1600851 [wpt PR 20574] - [Import Maps] Migrate Jest-based resolut…
Browse files Browse the repository at this point in the history
…ion tests into JSON-based tests, a=testonly

Automatic update from web-platform-tests
[Import Maps] Migrate Jest-based resolution tests into JSON-based tests

This CL

- Defines JSON test object format (see README.md) that describes
  the configurations and specifiers to be tested,
- Implements the WPT test helper for executing the tests
  based on the JSON test objects (common-test-helper.js),
- Converts Jest-based resolution tests into JSONs
  (with some refinement, and removing test cases depending
  on interoperability issues of underlying URL parsers), and
- Removes imported resolution tests.

The dependency to Blink internals remains after this CL.
Removing this dependency is planned and discussed at
WICG/import-maps#170.

Bug: 1026809, WICG/import-maps#170
Change-Id: I993cc9cd7746d7175142f8296ac434571f5d7157
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1927852
Commit-Queue: Hiroshige Hayashizaki <hiroshigechromium.org>
Reviewed-by: Domenic Denicola <domenicchromium.org>
Cr-Commit-Position: refs/heads/master{#721239}

--

wpt-commits: de73fe8cc89c55dea4fee1fc46d98d73dad2b21b
wpt-pr: 20574

UltraBlame original commit: 655a6e51517f24120040773fa1aa41f69532af6c
  • Loading branch information
marco-c committed Dec 10, 2019
1 parent f87f734 commit 6bdd57d
Show file tree
Hide file tree
Showing 17 changed files with 7,848 additions and 6,816 deletions.
723 changes: 723 additions & 0 deletions testing/web-platform/tests/import-maps/common/README.md

Large diffs are not rendered by default.

204 changes: 204 additions & 0 deletions testing/web-platform/tests/import-maps/common/resolving.tentative.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,204 @@
<
!
DOCTYPE
html
>
<
meta
name
=
"
timeout
"
content
=
"
long
"
>
<
script
src
=
"
/
resources
/
testharness
.
js
"
>
<
/
script
>
<
script
src
=
"
/
resources
/
testharnessreport
.
js
"
>
<
/
script
>
<
body
>
<
script
type
=
"
module
"
>
import
{
runTestsFromJSON
}
from
"
.
/
resources
/
common
-
test
-
helper
.
js
"
;
for
(
const
json
of
[
'
resources
/
scopes
.
json
'
'
resources
/
empty
-
import
-
map
.
json
'
'
resources
/
packages
-
via
-
trailing
-
slashes
.
json
'
'
resources
/
tricky
-
specifiers
.
json
'
'
resources
/
url
-
specifiers
.
json
'
'
resources
/
data
-
base
-
url
.
json
'
'
resources
/
scopes
-
exact
-
vs
-
prefix
.
json
'
'
resources
/
overlapping
-
entries
.
json
'
]
)
{
promise_test
(
(
)
=
>
runTestsFromJSON
(
json
)
"
Test
helper
:
fetching
and
sanity
checking
test
JSON
:
"
+
json
)
;
}
<
/
script
>
Loading

0 comments on commit 6bdd57d

Please sign in to comment.