Moved is.thenable from Array checks to Object checks

arasatasaygin · Mar 27, 2018 · 360b886 · 360b886
1 parent 43bd1b8
commit 360b886
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 16 deletions.
diff --git a/is.js b/is.js
@@ -824,6 +824,11 @@
     // propertyDefined method does not support 'all' and 'any' interfaces
     is.propertyDefined.api = ['not'];
 
+    // is a given value thenable (like Promise)?
+    is.thenable = function(value) {
+        return is.object(value) && typeof value.then === 'function';
+    };
+
     // Array checks
     /* -------------------------------------------------------------------------- */
 
@@ -856,10 +861,6 @@
         return true;
     };
 
-    // is a given value thenable (like Promise)?
-    is.thenable = function(value) {
-        return is.object(value) && typeof value.then === 'function';
-    };
 
     // API
     // Set 'not', 'all' and 'any' interfaces to methods based on their api property

diff --git a/test/test.js b/test/test.js
@@ -1125,6 +1125,18 @@
             });
         });
         checkApi('domNode');
+
+        describe('is.thenable', function() {
+            it('should return true if passed parameter type is Promise', function() {
+                var promise = Promise.resolve(true); // eslint-disable-line no-undef
+                expect(is.thenable(promise)).to.be.true;
+            });
+            it('should return false if passed parameter type is not Promise', function() {
+                var notPromise = 'test';
+                expect(is.thenable(notPromise)).to.be.false;
+            });
+        });
+        checkApi('thenable');
     });
 
     describe('array checks', function() {
@@ -1177,16 +1189,4 @@
         checkApi('inArray', ['not']);
     });
 
-    describe('is.thenable', function() {
-        it('should return true if passed parameter type is Promise', function() {
-            var promise = Promise.resolve(true); // eslint-disable-line no-undef
-            expect(is.thenable(promise)).to.be.true;
-        });
-        it('should return false if passed parameter type is not Promise', function() {
-            var notPromise = 'test';
-            expect(is.thenable(notPromise)).to.be.false;
-        });
-    });
-    checkApi('thenable');
-
 }(this));