jest-resolve - wrap realpath with try catch block

[aliaksandr-yermalayeu]

Summary

Fixes #8291

Test plan

Minor changes in snapshots

[facebook-github-bot]

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need the corporate CLA signed.

If you have received this in error or have any questions, please contact us at [email protected]. Thanks!

[codecov-io]

Codecov Report

Merging #8292 into master will increase coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #8292      +/-   ##
==========================================
+ Coverage   63.85%   63.85%   +<.01%     
==========================================
  Files         274      274              
  Lines       11389    11390       +1     
  Branches     2771     2770       -1     
==========================================
+ Hits         7272     7273       +1     
  Misses       3498     3498              
  Partials      619      619

Impacted Files	Coverage Δ
packages/jest-resolve/src/index.ts	44.6% <50%> (+0.4%)	⬆️

---

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 ccf90c0...c2a3406. Read the comment docs.

[thymikee]

Looks like the snapshots are still off, could you rebase it once again to the latest master?

[aliaksandr-yermalayeu]

@thymikee, I updated snapshots, could please take one more look? One check which is failing seems not related to my changes.

[thymikee]

LGTM. Can you add a changelog entry?

[aliaksandr-yermalayeu]

@thymikee, updated changelog, does it looks correct?

[aliaksandr-yermalayeu]

@thymikee, thanks for review!

[jeysal]

LGTM, but if I understand @SimenB's comment in #8291 correctly, the preferred solution would be to only change it in SimenB/realpath-native#39?

[aliaksandr-yermalayeu]

Correct, but the problem is that the owners of realpath may not be really interested in this change. Three months have passed, but it doesn't seem to be getting anywhere.

Here is another possible solution that may probably work in this case:
#8530

[thymikee]

@SimenB rebased, this should be good to review again

[SimenB]

@thymikee #9952

[thymikee]

Can we close this then?

[SimenB]

Should be fixed with the above linked PR

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