Skip to content

Commit 5c70cef

Browse files
ah-yuMylesBorins
authored andcommitted
test: dont need to remove nonexistent directory
the TempDirection is empty , so tempdir/test1 and tempdir/test2 are nonexistent. we dont need to delete nonexistent directories. PR-URL: #17119 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Daniel Bevenius <[email protected]>
1 parent 7ebaf83 commit 5c70cef

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

test/parallel/test-fs-mkdir.js

-4
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,6 @@ common.refreshTmpDir();
3636
{
3737
const pathname = `${common.tmpDir}/test1`;
3838

39-
unlink(pathname);
40-
4139
fs.mkdir(pathname, common.mustCall(function(err) {
4240
assert.strictEqual(err, null);
4341
assert.strictEqual(common.fileExists(pathname), true);
@@ -51,8 +49,6 @@ common.refreshTmpDir();
5149
{
5250
const pathname = `${common.tmpDir}/test2`;
5351

54-
unlink(pathname);
55-
5652
fs.mkdir(pathname, 0o777, common.mustCall(function(err) {
5753
assert.strictEqual(err, null);
5854
assert.strictEqual(common.fileExists(pathname), true);

0 commit comments

Comments
 (0)