[gatein-commits] gatein SVN: r6189 - in components/pc/trunk: federation/src/test/java/org/gatein/pc/federation and 1 other directory.
Author: chris.laprun(a)jboss.com
Date: 2011-04-11 07:24:52 -0400 (Mon, 11 Apr 2011)
New Revision: 6189
Modified:
components/pc/trunk/api/src/main/java/org/gatein/pc/api/PortletContext.java
components/pc/trunk/federation/src/test/java/org/gatein/pc/federation/FederatingPortletInvokerTestCase.java
Log:
- GTNPC-58: Fixed test case. Note that the previous commit changed the value of
INVOKER_SEPARATOR which used to be '.' to '+'
to make sure that there wasn't any unintentional collisions with SEPARATOR value
('.') in the parsing and outputing code.
Modified: components/pc/trunk/api/src/main/java/org/gatein/pc/api/PortletContext.java
===================================================================
--- components/pc/trunk/api/src/main/java/org/gatein/pc/api/PortletContext.java 2011-04-11
11:03:57 UTC (rev 6188)
+++ components/pc/trunk/api/src/main/java/org/gatein/pc/api/PortletContext.java 2011-04-11
11:24:52 UTC (rev 6189)
@@ -37,7 +37,11 @@
private static final String PREFIX = "/";
private static final char SEPARATOR = '.';
- /** The separator used in the id to route to the correct invoker. */
+
+ /**
+ * The separator used in the id to route to the correct invoker. TODO: change value
back once tests have been done to
+ * remove incidental use of value
+ */
public static final String INVOKER_SEPARATOR = "+";
public static final String PRODUCER_CLONE_ID_PREFIX = "_";
Modified:
components/pc/trunk/federation/src/test/java/org/gatein/pc/federation/FederatingPortletInvokerTestCase.java
===================================================================
---
components/pc/trunk/federation/src/test/java/org/gatein/pc/federation/FederatingPortletInvokerTestCase.java 2011-04-11
11:03:57 UTC (rev 6188)
+++
components/pc/trunk/federation/src/test/java/org/gatein/pc/federation/FederatingPortletInvokerTestCase.java 2011-04-11
11:24:52 UTC (rev 6189)
@@ -182,7 +182,7 @@
@Test
public void testGetPortletOnFederatedInvoker() throws PortletInvokerException
{
- PortletContext federatedContext =
PortletContext.createPortletContext(federatedInvoker.getId() + "." +
portletContext.getId());
+ PortletContext federatedContext =
PortletContext.createPortletContext(federatedInvoker.getId() +
PortletContext.INVOKER_SEPARATOR + portletContext.getId());
Portlet samePortlet = federatedInvoker.getPortlet(federatedContext);
assertNotNull(samePortlet);
assertEquals(samePortlet.getContext(), federatedContext);