Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[heft-jest] Fix node-modules-symlink-resolver #5013

Merged
merged 1 commit into from
Nov 23, 2024

Conversation

dmichon-msft
Copy link
Contributor

@dmichon-msft dmichon-msft commented Nov 22, 2024

Summary

For Jest, specifically, the realpathSync function inside of jest-resolve is supposed to return the input path if part of the path does not exist.

Details

Since jest-resolve's realpathSync internally calls jest-util's tryRealpathSync, and the latter is already overridden, this PR just removes the separate override in jest-resolve in favor of relying on tryRealpathSync being overridden centrally.

Note that this ends up caching all the paths twice, but that's less of a concern than the syscalls this feature is avoiding.

How it was tested

Local test invocation.

Impacted documentation

None

@iclanton iclanton enabled auto-merge (squash) November 23, 2024 00:10
@iclanton iclanton merged commit 871fdf2 into microsoft:main Nov 23, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed
Development

Successfully merging this pull request may close these issues.

2 participants