Updated Branches: refs/heads/wicket-1.5.x 0a17ce37d -> 89049eeb1
WICKET-4850 BaseWicketTester discards cookies with MaxAge = -1 when processing a new request Project: http://git-wip-us.apache.org/repos/asf/wicket/repo Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/89049eeb Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/89049eeb Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/89049eeb Branch: refs/heads/wicket-1.5.x Commit: 89049eeb10f67e600566617ab918900560ebb0d0 Parents: 0a17ce3 Author: Martin Tzvetanov Grigorov <[email protected]> Authored: Fri Nov 2 16:31:18 2012 +0200 Committer: Martin Tzvetanov Grigorov <[email protected]> Committed: Fri Nov 2 16:32:41 2012 +0200 ---------------------------------------------------------------------- .../wicket/util/tester/BaseWicketTester.java | 5 ++++- .../wicket/util/tester/WicketTesterTest.java | 4 ++-- 2 files changed, 6 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/wicket/blob/89049eeb/wicket-core/src/main/java/org/apache/wicket/util/tester/BaseWicketTester.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/main/java/org/apache/wicket/util/tester/BaseWicketTester.java b/wicket-core/src/main/java/org/apache/wicket/util/tester/BaseWicketTester.java index 6deab0e..5eee438 100644 --- a/wicket-core/src/main/java/org/apache/wicket/util/tester/BaseWicketTester.java +++ b/wicket-core/src/main/java/org/apache/wicket/util/tester/BaseWicketTester.java @@ -401,7 +401,10 @@ public class BaseWicketTester { for (Cookie cookie : cookies) { - if (cookie.getMaxAge() > 0) + // maxAge == -1 -> means session cookie + // maxAge == 0 -> delete the cookie + // maxAge > 0 -> the cookie will expire after this age + if (cookie.getMaxAge() != 0) { request.addCookie(cookie); } http://git-wip-us.apache.org/repos/asf/wicket/blob/89049eeb/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java b/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java index 54141b7..25eb96d 100644 --- a/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java +++ b/wicket-core/src/test/java/org/apache/wicket/util/tester/WicketTesterTest.java @@ -1074,14 +1074,14 @@ public class WicketTesterTest extends WicketTestCase } /** - * Tests that setting a cookie with age < 0 will not be stored after the request cycle. + * Tests that setting a cookie with age == 0 will not be stored after the request cycle. */ @Test public void dontTransferCookiesWithNegativeAge() { String cookieName = "wicket4289Name"; String cookieValue = "wicket4289Value"; - int cookieAge = -1; // age < 0 => do not store it + int cookieAge = 0; // age = 0 => do not store it Cookie cookie = new Cookie(cookieName, cookieValue); cookie.setMaxAge(cookieAge);
