Fixes deterministic check on undefined

Author: TotalTechGeek

compiler.js

@@ -85,7 +85,7 @@ export function isDeterministic (method, engine, buildState) {
     const lower = method[func]
 
     if (engine.isData(method, func)) return true
-    if (lower === undefined) return true
+    if (func === undefined) return true
 
     // eslint-disable-next-line no-throw-literal
     if (!engine.methods[func]) throw { type: 'Unknown Operator', key: func }

general.test.js

@@ -153,6 +153,22 @@ describe('Various Test Cases', () => {
     for (const engine of [...normalEngines, ...permissiveEngines]) await testEngine(engine, { max: 5 }, {}, 5)
   })
 
+  it('calls the function the correct amount of times', async () => {
+    for (const engine of [...normalEngines, ...permissiveEngines]) {
+      let called = 0
+      engine.addMethod('inc', () => {
+        called++
+        return 1
+      }, { optimizeUnary: true })
+
+      const f = await engine.build({ inc: undefined })
+      await f()
+      await f()
+      await f()
+      if (called !== 3) throw new Error('Should have called the function 3 times')
+    }
+  })
+
   it('sees empty structures as false', async () => {
     for (const engine of [...normalEngines, ...permissiveEngines]) await testEngine(engine, { if: [{ var: 'x' }, true, false] }, { x: {} }, false)
     for (const engine of [...normalEngines, ...permissiveEngines]) await testEngine(engine, { if: [{ var: 'x' }, true, false] }, { x: [] }, false)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "json-logic-engine",
-  "version": "5.0.0-6",
+  "version": "5.0.0-7",
   "description": "Construct complex rules with JSON & process them.",
   "main": "./dist/cjs/index.js",
   "module": "./dist/esm/index.js",