Merge pull request #45 from tmont/master
Handle errors for non-existent directories in readdirRecursive()
This commit is contained in:
commit
c8fc1258b9
3 changed files with 17 additions and 6 deletions
1
.gitignore
vendored
1
.gitignore
vendored
|
|
@ -1,2 +1,3 @@
|
||||||
node_modules
|
node_modules
|
||||||
npm-debug.log
|
npm-debug.log
|
||||||
|
.idea
|
||||||
|
|
|
||||||
|
|
@ -71,15 +71,16 @@ exports.readdirRecursive = function(baseDir, fn) {
|
||||||
var waitCount = 0;
|
var waitCount = 0;
|
||||||
|
|
||||||
function readdirRecursive(curDir) {
|
function readdirRecursive(curDir) {
|
||||||
var files = [],
|
var prependcurDir = function(fname){
|
||||||
curFiles,
|
return _path.join(curDir, fname);
|
||||||
nextDirs,
|
};
|
||||||
prependcurDir = function(fname){
|
|
||||||
return _path.join(curDir, fname);
|
|
||||||
};
|
|
||||||
|
|
||||||
waitCount++;
|
waitCount++;
|
||||||
fs.readdir(curDir, function(e, curFiles) {
|
fs.readdir(curDir, function(e, curFiles) {
|
||||||
|
if (e) {
|
||||||
|
fn(e);
|
||||||
|
return;
|
||||||
|
}
|
||||||
waitCount--;
|
waitCount--;
|
||||||
|
|
||||||
curFiles = curFiles.map(prependcurDir);
|
curFiles = curFiles.map(prependcurDir);
|
||||||
|
|
|
||||||
|
|
@ -46,6 +46,15 @@ module.exports = testCase({
|
||||||
checkResult(test, allFiles);
|
checkResult(test, allFiles);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
},
|
||||||
|
|
||||||
|
test_readdirRecursiveWithNonExistentDirectory: function(test) {
|
||||||
|
wrench.readdirRecursive('', function (e, files) {
|
||||||
|
test.ok(e);
|
||||||
|
test.equal(e.code, 'ENOENT');
|
||||||
|
test.equal(files, null);
|
||||||
|
test.done();
|
||||||
|
});
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|
|
||||||
Reference in a new issue