From 7d677b23d32d61ee5b0f91c11ea2910a8a10edf4 Mon Sep 17 00:00:00 2001 From: Andy Clement Date: Mon, 18 Aug 2008 00:21:45 +0000 Subject: [PATCH] moved test into ParserErrorMessageTests --- .../expression/spel/ParserErrorMessagesTests.java | 5 +++++ .../org/springframework/expression/spel/ParsingTests.java | 3 --- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/org.springframework.expression/src/test/java/org/springframework/expression/spel/ParserErrorMessagesTests.java b/org.springframework.expression/src/test/java/org/springframework/expression/spel/ParserErrorMessagesTests.java index 5abf578d2b..ee6a40426f 100644 --- a/org.springframework.expression/src/test/java/org/springframework/expression/spel/ParserErrorMessagesTests.java +++ b/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 + } + } diff --git a/org.springframework.expression/src/test/java/org/springframework/expression/spel/ParsingTests.java b/org.springframework.expression/src/test/java/org/springframework/expression/spel/ParsingTests.java index af83f4829d..e49587b8aa 100644 --- a/org.springframework.expression/src/test/java/org/springframework/expression/spel/ParsingTests.java +++ b/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))"); }