I can't run the tests, perhaps a Windows issue?

[hornta]

I just cloned the repository and these commands.

• npm i
• npm test (this one failed with the output below)

PS Z:\git\eslint-plugin-import> npm test

> eslint-plugin-import@2.27.5 pretest
> linklocal

tests\files\order-redirect
resolvers\webpack
tests\files\symlinked-module
tests\files\order-redirect-scoped
utils
resolvers\node

Linked 6 dependencies

> eslint-plugin-import@2.27.5 test
> npm run tests-only


> eslint-plugin-import@2.27.5 tests-only
> npm run mocha tests/src


> eslint-plugin-import@2.27.5 mocha
> cross-env BABEL_ENV=test nyc mocha tests/src

Z:\git\eslint-plugin-import\node_modules\babel-core\lib\transformation\file\options\build-config-chain.js:154
      throw err;
      ^

SyntaxError: Z:\git\eslint-plugin-import\resolvers\webpack\.babelrc: Error while parsing JSON - Bad number at line 1 column 3 of the JSON5 data. Still to read: "./../.babelrc"
    at error (Z:\git\eslint-plugin-import\node_modules\json5\lib\json5.js:56:25)
    at number (Z:\git\eslint-plugin-import\node_modules\json5\lib\json5.js:212:17)
    at value (Z:\git\eslint-plugin-import\node_modules\json5\lib\json5.js:491:20)
    at Object.parse (Z:\git\eslint-plugin-import\node_modules\json5\lib\json5.js:508:18)
    at ConfigChainBuilder.addConfig (Z:\git\eslint-plugin-import\node_modules\babel-core\lib\transformation\file\options\build-config-chain.js:150:65)   
    at ConfigChainBuilder.findConfigs (Z:\git\eslint-plugin-import\node_modules\babel-core\lib\transformation\file\options\build-config-chain.js:96:16)  
    at buildConfigChain (Z:\git\eslint-plugin-import\node_modules\babel-core\lib\transformation\file\options\build-config-chain.js:61:13)
    at OptionManager.init (Z:\git\eslint-plugin-import\node_modules\babel-core\lib\transformation\file\options\option-manager.js:354:58)
    at compile (Z:\git\eslint-plugin-import\node_modules\babel-register\lib\node.js:103:45)
    at loader (Z:\git\eslint-plugin-import\node_modules\babel-register\lib\node.js:144:14) {
  at: 2,
  lineNumber: 1,
  columnNumber: 3
}

Node.js v18.11.0
node:events:491
      throw er; // Unhandled 'error' event
      ^

Error: spawn nyc ENOENT
    at notFoundError (Z:\git\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:11:11)
    at verifyENOENT (Z:\git\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:46:16)
    at cp.emit (Z:\git\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:33:19)
    at ChildProcess._handle.onexit (node:internal/child_process:291:12)
Emitted 'error' event on ChildProcess instance at:
    at cp.emit (Z:\git\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:36:37)
    at ChildProcess._handle.onexit (node:internal/child_process:291:12) {
  errno: 'ENOENT',
  code: 'ENOENT',
  syscall: 'spawn nyc'
}

Node.js v18.11.0

Specs:
windows 11
node 18.11.0
npm 9.4.0

[ljharb]

That seems to be coming from inside of babel. What does resolvers\webpack\.babelrc have in it?

[hornta]

It has ../../.babelrc. I've no idea why it isn't json but I changed it to { "extends": "../../.babelrc" } which made that particular error go away and now I'm getting this output from running the tests.

PS Z:\git\eslint-plugin-import> npm test     

> eslint-plugin-import@2.27.5 pretest
> linklocal

tests\files\order-redirect
resolvers\webpack
tests\files\symlinked-module
tests\files\order-redirect-scoped
utils
resolvers\node

Linked 6 dependencies

> eslint-plugin-import@2.27.5 test
> npm run tests-only


> eslint-plugin-import@2.27.5 tests-only
> npm run mocha tests/src


> eslint-plugin-import@2.27.5 mocha
> cross-env BABEL_ENV=test nyc mocha tests/src

node:internal/modules/cjs/loader:998
  throw err;
  ^

Error: Cannot find module 'Z:\git\eslint-plugin-import\node'
    at Module._resolveFilename (node:internal/modules/cjs/loader:995:15)
    at Module._load (node:internal/modules/cjs/loader:841:27)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:23:47 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}

Node.js v18.11.0

