1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: dont need to remove nonexistent directory

Browse Source 
the TempDirection is empty , so tempdir/test1 and tempdir/test2
are nonexistent. we dont need to delete nonexistent directories.

PR-URL: https://github.com/nodejs/node/pull/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>
This commit is contained in:
buji 2017-11-19 00:33:07 +08:00 committed by James M Snell
parent a68b8b708f
commit fdbee334cd
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
test/parallel/test-fs-mkdir.js
View File

@ -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);
@ -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);