public void testRequiresLogoutUrlWorksWithQueryParams() {
    MockHttpServletRequest request = new MockHttpServletRequest();
    request.setContextPath("/context");
    MockHttpServletResponse response = new MockHttpServletResponse();

    request.setServletPath("/logout");
    request.setRequestURI("/context/logout?param=blah");
    request.setQueryString("otherparam=blah");

    assertTrue(filter.requiresLogout(request, response));
  }
  public void testRequiresLogoutUrlWorksWithPathParams() {
    MockHttpServletRequest request = new MockHttpServletRequest();
    MockHttpServletResponse response = new MockHttpServletResponse();

    request.setRequestURI("/context/logout;someparam=blah?param=blah");
    request.setServletPath("/logout;someparam=blah");
    request.setQueryString("otherparam=blah");

    DefaultHttpFirewall fw = new DefaultHttpFirewall();
    assertTrue(filter.requiresLogout(fw.getFirewalledRequest(request), response));
  }