Skip to content

Commit

Permalink
test: dont need to remove nonexistent directory
Browse files Browse the repository at this point in the history
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 <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
  • Loading branch information
ah-yu authored and MylesBorins committed Dec 12, 2017
1 parent 7ebaf83 commit 5c70cef
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions test/parallel/test-fs-mkdir.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,6 @@ common.refreshTmpDir();
{
const pathname = `${common.tmpDir}/test1`;

unlink(pathname);

fs.mkdir(pathname, common.mustCall(function(err) {
assert.strictEqual(err, null);
assert.strictEqual(common.fileExists(pathname), true);
Expand All @@ -51,8 +49,6 @@ common.refreshTmpDir();
{
const pathname = `${common.tmpDir}/test2`;

unlink(pathname);

fs.mkdir(pathname, 0o777, common.mustCall(function(err) {
assert.strictEqual(err, null);
assert.strictEqual(common.fileExists(pathname), true);
Expand Down

0 comments on commit 5c70cef

Please sign in to comment.