Skip to content

tools: lint new WPTRunner() paths for no duplicates #61618

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.

Sign up for GitHub

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

Jump to bottom
Draft
panva wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

tools: lint new WPTRunner() paths for no duplicates #61618

panva wants to merge 1 commit into from
+108 −0

Conversation

@panva
Copy link
Member

@panva panva commented Feb 1, 2026

A lint rule to prevent future issues of duplicate wpt tests

Refs: #61617
Refs: #61316 (comment)

Local result:

➜  node git:(main) ✗ make lint-js    
Running JS linter...

/node/test/wpt/test-webidl.js
  6:30  error  WPTRunner path 'webidl' overlaps with 'webidl/ecmascript-binding/es-exceptions' in /node/test/wpt/test-domexception.js. One path is a subset of the other, which would run the same tests multiple times  node-core/no-duplicate-wpt-runner-paths

✖ 1 problem (1 error, 0 warnings)

@panva panva requested a review from aduh95 February 1, 2026 16:14
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. labels Feb 1, 2026
@panva
Copy link
Member Author

panva commented Feb 1, 2026

Draft until #61617 lands

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aduh95 aduh95 Awaiting requested review from aduh95

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@panva @nodejs-github-bot