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);