test(integrations/import): align fixture to ESLint ^9 #2931
Conversation
|
e090489 to
dd29176
Compare
| @@ -35,6 +35,9 @@ describe('Integration with eslint-plugin-import', () => { | |||
| return | |||
| } | |||
|
|
|||
| cp.execSync(`${ESLINT} a.vue`, { cwd: PLUGIN_DIR, stdio: 'inherit' }) | |||
| cp.execSync(`${ESLINT} --no-config-lookup a.vue`, { | |||
There was a problem hiding this comment.
Previously, the integration fixture used ESLint v7 and relied solely on legacy .eslintrc.* resolution, so it didn’t look up the repo‑root flat config (eslint.config.mjs).
As a result, the import of 'eslint/config' (introduced with ESLint v9) wasn’t invoked, and the error did not occur.
As a mitigation, this PR adds --no-config-lookup to the fixture’s ESLint invocation so it won’t traverse to the repo root and load eslint.config.mjs.
This isolates the fixture, avoids importing 'eslint/config', and prevents ERR_PACKAGE_PATH_NOT_EXPORTED in CI (including the ESLint v8 matrix job).
Update tests/integrations/eslint-plugin-import/package.json to use ESLint ^9 (or ^8) and eslint-plugin-import ^2.31.0, matching the plugin’s peerDependencies (^8 || ^9).
Test-only; no user-facing changes.
closed #2930
Linked to: #2916