require()ing a file shouldn't automatically make it CJS

[nicolo-ribaudo]

🐛 Bug Report

When a .js file is required with require(), Jest assumes that it's a CJS file even when it is an ESM file.

To Reproduce

See repo below

Expected behavior

It should log ERROR! [ERR_REQUIRE_ESM] Must use import to load ES Module: because esm-dep.js is an ESM file, but it logs IMPORTED! { foo: 2 }.

Link to repl or repo (highly encouraged)

https://github.com/nicolo-ribaudo/jest-bug-require-esm

Note that package.json contains "type": "module".

envinfo

  System:
    OS: Linux 5.8 Ubuntu 20.04.2 LTS (Focal Fossa)
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
  Binaries:
    Node: 14.16.0 - /usr/local/bin/node
    Yarn: 1.22.5 - /usr/bin/yarn
    npm: 6.14.11 - /usr/local/bin/npm
  npmPackages:
    jest: ^27.0.0-next.6 => 27.0.0-next.6 

Ref #9430 (comment)

[SimenB]

Good catch 👍 I'll fix some time during Easter 🙂

[nicolo-ribaudo]

I see that Easter is sooner than expected 😛
Thanks a lot!

[SimenB]

🐣

[SimenB]

@nicolo-ribaudo released in 27.0.0-next.7

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.