code coverage

Afsin Ustundag · Afsin Ustundag · commit 58804674e5fa · 2015-06-24T23:39:57.000-04:00
diff --git a/lib/jsonave.js b/lib/jsonave.js
@@ -259,12 +259,6 @@ var Tracer = {
             this.traceProperties(ps, accumulator);
         };
     },
-    traceCommaDelimitedPropertiesGen: function (expr) {
-        var properties = expr.split(',');
-        return function (accumulator) {
-            this.traceProperties(properties, accumulator);
-        };
-    },
     traceExpandedPropertiesGen: function (parameters) {
         return function (accumulator) {
             var obj = accumulator.currentObject();
@@ -474,21 +468,6 @@ exports.expression = (function () {
     var cache = {};
 
     return function (inputObject) {
-        if (inputObject.json === null || inputObject.json === undefined) {
-            return inputObject.json;
-        }
-
-        var opts = Object.keys(inputObject).reduce(function (r, key) {
-            if ((key !== 'json') && (key !== 'path')) {
-                r[key] = inputObject[key];
-            }
-            return r;
-        }, {});
-        if (!opts.hasOwnProperty('wrap')) {
-            opts.wrap = true;
-        }
-        opts.emptyValue = [];
-        var jp = instance(inputObject.path, opts);
-        return jp(inputObject.json);
+        return exports(inputObject);
     };
 })();
diff --git a/test/legacy/test.at_and_dollar.js b/test/legacy/test.at_and_dollar.js
@@ -21,7 +21,7 @@ var t1 = {
 };
 
 xdescribe('@ and $ test', function () {
-    it('undefined, null', function () {
+    xit('undefined, null', function () {
         expect(jsonPath({
             json: undefined,
             path: 'foo'
diff --git a/test/test-store.js b/test/test-store.js
@@ -93,6 +93,20 @@ describe('jsonpath store example default options', function () {
         expect(actual).to.deep.equal(expected);
     });
 
+    it('$.store.book[-2].price', function () {
+        var jp = jsonpath.instance('$.store.book[-2].price');
+        var actual = jp(store);
+        var expected = 8.99;
+        expect(actual).to.equal(expected);
+    });
+
+    it('$.store.book[-2,-1].price', function () {
+        var jp = jsonpath.instance('$.store.book[-2,-1]');
+        var actual = jp(store);
+        var expected = [store.store.book[2], store.store.book[3]];
+        expect(actual).to.deep.equal(expected);
+    });
+
     it('$..book[*][category,author]', function () {
         var jp = jsonpath.instance('$..book[*][category,author]');
         var actual = jp(store);
