move array type checking from enumerator to .all

Author: bekzod

lib/es6-promise/enumerator.js

@@ -30,31 +30,23 @@ function Enumerator(Constructor, input) {
     makePromise(this.promise);
   }
 
-  if (isArray(input)) {
-    this._input     = input;
-    this.length     = input.length;
-    this._remaining = input.length;
+  this._input     = input;
+  this.length     = input.length;
+  this._remaining = input.length;
 
-    this._result = new Array(this.length);
+  this._result = new Array(this.length);
 
-    if (this.length === 0) {
+  if (this.length === 0) {
+    fulfill(this.promise, this._result);
+  } else {
+    this.length = this.length || 0;
+    this._enumerate();
+    if (this._remaining === 0) {
       fulfill(this.promise, this._result);
-    } else {
-      this.length = this.length || 0;
-      this._enumerate();
-      if (this._remaining === 0) {
-        fulfill(this.promise, this._result);
-      }
     }
-  } else {
-    reject(this.promise, validationError());
   }
 }
 
-function validationError() {
-  return new Error('Array Methods must be provided an Array');
-};
-
 Enumerator.prototype._enumerate = function() {
   let { length, _input } = this;
 

lib/es6-promise/promise/all.js

@@ -1,5 +1,8 @@
 import Enumerator from '../enumerator';
-
+import {
+  isArray,
+  isMaybeThenable
+} from '../utils';
 /**
   `Promise.all` accepts an array of promises, and returns a new promise which
   is fulfilled with an array of fulfillment values for the passed promises, or
@@ -48,5 +51,9 @@ import Enumerator from '../enumerator';
   @static
 */
 export default function all(entries) {
+  if (!isArray(entries)){
+    return new Constructor((_, reject) => reject(new TypeError('You must pass an array to all.')));
+  }
+
   return new Enumerator(this, entries).promise;
 }