Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

WPT: Add dom/abortsignal tests #3436

Merged
merged 1 commit into from
Feb 3, 2025
Merged

WPT: Add dom/abortsignal tests #3436

merged 1 commit into from
Feb 3, 2025

Conversation

npaun
Copy link
Member

@npaun npaun commented Jan 29, 2025

Enables tests from the dom/abortsignal module from WPT.

The following features were added to the harness to support these tests:

  • The Test class, including step, step_func, step_timeout, step_func_done and done
  • The async_test test type

@npaun npaun requested review from jasnell and anonrig January 29, 2025 23:14
@npaun npaun requested review from a team as code owners January 29, 2025 23:14
@npaun
Copy link
Member Author

npaun commented Jan 29, 2025

@anonrig

Bad news: I have a Windows build failure too now.
Good news: It'll be convenient for me to fix it now that it's on my own branch lol.

@jasnell
Copy link
Member

jasnell commented Jan 29, 2025

Great work, will review tomorrow.

build/wpt_modules.bzl Outdated Show resolved Hide resolved
build/BUILD.wpt Outdated Show resolved Hide resolved
src/workerd/api/wpt/dom/abort-test.ts Show resolved Hide resolved
src/workerd/api/wpt/url-test.ts Outdated Show resolved Hide resolved
src/workerd/api/wpt/dom/abort-test.ts Outdated Show resolved Hide resolved
src/wpt/harness.ts Outdated Show resolved Hide resolved
src/wpt/harness.ts Outdated Show resolved Hide resolved
src/wpt/harness.ts Outdated Show resolved Hide resolved
src/wpt/harness.ts Show resolved Hide resolved
src/wpt/harness.ts Outdated Show resolved Hide resolved
@npaun npaun force-pushed the dom/abort-signal branch 5 times, most recently from 08ea879 to 307c8a8 Compare February 3, 2025 15:30
@npaun npaun requested a review from anonrig February 3, 2025 15:30
Try to fix Windows builds; drops files belonging to another target from include
@npaun npaun merged commit 75d6a76 into main Feb 3, 2025
17 checks passed
@npaun npaun deleted the dom/abort-signal branch February 3, 2025 23:12
@fhanau
Copy link
Collaborator

fhanau commented Feb 4, 2025

This breaks the downstream build – please fix or revert

npaun added a commit that referenced this pull request Feb 4, 2025
fhanau pushed a commit that referenced this pull request Feb 4, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants