Merge Symbol.asyncIterator into class, add full-test command for travis

Author: basicdays

.travis.yml

@@ -4,3 +4,5 @@ node_js:
   - '6'
   - '4'
   - '4.5'
+install: make --no-builtin-rules --jobs=4
+script: make --no-builtin-rules full-test

Makefile

@@ -48,6 +48,8 @@ build: node_modules ${build_files} ${build_asset_files} ${build_package_files}
 
 # Tasks
 
+full-test: lint test cover
+
 lint:
 	eslint lib
 

lib/stream-async-iterator.js

@@ -87,6 +87,13 @@ export default class StreamAsyncIterator {
         stream.once('end', handleStreamEnd);
     }
 
+    /**
+     * @returns {StreamAsyncIterator} - this
+     */
+    [Symbol.asyncIterator]() {
+        return this;
+    }
+
     /**
      * Returns the next iteration of data. Rejects if the stream errored out.
      * @returns {Promise<StreamAsyncIterator~Iteration>}
@@ -149,9 +156,3 @@ export default class StreamAsyncIterator {
         })
     }
 }
-if (typeof Symbol.asyncIterator === 'symbol') {
-    Object.defineProperty(StreamAsyncIterator.prototype, Symbol.asyncIterator, {
-        configurable: true,
-        value: function() { return this; }
-    });
-}

package.json

@@ -4,7 +4,7 @@
   "description": "ES async interator wrapper for node streams",
   "main": "stream-async-iterator.js",
   "scripts": {
-    "test": "make test"
+    "test": "make full-test"
   },
   "repository": {
     "type": "git",