1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2025-01-03 04:48:52 -05:00

chore(wpt): Enable WPT worker tests (#12222)

Classic workers were implemented in denoland#11338, which also enabled the WPT
tests in the `workers` directory. However, the rest of WPT worker tests
were not enabled because a number of them were hanging due to
web-platform-tests/wpt#29777. Now that that WPT issue is fixed, the bulk
of worker tests can be enabled.

There are still a few tests that hang, and so haven't been enabled. In
particular:

- The following tests seem to hang because a promise fails to resolve.
  We can detect such cases in non-worker tests because the process will
  exit without calling the WPT completion callback, but in worker tests
  the worker message ops will keep the event loop running. This will be
  fixed when we add timeouts to WPT tests (denoland#9460).

  - `/fetch/api/basic/error-after-response.any.worker.html`
  - `/html/webappapis/microtask-queuing/queue-microtask-exceptions.any.worker.html`
  - `/webmessaging/message-channels/worker-post-after-close.any.worker.html`
  - `/webmessaging/message-channels/worker.any.worker.html`
  - `/websockets/Create-on-worker-shutdown.any.worker.html`

- The following tests apparently hang because a promise rejection is
  never handled, which will kill the process in the main thread but not
  in workers (denoland#12221).

  - `/streams/readable-streams/async-iterator.any.worker.html`
  - `/workers/interfaces/WorkerUtils/importScripts/report-error-setTimeout-cross-origin.sub.any.worker.html`
  - `/workers/interfaces/WorkerUtils/importScripts/report-error-setTimeout-redirect-to-cross-origin.sub.any.worker.html`
  - `/workers/interfaces/WorkerUtils/importScripts/report-error-setTimeout-same-origin.sub.any.worker.html`
This commit is contained in:
Andreu Botella 2021-10-08 17:44:38 +02:00 committed by GitHub
parent a5d3c8b06c
commit 66804d26f3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 14267 additions and 42 deletions

View file

@ -648,7 +648,8 @@ function discoverTestsToRun(
if ( if (
!url.pathname.endsWith(".any.html") && !url.pathname.endsWith(".any.html") &&
!url.pathname.endsWith(".window.html") && !url.pathname.endsWith(".window.html") &&
!url.pathname.endsWith(".worker.html") !url.pathname.endsWith(".worker.html") &&
!url.pathname.endsWith(".worker-module.html")
) { ) {
continue; continue;
} }

File diff suppressed because it is too large Load diff