=============================== Coverage summary ===============================
Statements   : 0% ( 0/5935 )
Branches     : 0% ( 0/3834 )
Functions    : 0% ( 0/821 )
Lines        : 0% ( 0/3653 )
================================================================================
----------------------------------------|----------|----------|----------|----------|-------------------|
File                                    |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
----------------------------------------|----------|----------|----------|----------|-------------------|
All files                               |        0 |        0 |        0 |        0 |                   |
 config                                 |        0 |      100 |      100 |        0 |                   |
  electron.js                           |        0 |      100 |      100 |        0 |                 4 |
  errors.js                             |        0 |      100 |      100 |        0 |                 6 |
  react-native.js                       |        0 |      100 |      100 |        0 |                 4 |
  react.js                              |        0 |      100 |      100 |        0 |                 8 |
  recommended.js                        |        0 |      100 |      100 |        0 |                 5 |
  stage-0.js                            |        0 |      100 |      100 |        0 |                 7 |
  typescript.js                         |        0 |      100 |      100 |        0 |              8,10 |
  warnings.js                           |        0 |      100 |      100 |        0 |                 5 |
 resolvers/node                         |        0 |        0 |        0 |        0 |                   |
  index.js                              |        0 |        0 |        0 |        0 |... 58,59,60,62,65 |
 resolvers/node/coverage                |        0 |        0 |        0 |        0 |                   |
  block-navigation.js                   |        0 |        0 |        0 |        0 |... 53,54,58,59,63 |
  prettify.js                           |        0 |        0 |        0 |        0 |                 1 |
  sorter.js                             |        0 |        0 |        0 |        0 |... 52,153,154,158 |
 resolvers/node/coverage/lcov-report    |        0 |        0 |        0 |        0 |                   |
  block-navigation.js                   |        0 |        0 |        0 |        0 |... 53,54,58,59,63 |
  prettify.js                           |        0 |        0 |        0 |        0 |                 1 |
  sorter.js                             |        0 |        0 |        0 |        0 |... 52,153,154,158 |
 resolvers/webpack                      |        0 |        0 |        0 |        0 |                   |
  config.js                             |        0 |      100 |      100 |        0 |                 4 |
  index.js                              |        0 |        0 |        0 |        0 |... 43,444,445,447 |
 resolvers/webpack/coverage             |        0 |        0 |        0 |        0 |                   |
  block-navigation.js                   |        0 |        0 |        0 |        0 |... 53,54,58,59,63 |
  prettify.js                           |        0 |        0 |        0 |        0 |                 1 |
  sorter.js                             |        0 |        0 |        0 |        0 |... 52,153,154,158 |
 resolvers/webpack/coverage/lcov-report |        0 |        0 |        0 |        0 |                   |
  block-navigation.js                   |        0 |        0 |        0 |        0 |... 53,54,58,59,63 |
  prettify.js                           |        0 |        0 |        0 |        0 |                 1 |
  sorter.js                             |        0 |        0 |        0 |        0 |... 52,153,154,158 |
 src                                    |        0 |        0 |        0 |        0 |                   |
  ExportMap.js                          |        0 |        0 |        0 |        0 |... 06,819,821,824 |
  docsUrl.js                            |        0 |        0 |        0 |        0 |               3,6 |
  importDeclaration.js                  |        0 |      100 |        0 |        0 |               2,3 |
  index.js                              |        0 |      100 |      100 |        0 |              1,57 |
 src/core                               |        0 |        0 |        0 |        0 |                   |
  importType.js                         |        0 |        0 |        0 |        0 |... 21,122,123,127 |
  packagePath.js                        |        0 |        0 |        0 |        0 |... 11,15,16,18,20 |
  staticRequire.js                      |        0 |        0 |        0 |        0 |                 3 |
 src/rules                              |        0 |        0 |        0 |        0 |                   |
  consistent-type-specifier-style.js    |        0 |        0 |        0 |        0 |... 00,202,204,210 |
  default.js                            |        0 |        0 |        0 |        0 |... 27,28,29,30,37 |
  dynamic-import-chunkname.js           |        0 |        0 |        0 |        0 |... 08,112,113,116 |
  export.js                             |        0 |        0 |        0 |        0 |... 34,236,237,239 |
  exports-last.js                       |        0 |        0 |        0 |        0 |... 27,30,31,32,33 |
  extensions.js                         |        0 |        0 |        0 |        0 |... 80,181,182,190 |
  first.js                              |        0 |        0 |        0 |        0 |... 30,131,136,139 |
  group-exports.js                      |        0 |        0 |        0 |        0 |... 40,141,142,152 |
  imports-first.js                      |        0 |      100 |      100 |        0 |            3,5,15 |
  max-dependencies.js                   |        0 |        0 |        0 |        0 |... 45,47,51,52,54 |
  named.js                              |        0 |        0 |        0 |        0 |... 24,127,129,135 |
  namespace.js                          |        0 |        0 |        0 |        0 |... 07,208,209,210 |
  newline-after-import.js               |        0 |        0 |        0 |        0 |... 94,195,198,205 |
  no-absolute-path.js                   |        0 |        0 |        0 |        0 |... 28,29,31,37,38 |
  no-amd.js                             |        0 |        0 |      100 |        0 |... 29,32,34,35,39 |
  no-anonymous-default-export.js        |        0 |        0 |        0 |        0 |... 99,101,105,106 |
  no-commonjs.js                        |        0 |        0 |        0 |        0 |... 26,128,130,133 |
  no-cycle.js                           |        0 |        0 |        0 |        0 |... 42,147,149,156 |
  no-default-export.js                  |        0 |        0 |        0 |        0 |... 31,32,33,34,35 |
  no-deprecated.js                      |        0 |        0 |        0 |        0 |... 27,131,132,133 |
  no-duplicates.js                      |        0 |        0 |        0 |        0 |... 46,347,348,349 |
  no-dynamic-require.js                 |        0 |        0 |        0 |        0 |... 59,60,67,68,70 |
  no-empty-named-blocks.js              |        0 |        0 |        0 |        0 |... 82,83,85,91,95 |
  no-extraneous-dependencies.js         |        0 |        0 |        0 |        0 |... 83,292,293,298 |
  no-import-module-exports.js           |        0 |        0 |        0 |        0 |... 70,74,76,79,80 |
  no-internal-modules.js                |        0 |        0 |        0 |        0 |... 28,132,139,141 |
  no-mutable-exports.js                 |        0 |        0 |        0 |        0 |... 46,47,48,49,54 |
  no-named-as-default-member.js         |        0 |        0 |        0 |        0 |... 75,77,79,80,82 |
  no-named-as-default.js                |        0 |        0 |        0 |        0 |... 27,28,31,33,40 |
  no-named-default.js                   |        0 |        0 |        0 |        0 |... 17,18,19,22,23 |
  no-named-export.js                    |        0 |        0 |        0 |        0 |... 28,29,32,33,34 |
  no-namespace.js                       |        0 |        0 |        0 |        0 |... 66,167,171,173 |
  no-nodejs-modules.js                  |        0 |        0 |        0 |        0 |6,7,11,37,38,40,41 |
  no-relative-packages.js               |        0 |        0 |        0 |        0 |... 42,47,50,56,69 |
  no-relative-parent-imports.js         |        0 |        0 |        0 |        0 |... 33,36,38,39,46 |
  no-restricted-paths.js                |        0 |        0 |        0 |        0 |... 35,236,240,241 |
  no-self-import.js                     |        0 |        0 |        0 |        0 | 11,14,15,22,35,36 |
  no-unassigned-import.js               |        0 |        0 |        0 |        0 |... 39,40,44,49,55 |
  no-unresolved.js                      |        0 |        0 |        0 |        0 |... 47,48,49,50,58 |
  no-unused-modules.js                  |        0 |        0 |        0 |        0 |... 52,953,955,956 |
  no-useless-path-segments.js           |        0 |        0 |        0 |        0 |... 30,131,135,145 |
  no-webpack-loader-syntax.js           |        0 |        0 |        0 |        0 |      5,6,10,22,23 |
  order.js                              |        0 |        0 |        0 |        0 |... 83,784,787,790 |
  prefer-default-export.js              |        0 |        0 |        0 |        0 |... 08,109,110,111 |
  unambiguous.js                        |        0 |        0 |      100 |        0 |  9,22,23,26,28,29 |
 utils                                  |        0 |        0 |        0 |        0 |                   |
  ModuleCache.js                        |        0 |        0 |        0 |        0 |... 38,43,44,47,50 |
  declaredScope.js                      |        0 |        0 |        0 |        0 |       3,5,6,7,8,9 |
  hash.js                               |        0 |        0 |        0 |        0 |... 52,54,56,58,59 |
  ignore.js                             |        0 |        0 |        0 |        0 |... 50,51,55,59,61 |
  module-require.js                     |        0 |      100 |        0 |        0 |... 20,21,24,26,30 |
  moduleVisitor.js                      |        0 |        0 |        0 |        0 |... 40,144,146,153 |
  parse.js                              |        0 |        0 |        0 |        0 |... 40,142,143,148 |
  pkgDir.js                             |        0 |        0 |        0 |        0 |      3,4,6,8,9,10 |
  pkgUp.js                              |        0 |        0 |        0 |        0 |... 47,48,51,55,56 |
  readPkgUp.js                          |        0 |        0 |        0 |        0 |... 41,42,45,46,51 |
  resolve.js                            |        0 |        0 |        0 |        0 |... 25,229,233,234 |
  unambiguous.js                        |        0 |        0 |        0 |        0 |3,5,16,17,21,28,29 |
  visit.js                              |        0 |        0 |        0 |        0 |... 17,18,19,22,23 |
