- Fix: If resultType is "all", if path resolves internally to a

    non-array (string), ensure it is converted to an array before
    converting to pointer for `pointer`
- Testing: Improve coverage

Author: brettz9

CHANGES.md

@@ -7,6 +7,9 @@
 - Fix: wrap: false returning inconsistent data types (@CacheControl)
 - Fix: Ensure throwing with a bad result type
 - Fix: Avoid erring when value before parent selector is falsey
+- Fix: If `resultType` is "all", if path resolves internally to a
+    non-array (string), ensure it is converted to an array before
+    converting to pointer for `pointer`
 - Enhancement: Allow path as array in non-object signature
 - Linting (ESLint): As per latest ash-nazg
 - Linting (ESLint): Remove redundant "use strict" with switch to ESM

src/jsonpath.js

@@ -347,13 +347,16 @@ JSONPath.prototype._getPreferredOutput = function (ea) {
     switch (resultType) {
     default:
         throw new TypeError('Unknown result type');
-    case 'all':
-        ea.pointer = JSONPath.toPointer(ea.path);
+    case 'all': {
+        const path = Array.isArray(ea.path)
+            ? ea.path
+            : JSONPath.toPathArray(ea.path);
+        ea.pointer = JSONPath.toPointer(path);
         ea.path = typeof ea.path === 'string'
             ? ea.path
             : JSONPath.toPathString(ea.path);
         return ea;
-    case 'value': case 'parent': case 'parentProperty':
+    } case 'value': case 'parent': case 'parentProperty':
         return ea[resultType];
     case 'path':
         return JSONPath.toPathString(ea[resultType]);

test/test.callback.js

@@ -54,4 +54,52 @@ describe('JSONPath - Callback', function () {
         jsonpath({json, path: '$.store.bicycle', resultType: 'value', wrap: false, callback});
         assert.deepEqual(result, expected);
     });
+
+    it('Callback with `resultType`: "all"', () => {
+        const expected = [
+            'value',
+            {
+                path: "$['store']['bicycle']",
+                value: json.store.bicycle,
+                parent: json.store,
+                parentProperty: 'bicycle',
+                pointer: '/store/bicycle',
+                hasArrExpr: undefined
+            },
+            {
+                path: "$['store']['bicycle']",
+                value: json.store.bicycle,
+                parent: json.store,
+                parentProperty: 'bicycle',
+                pointer: '/store/bicycle',
+                hasArrExpr: undefined
+            }
+        ];
+        let result;
+        /**
+         *
+         * @param {PlainObject} data
+         * @param {string} type
+         * @param {PlainObject} fullData
+         * @returns {void}
+         */
+        function callback (data, type, fullData) {
+            if (!result) {
+                result = [];
+            }
+            result.push(type, data, fullData);
+        }
+        jsonpath({json, path: '$.store.bicycle', resultType: 'all', wrap: false, callback});
+        assert.deepEqual(result[0], expected[0]);
+        // Todo[chai@>=5]: Error with circular perhaps will be resolved per https://github.com/chaijs/chai/issues/1109#issuecomment-395980733
+        //   so can try directly comparing whole object (as commented out) rather
+        //   than in parts as below that
+        // assert.deepEqual(result, expected);
+        Object.keys(result[1]).forEach((prop) => {
+            assert.deepEqual(result[1][prop], expected[1][prop]);
+        });
+        Object.keys(result[2]).forEach((prop) => {
+            assert.deepEqual(result[2][prop], expected[2][prop]);
+        });
+    });
 });