GRAILS-9110 TestFor(UrlMappings) matches closure parameters using toString #201

Closed
wants to merge 3 commits into
from

Projects

None yet

3 participants

@bluesliverx
Contributor

I removed the toString code on the UrlMappingsTestMixin class. This should fix the problem.

@graemerocher
Member

Any test case for the pull request?

@bluesliverx
Contributor

Not yet, can you point me at a good place for the test? I'm not familiar with how grails-core tests are laid out.

@bluesliverx
Contributor

I added testing for it, and everything appears to be working as expected. There is a failing test in grails.persistence.EntityTransformTests, but I believe this is completely unrelated.

@jeffbrown
Member

The code has been put on the 2.0.x, 2.1.x and master branches.

Thanks!

@jeffbrown jeffbrown closed this May 18, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment