Merge pull request #59: allow # in IDs

Author: kennknowles

.travis.yml

@@ -8,7 +8,10 @@ python:
 install: 
  - "pip install ."
  - "pip install pytest"
- - "pip install coverage coveralls"
+ - "pip install coveralls"
+ # Coveralls 4.0 doesn't support Python 3.2
+ - if [ "$TRAVIS_PYTHON_VERSION" == "3.2" ]; then pip install coverage==3.7.1; fi
+ - if [ "$TRAVIS_PYTHON_VERSION" != "3.2" ]; then pip install coverage; fi
 script: coverage run setup.py test
 after_success:
  - coveralls

jsonpath_rw/lexer.py

@@ -61,7 +61,7 @@ def tokenize(self, string):
     t_ignore = ' \t'
 
     def t_ID(self, t):
-        r'[a-zA-Z_@][a-zA-Z0-9_@\-]*'
+        r'[a-zA-Z_@#][a-zA-Z0-9_@\-]*'
         t.type = self.reserved_words.get(t.value, 'ID')
         return t
 

tests/test_lexer.py

@@ -53,6 +53,7 @@ def test_simple_inputs(self):
         self.assert_lex_equiv('`this`', [self.token('this', 'NAMED_OPERATOR')])
         self.assert_lex_equiv('|', [self.token('|', '|')])
         self.assert_lex_equiv('where', [self.token('where', 'WHERE')])
+        self.assert_lex_equiv('a.#text', [self.token('a', 'ID'), self.token('.', '.'), self.token('#text', 'ID')])
 
     def test_basic_errors(self):
         def tokenize(s):
@@ -66,4 +67,4 @@ def tokenize(s):
         self.assertRaises(JsonPathLexerError, tokenize, '"`')
         self.assertRaises(JsonPathLexerError, tokenize, "'`")
         self.assertRaises(JsonPathLexerError, tokenize, '?')
-        self.assertRaises(JsonPathLexerError, tokenize, '$.foo.bar.#')
+        self.assertRaises(JsonPathLexerError, tokenize, '$.foo.bar.%')