[gatein-commits] gatein SVN: r6177 - components/pc/trunk/portlet/src/test/java/org/gatein/pc/test/portlet/state.

Author: chris.laprun(a)jboss.com Date: 2011-04-07 15:57:54 -0400 (Thu, 07 Apr 2011) New Revision: 6177 Modified: components/pc/trunk/portlet/src/test/java/org/gatein/pc/test/portlet/state/AbstractStatefulPortletInvokerTestCase.java Log: - GTNPC-58: Fixed improper tests that were actually passing due to too lenient implementation of PortletContext. Modified: components/pc/trunk/portlet/src/test/java/org/gatein/pc/test/portlet/state/AbstractStatefulPortletInvokerTestCase.java =================================================================== --- components/pc/trunk/portlet/src/test/java/org/gatein/pc/test/portlet/state/AbstractStatefulPortletInvokerTestCase.java 2011-04-07 17:25:59 UTC (rev 6176) +++ components/pc/trunk/portlet/src/test/java/org/gatein/pc/test/portlet/state/AbstractStatefulPortletInvokerTestCase.java 2011-04-07 19:57:54 UTC (rev 6177) @@ -1099,10 +1099,8 @@ PortletContext portletContext = importedPortlet.getContext(); - //make sure the expected portlet context and the one we get back from the import are the same -// assertEquals(originalPortletContext.getComponents().getApplicationName(), portletContext.getComponents().getApplicationName()); -// assertEquals(originalPortletContext.getComponents().getPortletName(), portletContext.getComponents().getPortletName()); - assertEquals(originalPortletContext, portletContext); + //check that the portlet info for the original and new imported portlet are the same (which is assumed to ensure that the contexts refer to the same portlet) + assertEquals(getPortlet(originalPortletContext).getInfo(), getPortlet(portletContext).getInfo()); if (originalPortletContext instanceof StatefulPortletContext) {