reinitialize ast after parse error; fixes #9

dchester · Nov 21, 2015 · 265773c · 265773c
1 parent d5722d2
commit 265773c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/lib/parser.js b/lib/parser.js
@@ -4,6 +4,13 @@ var gparser = require('../generated/parser');
 var Parser = function() {
 
   var parser = new gparser.Parser();
+
+  var _parseError = parser.parseError;
+  parser.yy.parseError = function() {
+    parser.yy.ast.initialize();
+    _parseError.apply(parser, arguments);
+  }
+
   return parser;
 
 };

diff --git a/test/parse.js b/test/parse.js
@@ -124,6 +124,15 @@ suite('parse', function() {
     ])
   });
 
+  test('parser ast is reinitialized after parse() throws', function() {
+    assert.throws(function() { var path = jp.parse('store.book...') })
+    var path = jp.parse('$..price');
+    assert.deepEqual(path, [
+      { "expression": { "type": "root", "value": "$" } },
+      { "expression": { "type": "identifier", "value": "price" }, "operation": "member", "scope": "descendant"}
+    ])
+  });
+
 });
 
 suite('parse-negative', function() {