diff --git a/apps/portlet/pom.xml b/apps/portlet/pom.xml index 2ef93f7e0..15ac1486c 100644 --- a/apps/portlet/pom.xml +++ b/apps/portlet/pom.xml @@ -40,21 +40,6 @@ http://svn.apache.org/viewcvs.cgi/struts/struts2/trunk/apps/portlet/ - - - - org.apache.maven.plugins - maven-surefire-plugin - - - - org/apache/struts2/portlet/test/* - - - - - - pluto diff --git a/apps/portlet/src/test/java/org/apache/struts2/portlet/test/BasePortletTest.java b/apps/portlet/src/test/java/org/apache/struts2/portlet/test/BasePortletTest.java index 1c7e76c44..d2cb12c89 100644 --- a/apps/portlet/src/test/java/org/apache/struts2/portlet/test/BasePortletTest.java +++ b/apps/portlet/src/test/java/org/apache/struts2/portlet/test/BasePortletTest.java @@ -10,11 +10,11 @@ import org.mortbay.jetty.webapp.WebAppContext; public abstract class BasePortletTest extends WebTestCase { protected Server server; - - private int port = 9876; private String contextPath = "/test"; + private int port; + public void setUp() throws Exception { System.setProperty("org.apache.pluto.embedded.portletId", getPortletName()); server = new Server(port); @@ -26,6 +26,9 @@ public abstract class BasePortletTest extends WebTestCase { webapp.addServlet(portletServlet, "/PlutoInvoker/" + getPortletName()); server.addHandler(webapp); server.start(); + // Retrieve the actual port that is used, in case a random, free port is + // picked + int port = server.getConnectors()[0].getLocalPort(); getTestContext().setBaseUrl("http://localhost:" + port + contextPath); } @@ -58,10 +61,6 @@ public abstract class BasePortletTest extends WebTestCase { clickElementByXPath("//span[@class='help']/.."); } - public void setPort(int port) { - this.port = port; - } - public void setContextPath(String contextPath) { if(!contextPath.startsWith("/")) { this.contextPath = "/" + contextPath; @@ -71,5 +70,9 @@ public abstract class BasePortletTest extends WebTestCase { } } + public void setPort(int port) { + this.port = port; + } + public abstract String getPortletName(); }