> there's still work to be done. The testsuite portlets don't error out, but
> the test links are missing from the portlets. Ben, can you take a look at
> this?
> /Craig

Craig,

I do not really have time to test this out right now, but I suspect that
this patch should solve the problem.  There was a FIXME comment in the
TestPortlet doView() to change the request attribute name 'tests' to
'testConfigs'.  This patch updates the JSP to reflect that change.

Thanks,

Ben
Index: pluto-testsuite/src/main/webapp/jsp/introduction.jsp
===================================================================
--- pluto-testsuite/src/main/webapp/jsp/introduction.jsp	(revision 602488)
+++ pluto-testsuite/src/main/webapp/jsp/introduction.jsp	(working copy)
@@ -37,7 +37,7 @@
 <p>
   Please select one of the following tests:
   <table>
-    <c:forEach var="testConfig" items="${tests}" varStatus="status">
+    <c:forEach var="testConfig" items="${testConfigs}" varStatus="status">
       <tr>
         <td>
           # <c:out value="${status.index}"/>.
@@ -70,7 +70,4 @@
       </tr>
     </c:forEach>
   </table>
-</p>
-
-
-
+</p>
\ No newline at end of file

Reply via email to