HomePhabricator Dev

Bug 1610933 [wpt PR 21353] - Replace some "assert_throws(new FooError, stuff)"…

This commit no longer exists in the repository. It may have been part of a branch which was deleted.

Description

Bug 1610933 [wpt PR 21353] - Replace some "assert_throws(new FooError, stuff)" calls with assert_throws_js, a=testonly

Automatic update from web-platform-tests
Replace some "assert_throws(new FooError, stuff)" calls with assert_throws_js.

This diff was generated by running:

find . -type f -print0 | xargs -0 perl -pi -e 'BEGIN { $/ = undef; } s/assert_throws\(([ \n]*)new ([A-Za-z]*Error) *(, *.)/assert_throws_js(\1\2\3/gs'

and then manually adjusting url/failure.html to use the TypeError from the right
global when doing cross-global things.

This does affect indentation poorly in cases when the first arg was on the same
line as the assert_throws, there was a newline after the ',' after the first
arg, and the following args were lined up with the first arg. Fixing that,
especially when there are multiple lines after the first arg, is not trivial
with a regexp.

wpt-commits: a3cddf04f127ef7f14556bd0326c804c2e72fb2d
wpt-pr: 21353

Details

Provenance
bzbarskyAuthored on

Commit No Longer Exists

This commit no longer exists in the repository.