Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

GRAILS-5767 - the json and rico builders are coded to throw IllegalAr…

…gumentException when invalid methods are invoked in builder code. with Groovy 1.6 an InvokerInvocationException was being thrown from the reflection API. With Groovy 1.7 the IllegalArgumentException thrown by the builders is showing up, which I think is the right thing, pending confirmation.
  • Loading branch information...
commit c28cdc261e13ec72cf152b2383cfcf1db45b2e6f 1 parent 7f59d92
@jeffbrown jeffbrown authored
View
4 grails/src/test/grails/util/JSonBuilderTests.java
@@ -61,14 +61,14 @@ public void testOpenRicoBuilderElement() throws Exception {
try {
parse("json{ message( \"Hello World\" ){ item() } }");
fail();
- } catch (InvokerInvocationException e) {
+ } catch (IllegalArgumentException e) {
// expected
}
try {
parse("json{ message( \"Hello World\" ){ item(\"test\") } }");
fail();
- } catch (InvokerInvocationException e) {
+ } catch (IllegalArgumentException e) {
// expected
}
View
4 grails/src/test/grails/util/OpenRicoBuilderTests.java
@@ -78,14 +78,14 @@ public void testOpenRicoBuilderElement() throws Exception {
try {
parse("ajax{ test(id:\"test\") { } }");
fail();
- } catch (InvokerInvocationException e) {
+ } catch (IllegalArgumentException e) {
// expected
}
try {
parse("element{ test(id:\"test\") { } }");
fail();
- } catch (InvokerInvocationException e) {
+ } catch (IllegalArgumentException e) {
// expected
}
Please sign in to comment.
Something went wrong with that request. Please try again.