Skip to content

Commit

Permalink
Bug 1623546 [wpt PR 22328] - [WPT/common/security-features] Sort subt…
Browse files Browse the repository at this point in the history
…ests to reduce diffs, a=testonly

Automatic update from web-platform-tests
[WPT/common/security-features] Sort subtests to reduce diffs

When `spec.src.json` is modified, the order of subtests were
changed and thus unncessesary diffs were generated, even if
the subtest contents are not modified.
This CL sorts subtests to avoid such diffs.

Generated files will be modified in
https://chromium-review.googlesource.com/c/chromium/src/+/2109511

Bug: 906850
Change-Id: Ic17d8122b0b5139e89ec2b4423511b933941ad65
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109570
Reviewed-by: Kenichi Ishibashi <bashichromium.org>
Reviewed-by: Eriko Kurimoto <elkurinchromium.org>
Reviewed-by: Hiroki Nakagawa <nhirokichromium.org>
Commit-Queue: Hiroshige Hayashizaki <hiroshigechromium.org>
Cr-Commit-Position: refs/heads/master{#751602}

--

wpt-commits: 6a665f63ae39b552f60b971f11c34b215861fcb3
wpt-pr: 22328

UltraBlame original commit: 70b8e391cc964657aef75e569c31abd76f0066cf
  • Loading branch information
marco-c committed Mar 23, 2020
1 parent ef9c983 commit db9af61
Show file tree
Hide file tree
Showing 3 changed files with 29 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1210,16 +1210,38 @@
{
}

serialized_scenarios
=
sorted
(

[
dump_test_parameters
(
scenario
)
for
scenario
in
scenarios
]
)

parameters
[
'
scenarios
'
]
=
dump_test_parameters
"
\
n
"
.
join
(
scenarios
serialized_scenarios
)
.
replace
Expand All @@ -1237,7 +1259,7 @@
"
"
*
8
10
)

test_directory
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -178,11 +178,13 @@ script
>
TestCase
(
[
%
(
scenarios
)
s
]
new
SanityChecker
(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -118,11 +118,13 @@ script
>
TestCase
(
[
%
(
scenarios
)
s
]
new
SanityChecker
(
Expand Down

0 comments on commit db9af61

Please sign in to comment.