fix: refactor _importCoreModule so required core module is consistent

[jonasongg]

fixes #15077

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: jonasongg / name: jonasongg (851a72c, 0d7ecf5, f530b65)
• ✅ login: SimenB / name: Simen Bekkhus (c71a8c9)

[netlify]

✅ Deploy Preview for jestjs ready!

Name	Link
🔨 Latest commit	c71a8c9
🔍 Latest deploy log	https://app.netlify.com/sites/jestjs/deploys/67b82be52288a8000893497d
😎 Deploy Preview	https://deploy-preview-15517--jestjs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[SimenB]

Thanks for the PR! Mind adding a test that covers the bug as well?

[jonasongg]

@SimenB Given the original bug's flakiness, is it a good idea to test the private method _importCoreModule? Apologies for my unfamiliarity with the codebase - if you could point me in a general direction to go ahead with the testing?

[SimenB]

I was thinking we could do an integration test where we add a property to a node core module while it's loading, but I think it'd be fiddly as we cache the module, so we'd have to do something in parallel and somehow time it right. Doesn't sound like a stable test at all, so let's ship this as-is 👍

[github-actions]

This pull request 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.