Switch empty object to falsy, and add iterator check

Author: TotalTechGeek

asyncLogic.js

@@ -56,9 +56,16 @@ class AsyncLogicEngine {
    */
   truthy (value) {
     if (!value) return value
-    if (Array.isArray(value) && value.length === 0) return false
-    // Uncomment the following line to switch empty object to falsy.
-    // if (value && typeof value === 'object') return Object.keys(value).length > 0
+    // The following check could be erased, as it'd be caught by the iterator check,
+    // but it's here for performance reasons.
+    if (Array.isArray(value)) return value.length > 0
+    if (typeof value === 'object') {
+      if (value[Symbol.iterator]) {
+        if ('length' in value && value.length === 0) return false
+        if ('size' in value && value.size === 0) return false
+      }
+      if (value.constructor.name === 'Object') return Object.keys(value).length > 0
+    }
     return value
   }
 

logic.js

@@ -47,9 +47,16 @@ class LogicEngine {
    */
   truthy (value) {
     if (!value) return value
-    if (Array.isArray(value) && value.length === 0) return false
-    // Uncomment the following line to switch empty object to falsy.
-    // if (typeof value === 'object') return Object.keys(value).length > 0
+    // The following check could be erased, as it'd be caught by the iterator check,
+    // but it's here for performance reasons.
+    if (Array.isArray(value)) return value.length > 0
+    if (typeof value === 'object') {
+      if (value[Symbol.iterator]) {
+        if ('length' in value && value.length === 0) return false
+        if ('size' in value && value.size === 0) return false
+      }
+      if (value.constructor.name === 'Object') return Object.keys(value).length > 0
+    }
     return value
   }
 

suites/control/if.json

@@ -21,13 +21,13 @@
     },
     "Object checks",
     {
-        "description": "Objects are always truthy, even without keys.",
+        "description": "Empty object is falsy.",
         "rule": { "if": [{}, "apple", "banana"] },
         "data": null,
-        "result": "apple"
+        "result": "banana"
     },
     {
-        "description": "Objects are always truthy, with keys defined",
+        "description": "Objects are truthy, with keys defined",
         "rule": { "if": [{ "val": [] }, "apple", "banana"] },
         "data": { "some": "value" },
         "result": "apple"
@@ -240,6 +240,12 @@
         "description": "Returns first truthy value, some conditions truthy (8)",
         "rule": { "if": [{}, "apple", 0, "banana", 7, "carrot", "date"] },
         "data": null,
+        "result": "carrot"
+    },
+    {
+        "description": "Returns first truthy value, some conditions truthy (9)",
+        "rule": { "if": [{ "val": "a" }, "apple", 0, "banana", 7, "carrot", "date"] },
+        "data": { "a": { "b": 1 } },
         "result": "apple"
     },
     "Bad Arguments",

suites/truthiness.json

@@ -40,7 +40,7 @@
     {
         "description": "Truthy: {}",
         "rule": { "!!": [{}] },
-        "result": true,
+        "result": false,
         "data": null
     },
     {
@@ -59,7 +59,7 @@
     {
         "description": "Truthy: Zero Key Object",
         "rule": { "!!": { "val": "obj" } },
-        "result": true,
+        "result": false,
         "data": { "obj": {} }
     },
     {