fix: resolve bundled modules from correct location

[SimenB]

Summary

Take 2 at fixing #11452. This time by passing in require.resolve from the modules that do the require so that it looks like it's e.g. jest-config doing the require.resolve rather than jest-resolve, thus satisfying the dependency

Test plan

🤷

[SimenB]

@merceyz sorry to ping you, but do you have any tips for how to make the script I added as a "test" work? Using just portal:/path/to/jest makes yarn install the rest of the dependencies from npm rather that resolve locally. I tried adding resolutions for all packages, but that didn't help.

Using link instead of portal means the binary script is not added

[codecov-commenter]

Codecov Report

Merging #11493 (d007a65) into master (82d1a1a) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #11493      +/-   ##
==========================================
- Coverage   68.95%   68.94%   -0.02%     
==========================================
  Files         312      312              
  Lines       16310    16314       +4     
  Branches     4722     4726       +4     
==========================================
  Hits        11247    11247              
- Misses       5035     5039       +4     
  Partials       28       28              

Impacted Files	Coverage Δ
packages/jest-config/src/normalize.ts	89.09% <ø> (ø)
packages/jest-resolve/src/utils.ts	24.32% <0.00%> (-2.95%)	⬇️
packages/jest-runner/src/runTest.ts	3.47% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82d1a1a...d007a65. Read the comment docs.

[SimenB]

Moved test to #11513

[merceyz]

@SimenB No worries, responded in #11513 (comment)

[merceyz]

Instead of passing the require function around, wouldn't passing the filepath instead be better?
jest-resolve could then do

require.resolve(module, { paths: [filepath] })`

[SimenB]

Passing in __filename? That can probably work as well!

[merceyz]

Depends on where it should be resolved relative to, but yeah

[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.