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.

Sign up for GitHub

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

Jump to bottom

Additional fix for #4444 to prevent errors on windows #11423

Merged
merged 4 commits into from Sep 28, 2022
Merged

Additional fix for #4444 to prevent errors on windows #11423

Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2c2d59b
Additional fix for #4444 to prevent errors on windows
lukeapage May 19, 2021
adb4aca
Merge branch 'main' into patch-3
SimenB Sep 16, 2022
37697cf
Update CHANGELOG.md
SimenB Sep 28, 2022
6a43b1d
Merge branch 'main' into patch-3
SimenB Sep 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 6 additions & 0 deletions packages/jest-transform/src/ScriptTransformer.ts
View file
Open in desktop
Expand Up @@ -946,6 +946,12 @@ const readCacheFile = (cachePath: string): string | null => {
try {
fileData = fs.readFileSync(cachePath, 'utf8');
} catch (e: any) {
// on windows write-file-atomic is not atomic which can
// result in this error
if (e.code === 'ENOENT' && process.platform === 'win32') {
return null;
}

e.message = `jest: failed to read cache file: ${cachePath}\nFailure message: ${e.message}`;
removeFile(cachePath);
Copy link
Member

@SimenB SimenB Sep 16, 2022 •
edited

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we call this as well?

I guess not since ENOENT would probably be thrown again

throw e;
Expand Down