----------------------------------------|----------|----------|----------|----------|-------------------|
node:events:491
      throw er; // Unhandled 'error' event
      ^

Error: spawn nyc ENOENT
    at notFoundError (Z:\git\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:11:11)
    at verifyENOENT (Z:\git\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:46:16)
    at cp.emit (Z:\git\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:33:19)
    at ChildProcess._handle.onexit (node:internal/child_process:291:12)
Emitted 'error' event on ChildProcess instance at:
    at cp.emit (Z:\git\eslint-plugin-import\node_modules\cross-spawn\lib\enoent.js:36:37)
    at ChildProcess._handle.onexit (node:internal/child_process:291:12) {
  errno: 'ENOENT',
  code: 'ENOENT',
  syscall: 'spawn nyc'
}

Node.js v18.11.0

[hornta]

Update:
I noticed that the babelrc is a symbolic link so I enabled it in windows and recloned the repo and than ran npm test but it gives the same output as above.

[ljharb]

ah yes, that's indeed because it's a unix-style symlink.

I suspect that if you replace the symlink with a copy of the file it's targeting, that it should work?

[silverwind]

These Windows CI errors may be related:

https://ci.appveyor.com/project/ljharb/eslint-plugin-import/builds/46821162/job/gort0xlqll93sbix

Error: Cannot find module 'semver'
Require stack:
- /mnt/c/projects/eslint-plugin-import/tests/src/cli.js
- /mnt/c/projects/eslint-plugin-import/node_modules/mocha/lib/mocha.js
- /mnt/c/projects/eslint-plugin-import/node_modules/mocha/index.js
- /mnt/c/projects/eslint-plugin-import/node_modules/mocha/bin/_mocha
- ```

[ljharb]

I can’t reproduce those errors in my local Windows, so I’m not sure if they are related.

[cnshenj]

This is an nyc issue: istanbuljs/nyc#1205.
Simple solution: update nyc to 15.x in eslint-plugin-import.

[ljharb]

That's not an option, unfortunately, because nyc 15 drops node versions we support.

[soryy708]

I'm also experiencing this issue, on Windows 11 Home version 22H2 (build 22621.2715).
Is there a workaround?

[ljharb]

@soryy708 #2760 (comment)

[rotu]

I'm experiencing this too. I think it's time to drop support for node 16.

Apparently that's not acceptable per #3008 (review)

Absolutely not. This would be a breaking change, and platform support has zero bearing whatsoever on what node versions a library should support.

What version[s] does nyc drop that this project needs to still support?

Edit: answered here:

We support down to eslint 2, and we support down to node 4 (arguably we should support down to node 0.10, but we haven't prioritized that and nobody's asked for it)

Additionally, breaking changes are the most costly thing any package can impose on the ecosystem, and "dropping a node version" has a very small amount of benefit, so there'd need to be actually good reasons to do a breaking change before dropping a node version happened.

[ljharb]

The earliest nyc version that avoids these problems on windows isn't compatible with this project.

[rotu]

What versions of Node must this project be compatible with? On Windows it looks like we monkey-patch the nyc dependency to sidestep this:

eslint-plugin-import/.github/workflows/native-wsl.yml

Lines 90 to 92 in a3a7176

	- name: install the latest version of nyc
	if: matrix.configuration == 'native'
	run: npm install nyc@latest --no-save

Maybe the version requirement should instead be changed from "nyc": "^11.9.0" to "nyc": ">=11.9.0"?

[ljharb]

Unfortunately that would install nyc latest, on versions of node that can't support it.