[jboss-svn-commits] JBoss Portal SVN: r5539 - branches/JBoss_Portal_Branch_2_4/portlet/src/main/org/jboss/portal/test/portlet/jsr168/tck/portletresponses

[jboss-svn-commits at lists.jboss.org]

Author: julien at jboss.com
Date: 2006-10-31 15:42:09 -0500 (Tue, 31 Oct 2006)
New Revision: 5539

Modified:
   branches/JBoss_Portal_Branch_2_4/portlet/src/main/org/jboss/portal/test/portlet/jsr168/tck/portletresponses/PortletResponsesSequenceBuilder.java
Log:
correct portlet container test case that check that no render parameter set during action phase leads to an empty parameter map in the render

Modified: branches/JBoss_Portal_Branch_2_4/portlet/src/main/org/jboss/portal/test/portlet/jsr168/tck/portletresponses/PortletResponsesSequenceBuilder.java
===================================================================
--- branches/JBoss_Portal_Branch_2_4/portlet/src/main/org/jboss/portal/test/portlet/jsr168/tck/portletresponses/PortletResponsesSequenceBuilder.java	2006-10-31 20:40:11 UTC (rev 5538)
+++ branches/JBoss_Portal_Branch_2_4/portlet/src/main/org/jboss/portal/test/portlet/jsr168/tck/portletresponses/PortletResponsesSequenceBuilder.java	2006-10-31 20:42:09 UTC (rev 5539)
@@ -56,6 +56,7 @@
 import java.io.PrintWriter;
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Collections;
 
 /**
  * @author <a href="mailto:boleslaw.dawidowicz at jboss.org">Boleslaw Dawidowicz</a>
@@ -431,13 +432,21 @@
          }
       });
 
+      seq.addAction(2, UTP1.ACTION_JOINPOINT, new PortletActionTestAction()
+      {
+         protected AssertResult run(Portlet portlet, ActionRequest request, ActionResponse response)
+         {
+            // Don't set any parameter
+            return null;
+         }
+      });
+
       seq.addAction(2, UTP1.RENDER_JOINPOINT, new PortletRenderTestAction()
       {
          protected Result run(Portlet portlet, RenderRequest request, RenderResponse response)
          {
-            //we shouldn't receive anything as no parameters were set during action phase
-            assertNull(request.getParameter("key1"));
-            assertNull(request.getParameter("key2"));
+            // We shouldn't receive anything as no parameters were set during action phase
+            assertEquals(Collections.EMPTY_MAP, request.getParameterMap());
             return new EndTestResult();
          }
       });