Fixing handling of a case when array dimensions is the last thing in a Groovy source file (#5689)

Author: greg-at-moderne

rewrite-groovy/src/main/java/org/openrewrite/groovy/GroovyParserVisitor.java

@@ -722,7 +722,7 @@ private List<J.ArrayDimension> buildNewArrayDimensions(ArrayExpression expressio
             }
             while (true) {
                 int beginBracket = indexOfNextNonWhitespace(cursor, source);
-                if (source.charAt(beginBracket) != '[') {
+                if (beginBracket >= source.length() || source.charAt(beginBracket) != '[') {
                     break;
                 }
 

rewrite-groovy/src/test/java/org/openrewrite/groovy/GroovyParserTest.java

@@ -117,4 +117,15 @@ void inner() {
           )
         );
     }
+
+    @Test
+    void variableDeclarationWithArrayInstantiationAsTheLastStatement() {
+        rewriteRun(
+          groovy(
+            """
+            def arr = new String[2]
+            """
+          )
+        );
+    }
 }