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 19, 2017
1 parent 5f522a1 commit 7c57ab7
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 @@ -15,8 +15,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 @@ -30,8 +28,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 7c57ab7

Please sign in to comment.