Skip to content

test: add test for Module._stat #44713

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

Merged
nodejs-github-bot merged 6 commits into from
Oct 25, 2022
+113 −0
Merged

test: add test for Module._stat #44713

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
389ec8c
test: add test for Module._stat
RaisinTen Sep 18, 2022
b135612
test: use common.expectWarning()
RaisinTen Sep 19, 2022
69a23d7
fixup! test: add comment
RaisinTen Oct 22, 2022
f40bbb2
fixup! test: add comment
RaisinTen Oct 22, 2022
1550841
fixup! test: add comment
RaisinTen Oct 22, 2022
c19ddb9
fixup! test: add a more isolated Module._stat test
RaisinTen Oct 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixup! test: add comment
  • Loading branch information
@RaisinTen
RaisinTen committed Oct 23, 2022
commit 69a23d7cb91539641ce0dc1d8782b6d9110128f4
1 change: 1 addition & 0 deletions test/parallel/test-vfs.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ common.expectWarning(
process.on('warning', common.mustCall());

const originalStat = Module._stat;
TODO(aduh95): We'd like to have a better way to achieve this without monkey-patching.
Comment thread
RaisinTen marked this conversation as resolved.
Outdated
Module._stat = function(filename) {
Comment thread
RaisinTen marked this conversation as resolved.
if (!filename.startsWith(process.execPath)) {
return originalStat(filename);
Expand Down