Skip to content

Commit e9ace7e

Browse files
TrottMylesBorins
authored andcommitted
test: do not open fixture files for writing
test-fs-fsync makes a copy of a fixture file, but then doesn't do anything with it and instead operates on the original fixture file. This appears to be in error, and this change fixes that. PR-URL: #17808 Reviewed-By: Jon Moss <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 6994559 commit e9ace7e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/parallel/test-fs-fsync.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ const fileTemp = path.join(common.tmpDir, 'a.js');
3434
common.refreshTmpDir();
3535
fs.copyFileSync(fileFixture, fileTemp);
3636

37-
fs.open(fileFixture, 'a', 0o777, common.mustCall(function(err, fd) {
37+
fs.open(fileTemp, 'a', 0o777, common.mustCall(function(err, fd) {
3838
assert.ifError(err);
3939

4040
fs.fdatasyncSync(fd);

0 commit comments

Comments
 (0)