moved test into ParserErrorMessageTests

org.springframework.expression/src/test/java/org/springframework/expression/spel/ParserErrorMessagesTests.java

@@ -63,4 +63,9 @@ public class ParserErrorMessagesTests extends ExpressionTestCase {
 		parseAndCheckError("null is T('a')", SpelMessages.PARSE_PROBLEM, 10, "mismatched input ''a'' expecting ID"); // POOR
 	}
 
+	public void testExpressionLists02a() {
+		// either missing semi or rogue 5. RPAREN should at least be ')', and why doesn't it give the other possibles?
+		parseAndCheckError("( (3;4)5)", SpelMessages.PARSE_PROBLEM, 7, "mismatched input '5' expecting RPAREN"); // POOR
+	}
+
 }

org.springframework.expression/src/test/java/org/springframework/expression/spel/ParsingTests.java

@@ -391,9 +391,6 @@ public class ParsingTests extends TestCase {
 		parseCheck("( (3;4);5)", "((3;4);5)");
 	}
 
-	// TODO parser recovery for this next one: missing semi
-	// public void testExpressionLists02a() { parseCheck("( (3;4)5)","((3;4);5)");}
-	// // badly formed, missing a semi
 	public void testExpressionLists03() {
 		parseCheck("(3;(4;5))");
 	}