Bug 1724948 [wpt PR 29970] - Change `Tests.prototype.all_done` to require either...
commit182a78e3a322ca786d7f8c2b0de9ce5da5e7d909
authorAndreu Botella <abb@randomunok.com>
Thu, 23 Sep 2021 13:32:41 +0000 (23 13:32 +0000)
committermoz-wptsync-bot <wptsync@mozilla.com>
Wed, 29 Sep 2021 09:24:05 +0000 (29 09:24 +0000)
treec8cddedb62a0b989806b56b3540ed0d1a0e696e3
parent32b2053c3b7234b703b3c41328d0c556fad58028
Bug 1724948 [wpt PR 29970] - Change `Tests.prototype.all_done` to require either tests or pending remotes, a=testonly

Automatic update from web-platform-tests
Change `Tests.prototype.all_done` to require either tests or pending remotes

`Tests.prototype.all_done` used to treat the presence of at least one
test as a necessary condition for all tests to be done. This would be
false in cases where an exception is thrown inside the `setup` function,
but the code path that runs in that case doesn't use the `all_done()`
function.

However, if an exception is thrown inside the `setup` function in a
worker test, `tests.all_done()` is called in that code path, which
results in the completion message from the worker being ignored. This
change fixes this by changing the condition to require the presence of
either tests or remote contexts.

Closes #29777.

--

wpt-commits: c563d3aaf49ab4f40eb16b4e99be9230e536286f
wpt-pr: 29970
testing/web-platform/tests/resources/testharness.js