Author: chris.laprun(a)jboss.com Date: 2010-10-17 17:58:12 -0400 (Sun, 17 Oct 2010) New Revision: 4698 Modified: components/wsrp/trunk/common/src/test/java/org/gatein/wsrp/other/WSRPPortletURLTestCase.java Log: - GTNWSRP-103: Commented out test case since we now allow missing wsrp-requiresRewrite. Modified: components/wsrp/trunk/common/src/test/java/org/gatein/wsrp/other/WSRPPortletURLTestCase.java =================================================================== --- components/wsrp/trunk/common/src/test/java/org/gatein/wsrp/other/WSRPPortletURLTestCase.java 2010-10-16 16:38:46 UTC (rev 4697) +++ components/wsrp/trunk/common/src/test/java/org/gatein/wsrp/other/WSRPPortletURLTestCase.java 2010-10-17 21:58:12 UTC (rev 4698) @@ -353,10 +353,13 @@ { String message = "Should have detected missing parameter: "; + /* + // We now allow requiresRewrite to be missing and assume false instead... See GTNWSRP-103. String invalid = "wsrp_rewrite?wsrp-urlType=resource&wsrp-url=http%3A%2F%2Flocalhost%2F/wsrp_rewrite"; checkInvalidURL(invalid, message, WSRPRewritingConstants.RESOURCE_REQUIRES_REWRITE); + */ - invalid = "wsrp_rewrite?wsrp-urlType=resource&wsrp-requiresRewrite=true/wsrp_rewrite"; + String invalid = "wsrp_rewrite?wsrp-urlType=resource&wsrp-requiresRewrite=true/wsrp_rewrite"; checkInvalidURL(invalid, message, WSRPRewritingConstants.RESOURCE_URL); invalid = "wsrp_rewrite?wsrp-urlType=resource&wsrp-url=invalidURL&wsrp-requiresRewrite=true/wsrp_rewrite";