-
Notifications
You must be signed in to change notification settings - Fork 30k
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
test: replace fixturesDir to use fixtures module #15831
test: replace fixturesDir to use fixtures module #15831
Conversation
test/parallel/test-fs-symlink.js
Outdated
@@ -34,7 +35,7 @@ let fileTime; | |||
common.refreshTmpDir(); | |||
|
|||
// test creating and reading symbolic link | |||
const linkData = path.join(common.fixturesDir, '/cycles/root.js'); | |||
const linkData = path.join(fixtures.fixturesDir, '/cycles/root.js'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could be using method fixtures.path
instead of path.join
.
bf1eacd
to
9c89056
Compare
@pawelgolda good catch. I updated the PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thanks for making Node.js better!
@gireeshpunathil I rebased again with the upstream master and all the tests are passing locally. Maybe the CI job will pass now? |
It seems like something went wrong while rebasing. There are lots of unrelated commits in here. |
1b8f1a6
to
19f5b96
Compare
@BridgeAR, not sure how that happened. Should be fixed now. |
PR-URL: #15831 iReviewed-By: Ryan Graham <r.m.graham@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Landed in e04e927 |
PR-URL: nodejs/node#15831 iReviewed-By: Ryan Graham <r.m.graham@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
PR-URL: #15831 iReviewed-By: Ryan Graham <r.m.graham@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Change usage of fixturesDir to be from the common.fixtures module instead of common.
Checklist
make -j4 test
(UNIX), orvcbuild test
(Windows) passesAffected core subsystem(s)
test