test(linter/plugins): conformance tester keep other module exports when mocking RuleTester modules

[overlookmotel]

When mocking modules containing RuleTester, don't replace the whole module, just alter the RuleTester property within it. e.g. when mocking the RuleTester export of eslint package, leave all package's other exports unchanged.

[overlookmotel]

• test(linter/plugins): add eslint-plugin-sonarjs to conformance tests #18646
• test(linter/plugins): conformance tester shorten filename in test cases in snapshots #18682
• test(linter/plugins): conformance tester identify known parsers by parseForESLint function #18680
• test(linter/plugins): conformance tester keep other module exports when mocking RuleTester modules #18679 👈 (View in Graphite)
• test(linter/plugins): conformance tester catch errors in describe callbacks #18678
• test(linter/plugins): conformance tester allow skipping tests which did not run with RuleTester #18677
• main

---

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• 0-merge - adds this PR to the back of the merge queue
• hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Pull request overview

This PR refines how the conformance runner mocks RuleTester so that when a module exposes RuleTester alongside other exports (e.g. eslint), only the RuleTester export is replaced, preserving the rest of the module’s API.

Changes:

• Redefines TestGroup.ruleTesters from a list of module specifier strings to a list of { specifier, propName } objects, allowing selective replacement of a single export or the entire module.
• Updates the conformance runner’s runGroup logic to either replace module.exports with the conformance RuleTester or to overwrite a specific property on the loaded module, depending on propName.
• Migrates the eslint and react-hooks test groups to the new ruleTesters shape so that eslint-v7/v8/v9 retain their non-RuleTester exports while still using the conformance RuleTester.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
apps/oxlint/conformance/src/index.ts	Extends TestGroup.ruleTesters to carry specifier and propName, and updates the mocking logic to selectively replace either the whole module or just a named RuleTester export.
apps/oxlint/conformance/src/groups/react_hooks.ts	Adapts the React Hooks test group to the new ruleTesters format, targeting the RuleTester property of the eslint-v7/v8/v9 modules instead of replacing those modules entirely.
apps/oxlint/conformance/src/groups/eslint.ts	Adapts the ESLint core test group to the new ruleTesters shape while still replacing the dedicated rule-tester module’s exports with the conformance RuleTester.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[graphite-app]

Merge activity

• Jan 28, 11:02 PM UTC: overlookmotel added this pull request to the Graphite merge queue.
• Jan 28, 11:10 PM UTC: Merged by the Graphite merge queue.