Merge pull request #60 from rayokota/fix-regexp

Fix regexp which ends in /i or /m

the code is:

while (currentChar == 'i')
  position++
  path.charAt(position)

when the regex ends with /i, position points "beyond" the string
in JS, we chatAt simply returns ""
in Java, we get an index out of bounds

Author: aeberhart

src/main/java/com/dashjoin/jsonata/Tokenizer.java

@@ -144,7 +144,11 @@ Pattern scanRegex() {
                 start = position;
                 while (currentChar == 'i' || currentChar == 'm') {
                     position++;
-                    currentChar = path.charAt(position);
+                    if (position < length) {
+                        currentChar = path.charAt(position);
+                    } else {
+                        currentChar = 0;
+                    }
                 }
                 flags = path.substring(start, position) + 'g';
 

src/test/java/com/dashjoin/jsonata/StringTest.java

@@ -127,6 +127,13 @@ public void evalTest() {
         + "  $eval('$data.Wert1')\n"
         + ")").evaluate(null));
   }
+
+  @Test
+  public void regexTest() {
+    Assertions.assertEquals(Map.of("foo", 1), jsonata(
+        "($matcher := $eval('/^' & 'foo' & '/i'); $.$spread()[$.$keys() ~> $matcher])")
+        .evaluate(Map.of("foo", 1, "bar", 2)));
+  }
 
   @Disabled
   @Test