Interesting. I wrote that test in response to a problem that I was seeing
when I upgraded from 2.1.0 to 2.1.1.

I have a path /foo/:bar/baz. In 2.1.0 it was uri decoding the bar
parameter when it was passed to the controller. After upgrading to
2.1.1 it was not uri decoding the bar parameter when it got passed to the
controller.

Is there a better way to write a test that will see if it passes/fails in
master?

On Sunday, May 12, 2013, Nilanjan Raychaudhuri wrote:

> In framework/src/play/src/test/scala/play/core/router/RouterSpec.scala:
>
> > @@ -25,23 +25,26 @@ object RouterSpec extends Specification {
> >
> >
> >    "PathPattern" should {
> > -    val pathPattern = PathPattern(Seq(StaticPart("/path/"), StaticPart("to/"), DynamicPart("foo", "[^/]+", true)))
> > -    val pathString = "/path/to/some%20file"
> > +    val pathPatternWithDynamicLastPart = PathPattern(Seq(StaticPart("/path/"), StaticPart("to/"), DynamicPart("foo", "[^/]+", true)))
> > +    val pathPatternWithDynamicMiddlePart = PathPattern(Seq(StaticPart("/path/"), DynamicPart("foo", "[^/]+", true), StaticPart("to/")))
>
> Actually the last part should be StaticPart("/to") and it should work
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/playframework/Play20/pull/1107/files#r4187227>
> .
>