chore: Fixed tests and Update all dependencies (#43)

* Fixed unit tests after updated documentation.js on new version

* All tests in test.js file and not necessary include all folder.

* Add gitignore for node_modules folder.

* Prettier index.js file

* Update all dependencies

* Replaced function on arrow func for Foreach and use a chain of promises

Author: anthony-redFox

.gitignore

@@ -0,0 +1 @@
+node_modules

lib/index.js

@@ -86,35 +86,29 @@ module.exports = function(format, options, formatterOptions) {
       cb();
     },
     function(cb) {
-      documentation.build(
-        files.map(function(file) {
-          return file.path;
-        }),
-        options,
-        function(err, comments) {
-          formatter(
-            comments,
-            formatterOptions,
-            function(err, output) {
-              if (format === 'json' || format === 'md') {
-                this.push(
-                  new File({
-                    path: options.filename || 'API.' + format,
-                    contents: new Buffer(output)
-                  })
-                );
-              } else if (format === 'html') {
-                output.forEach(
-                  function(file) {
-                    this.push(file);
-                  }.bind(this)
-                );
-              }
-              cb();
-            }.bind(this)
-          );
-        }.bind(this)
-      );
+      documentation
+        .build(
+          files.map(function(file) {
+            return file.path;
+          }),
+          options
+        )
+        .then(comments => {
+          return formatter(comments, formatterOptions);
+        })
+        .then(output => {
+          if (format === 'json' || format === 'md') {
+            this.push(
+              new File({
+                path: options.filename || 'API.' + format,
+                contents: new Buffer(output)
+              })
+            );
+          } else if (format === 'html') {
+            output.forEach(file => this.push(file));
+          }
+          cb();
+        });
     }
   );
 };

package.json

@@ -4,19 +4,23 @@
   "version": "3.2.0",
   "author": "Tom MacWright",
   "dependencies": {
-    "documentation": "4.0.0-rc.0",
-    "through2": "^2.0.1",
-    "vinyl": "^2.0.0"
+    "through2": "^2.0.3",
+    "vinyl": "^2.1.0"
+  },
+  "peerDependencies": {
+    "documentation": ">=4.0.0",
+    "gulp": ">=3.0.0"
   },
   "devDependencies": {
-    "concat-stream": "^1.4.8",
+    "documentation": "^5.1.1",
+    "gulp": "^3.9.1",
+    "concat-stream": "^1.6.0",
     "cz-conventional-changelog": "^2.0.0",
-    "gulp": "^3.8.11",
-    "husky": "^0.13.3",
-    "lint-staged": "^3.4.0",
-    "prettier": "^1.0.0",
-    "standard-version": "^4.0.0",
-    "tap": "10.3.4"
+    "husky": "^0.14.3",
+    "lint-staged": "^4.0.3",
+    "prettier": "^1.5.3",
+    "standard-version": "^4.2.0",
+    "tap": "10.7.1"
   },
   "keywords": [
     "documentation",
@@ -35,7 +39,7 @@
     "precommit": "lint-staged --verbose",
     "format": "prettier --write '{lib,test}/**/*.js' --single-quote",
     "docs": "documentation readme index.js --section=API",
-    "test": "tap test"
+    "test": "tap test/test.js"
   },
   "config": {
     "commitizen": {

test/test.js

@@ -45,7 +45,7 @@ test('gulp-documentation html', function(t) {
     .pipe(documentation('html'))
     .pipe(
       concat(function(d) {
-        t.equal(d.length, 34);
+        t.ok(d.length === 34 || d.length === 36);
         t.end();
       })
     );