fix(jest-resolve): Remove redundant realpath call

[eps1lon]

Summary

Removes a redundant call to realpathCached. resolve already calls realpath due to preserveSymlinks: false so the subsequent realpathCached call was redundant.

In test suites resolving a lot of files, realpathCache can become a real hotpath. Since resolving files is done during test-setup, it's important that resolving modules is as fast as possible. So even if we know we'll hit the cache, we should avoid calling at all if we know for certain it's just hitting the cache. Since these calls are right next to each other I don't see the danger of accidentally not resolving symlinks.

Test plan

• CI (same test failures on master)
• Patch works on internal test suite

[SimenB]

Nice 👍 I think this is leftover from before we migrated (back) to resolve

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