Fix bug in isArray empty

taylorhakes · taylorhakes · commit 03f6f9a0d19e · 2019-06-15T00:15:38.000-04:00
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "promise-polyfill",
-  "version": "8.1.1",
+  "version": "8.1.2",
   "description": "Lightweight promise polyfill. A+ compliant",
   "main": "lib/index.js",
   "module": "src/index.js",
diff --git a/src/index.js b/src/index.js
@@ -5,7 +5,7 @@ import promiseFinally from './finally';
 var setTimeoutFunc = setTimeout;
 
 function isArray(x) {
-  return Boolean(x && x.length);
+  return Boolean(x && typeof x.length !== 'undefined');
 }
 
 function noop() {}
diff --git a/test/promise.js b/test/promise.js
@@ -373,7 +373,7 @@ describe('Promise', function() {
         }
       );
     });
-    it('throws on multiple promises', function() {
+    it('works on multiple resolved promises', function() {
       return Promise.all([Promise.resolve(), Promise.resolve()]).then(
         function() {
           assert.ok(true);
@@ -383,6 +383,16 @@ describe('Promise', function() {
         }
       );
     });
+    it('works on empty array', function() {
+      return Promise.all([]).then(
+        function(arr) {
+          assert.ok(arr.length === 0);
+        },
+        function() {
+          assert.fail();
+        }
+      );
+    });
   });
   describe('Promise.race', function() {
     it('throws on implicit undefined', function() {
@@ -499,5 +509,9 @@ describe('Promise', function() {
         }
       );
     });
+    it('works on empty array', function() {
+      var prom = Promise.race([]);
+      return assert(prom instanceof Promise);
+    });
   });
 });

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "promise-polyfill",`
`3`		`- "version": "8.1.1",`
	`3`	`+ "version": "8.1.2",`
`4`	`4`	`"description": "Lightweight promise polyfill. A+ compliant",`
`5`	`5`	`"main": "lib/index.js",`
`6`	`6`	`"module": "src/index.js",`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ import promiseFinally from './finally';`
`5`	`5`	`var setTimeoutFunc = setTimeout;`
`6`	`6`
`7`	`7`	`function isArray(x) {`
`8`		`- return Boolean(x && x.length);`
	`8`	`+ return Boolean(x && typeof x.length !== 'undefined');`
`9`	`9`	`}`
`10`	`10`
`11`	`11`	`function noop() {}`