Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

test: rename test-eventtarget-whatwg-*.js #43467

Merged
merged 1 commit into from
Jun 18, 2022

Conversation

daeyeon
Copy link
Member

@daeyeon daeyeon commented Jun 18, 2022

This renames some test filenames to align with the existing
parallel/test-whatwg-*.js for consistency.

According to the comment in each file, they are seemingly manually
ported from the wpt dom/events tests.

// Manually ported from: wpt@dom/events/AddEventListenerOptions-once.html

// Manually converted from https://github.com/web-platform-tests/wpt/blob/master/dom/events/AddEventListenerOptions-passive.html

// Manually ported from: wpt@dom/events/AddEventListenerOptions-signal.any.js

// Manually converted from https://github.com/web-platform-tests/wpt/blob/master/dom/events/CustomEvent.html

Refs: #43461

Signed-off-by: Daeyeon Jeong daeyeon.dev@gmail.com

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jun 18, 2022
@aduh95 aduh95 added fast-track PRs that do not need to wait for 48 hours to land. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed needs-ci PRs that need a full CI run. labels Jun 18, 2022
@github-actions
Copy link
Contributor

Fast-track has been requested by @aduh95. Please 👍 to approve.

@mscdex
Copy link
Contributor

mscdex commented Jun 18, 2022

For commits/PRs that only touch tests, it's better to use the test: prefix in commit messages instead.

This renames some test filenames to align with the existing
`parallel/test-whatwg-*.js` for consistency.

According to the comment in each file, they are seemingly manually
ported from the `wpt@dom/events` tests.

Signed-off-by: Daeyeon Jeong daeyeon.dev@gmail.com
@daeyeon daeyeon force-pushed the main.renaming-220618.Sat.33ec branch from 1e03c41 to 3ca9e2e Compare June 18, 2022 11:09
@daeyeon daeyeon changed the title events: rename test-eventtarget-whatwg-*.js test: rename test-eventtarget-whatwg-*.js Jun 18, 2022
@aduh95 aduh95 merged commit f209aee into nodejs:main Jun 18, 2022
@aduh95
Copy link
Contributor

aduh95 commented Jun 18, 2022

Landed in f209aee

@daeyeon daeyeon deleted the main.renaming-220618.Sat.33ec branch June 18, 2022 23:55
targos pushed a commit that referenced this pull request Jul 12, 2022
This renames some test filenames to align with the existing
`parallel/test-whatwg-*.js` for consistency.

According to the comment in each file, they are seemingly manually
ported from the `wpt@dom/events` tests.

Signed-off-by: Daeyeon Jeong daeyeon.dev@gmail.com

PR-URL: #43467
Refs: #43461
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
targos pushed a commit that referenced this pull request Jul 18, 2022
This renames some test filenames to align with the existing
`parallel/test-whatwg-*.js` for consistency.

According to the comment in each file, they are seemingly manually
ported from the `wpt@dom/events` tests.

Signed-off-by: Daeyeon Jeong daeyeon.dev@gmail.com

PR-URL: #43467
Refs: #43461
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
targos pushed a commit that referenced this pull request Jul 31, 2022
This renames some test filenames to align with the existing
`parallel/test-whatwg-*.js` for consistency.

According to the comment in each file, they are seemingly manually
ported from the `wpt@dom/events` tests.

Signed-off-by: Daeyeon Jeong daeyeon.dev@gmail.com

PR-URL: #43467
Refs: #43461
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
guangwong pushed a commit to noslate-project/node that referenced this pull request Oct 10, 2022
This renames some test filenames to align with the existing
`parallel/test-whatwg-*.js` for consistency.

According to the comment in each file, they are seemingly manually
ported from the `wpt@dom/events` tests.

Signed-off-by: Daeyeon Jeong daeyeon.dev@gmail.com

PR-URL: nodejs/node#43467
Refs: nodejs/node#43461
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. fast-track PRs that do not need to wait for 48 hours to land. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants