- Refactoring: Use test over exec

- Testing: Increase coverage

Author: brettz9

src/jsonpath.js

@@ -64,7 +64,7 @@ const vm = JSONPath.nodeVMSupported
 
             const funcString = funcs.reduce((s, func) => {
                 let fString = context[func].toString();
-                if (!(/function/u).exec(fString)) {
+                if (!(/function/u).test(fString)) {
                     fString = 'function ' + fString;
                 }
                 return 'var ' + func + '=' + fString + ';' + s;

test/test.eval.js

@@ -60,6 +60,35 @@ checkBuiltInVMAndNodeVM(function (vmType, setBuiltInState) {
             assert.deepEqual(result, expected);
         });
 
+        it('sandbox with function without "function" in string', () => {
+            const expected = [json.store.book];
+            const result = jsonpath({
+                json,
+                sandbox: {
+                    category () {
+                        return 'reference';
+                    }
+                },
+                path: "$..[?(@.category === category())]", wrap: false
+            });
+            assert.deepEqual(result, expected);
+        });
+
+        it('sandbox with function with "function" in string', () => {
+            const expected = [json.store.book];
+            const result = jsonpath({
+                json,
+                sandbox: {
+                    // eslint-disable-next-line object-shorthand
+                    category: function () {
+                        return 'reference';
+                    }
+                },
+                path: "$..[?(@.category === category())]", wrap: false
+            });
+            assert.deepEqual(result, expected);
+        });
+
         it('sandbox (with parsing function)', () => {
             const expected = [json.store.book];
             const result = jsonpath({