feat: filter add dir as param (#8)

Author: dead-horse

.travis.yml

@@ -6,5 +6,8 @@ node_js:
 - "2"
 - "3"
 - "4"
+- "6"
+- "8"
+- "9"
 script: "npm run-script test-travis"
 after_script: "npm install coveralls@2 && cat ./coverage/lcov.info | coveralls"

README.md

@@ -33,11 +33,11 @@ read(__dirname) === [
 
 ### read(root [, filter])
 
-`root` is the directory you wish to scan. `filter` is an optional filter for the files. By default, filter is:
+`root` is the directory you wish to scan. `filter` is an optional filter for the files with three params(name, index, dir). By default, filter is:
 
 ```js
-function (x) {
-  return x[0] !== '.'
+function (name) {
+  return name[0] !== '.'
 }
 ```
 

index.js

@@ -12,7 +12,9 @@ function read(root, filter, files, prefix) {
   if (!fs.existsSync(dir)) return files
   if (fs.statSync(dir).isDirectory())
     fs.readdirSync(dir)
-    .filter(filter)
+    .filter(function (name, index) {
+      return filter(name, index, dir)
+    })
     .forEach(function (name) {
       read(root, filter, files, path.join(prefix, name))
     })
@@ -25,4 +27,3 @@ function read(root, filter, files, prefix) {
 function noDotFiles(x) {
   return x[0] !== '.'
 }
-

package.json

@@ -9,7 +9,7 @@
   },
   "devDependencies": {
     "istanbul": "0",
-    "mocha": "*",
+    "mocha": "3",
     "should": "*"
   },
   "author": {

test/test.js

@@ -15,7 +15,7 @@ describe('fs.readdirSyncRecursive()', function () {
 
   it('should work at the root with a filter', function () {
     var files = read(path.join(__dirname, '..'), function (name) {
-      return name[0] !== '.' && name !== 'node_modules' && name !== 'coverage'
+      return name[0] !== '.' && name !== 'node_modules' && name !== 'coverage' && name !== 'package-lock.json'
     })
 
     files.length.should.equal(5)
@@ -29,6 +29,21 @@ describe('fs.readdirSyncRecursive()', function () {
 
   })
 
+  it('should filter pass dir', function () {
+    var files = read(path.join(__dirname, '..'), function (name, index, dir) {
+      return name[0] !== '.' && name !== 'node_modules' && name !== 'coverage' && dir !== __dirname && name !== 'package-lock.json'
+    })
+
+    files.length.should.equal(4)
+    files.sort().should.eql([
+      'index.js',
+      'LICENSE',
+      'package.json',
+      'README.md'
+    ].sort())
+
+  })
+
   it('should work with the symlinked file', function () {
     try {
       var linkname = __filename + '-link'
@@ -69,7 +84,7 @@ describe('fs.readdirSyncRecursive()', function () {
       fs.symlinkSync(path.join(__dirname, '..'), linkname, 'dir')
 
       var files = read(linkname, function (name) {
-        return name[0] !== '.' && name !== 'node_modules' && name !== 'coverage'
+        return name[0] !== '.' && name !== 'node_modules' && name !== 'coverage' && name !== 'package-lock.json'
       })
 
       files.length.should.equal(5)