[@jest/globals] Identifier 'jest' has already been declared

🐛 Bug Report

I, and probably many other people, love the idea of @jest/globals. Thanks again 👍 Obviously, I just tried 😃 The FE part, ES modules transpiled with babel, seems to be just fine. The BE part is probably not expecting to import/require jest variable from @jest/globals

To Reproduce

const {jest} = require('@jest/globals')

Run the test

Error: SyntaxError: Identifier 'jest' has already been declared

Expected behavior

The test will pass the same way as without @jest/globals

Link to repl or repo

see the test file in https://repl.it/repls/LastingRepentantRuntimes

[SimenB]

If I understand this correctly, the same would happen with the test file

Yup. Same in plain node

// file.js
const require = 42;

$ node file.js
/Users/simen/repos/infra/file.js:1
const require = 42
      ^

SyntaxError: Identifier 'require' has already been declared

Maybe we need to wrap in another function?

We’ve stopped manually wrapping it in a function, which solves so many issues with source maps and code coverage (there’s no offset) - we use vm.compileFunction instead. Or rather, we use it when available falling back to manual wrapper and vm.Script, and I’m planning to remove vm.Script in Jest 26. I’d really prefer some solution that didn’t require an extra function wrapper

[github-actions[bot]]

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.