right sqr bracket inside string literal fix

Afsin Ustundag · Afsin Ustundag · commit 9608dbf1e69b · 2015-06-28T07:47:01.000-04:00
diff --git a/lib/parser.js b/lib/parser.js
@@ -189,6 +189,18 @@ var findMatchingSquareBracket = function (path, index) {
                 return index;
             }
             ++index;
+        } else if (c === '\'') {
+            var rq = re.q_string.exec(path.substring(index));
+            if (!(rq && rq[0])) {
+                throw new Error('Unmatched quote');
+            }
+            index += rq[0].length;
+        } else if (c === '"') {
+            var rqq = re.qq_string.exec(path.substring(index));
+            if (!(rqq && rqq[0])) {
+                throw new Error('Unmatched quote');
+            }
+            index += rqq[0].length;
         } else {
             ++index;
         }
diff --git a/test/parser/test-break.js b/test/parser/test-break.js
@@ -61,4 +61,14 @@ describe('parser error conditions', function () {
         var fn = parser.normalize.bind(null, '$..book[engine,');
         expect(fn).to.throw(Error);
     });
+
+    it('$.prop["]nested', function () {
+        var fn = parser.normalize.bind(null, '$.prop["]nested');
+        expect(fn).to.throw(Error);
+    });
+
+    it('$.prop[\'nes]ted', function () {
+        var fn = parser.normalize.bind(null, '$.prop[\'nes]ted');
+        expect(fn).to.throw(Error);
+    });
 });
diff --git a/test/parser/test-normalize.js b/test/parser/test-normalize.js
@@ -316,3 +316,18 @@ describe('parser normalize ^', function () {
         expect(actual).to.deep.equal(expected);
     });
 });
+
+describe('parser normalize nested', function () {
+    it('$.prop["]nested"]', function () {
+        var actual = parser.normalize('$.prop["]nested"]');
+        var expected = [_n('root'), _p('prop'), _p(']nested')];
+        expect(actual).to.deep.equal(expected);
+    });
+
+    it('$.prop[\'nes]ted\']', function () {
+        var actual = parser.normalize('$.prop[\'nes]ted\']');
+        var expected = [_n('root'), _p('prop'), _p('nes]ted')];
+        expect(actual).to.deep.equal(expected);
+    });
+
+});
