1849172 - Add reject handler in various /workers/modules/ WPT test

Status: RESOLVED FIXED

(Core :: DOM: Workers, task)

Description

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

In some of WPT in /workers/modules/,
for example,
https://searchfox.org/mozilla-central/source/testing/web-platform/tests/workers/modules/dedicated-worker-import-data-url-cross-origin.html

it uses

 const msgEvent =
        await new Promise(resolve => worker.onmessage = resolve);

to do async testing,

But if the operator fails, the promise won't be resolved and will cause timeout,
and it also causes the following tests NOT_RUN.

We should add a reject handler for it, so the test can know it failed and continue the next test.

Comment 1

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

Attachment: Bug 1849172 - Add reject handler in /workers/modules/ WPTs.

Add reject handler for the tests, so if one test gets timeout,
the rest of the tests won't get NOT_RUN.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a4b3b09ad08d
Add reject handler in /workers/modules/ WPTs. r=dom-worker-reviewers,smaug

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41550 for changes under testing/web-platform/tests

Comment 4

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/a4b3b09ad08d

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot