Skip to content

Fix source of open FSREQCALLBACK handles - #342

Merged
tschaub merged 2 commits into
tschaub:mainfrom
jloleysens:fix/call-original-fsBinding-open
Nov 8, 2021
Merged

Fix source of open FSREQCALLBACK handles #342
tschaub merged 2 commits into
tschaub:mainfrom
jloleysens:fix/call-original-fsBinding-open

Conversation

@jloleysens

@jloleysens jloleysens commented Nov 8, 2021

Copy link
Copy Markdown
Contributor

Closes #341

When collecting the prototype for ReadFileContext the original fsBinding.open should still be called so that FS read operations can be handled as usual.

Comment thread lib/readfilecontext.js Outdated
@3cp

3cp commented Nov 8, 2021

Copy link
Copy Markdown
Collaborator

@tschaub LGTM.

@maximelafarie

Copy link
Copy Markdown

@3cp @tschaub Man, this is just in time! Will you do a patch/minor release as soon as it's merged?

@tschaub
tschaub merged commit abd285a into tschaub:main Nov 8, 2021
@tschaub

tschaub commented Nov 8, 2021

Copy link
Copy Markdown
Owner

Thanks for the fix, @jloleysens. Published in mock-fs@5.1.2.

@jloleysens
jloleysens deleted the fix/call-original-fsBinding-open branch November 9, 2021 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Using mock-fs with Jest results in open handles

4 participants