NoRedirectionStickinessHtml tests escaped url path

[cilim]

Hello everyone,
I'm the Rails contractor that's working on the stormpath-rails integration.

The testNoRedirectionStickinessHtml test in LoginIT.groovy asserts escaped url path, which causes tests to fail in Rails.

public void testNoRedirectionStickinessHtml() throws Exception {
     ...
        req
            .when()
                .get(FrameworkConstants.MeRoute)
            .then()
                .statusCode(302)
                .header("Location", urlMatchesPath("/login?next=%2Fme")) //we must be redirected to login

        ...
    }

Changing the %2F to an unescaped / solves the issue.

req
  .when()
     .get(FrameworkConstants.MeRoute)
  .then()
     .statusCode(302)
     .header("Location", urlMatchesPath("/login?next=/me"))

Do other frameworks need this character escaped and should we escape it in Rails, or are you going to unescape it?

[nbarbettini]

This test was specifically added to test a regression in the JSDK, so I could see an argument for removing it from the TCK entirely (it may be better suited for a JSDK IT). Thoughts @omgitstom or @edjiang?

[edjiang]

It should be like this, only asserting the path. Not sure when it got changed, though, or why.

#288

[cilim]

Found the commit: 80a0ae4

@dogeared can you change the %2F to an unescaped / or does it need to stay like this?

[edjiang]

We shouldn't be checking for the next parameter, as that's not in the framework spec.

See https://github.com/stormpath/stormpath-framework-spec/pull/94/files