This should be functional again. You don't need to specify - DexcludeTest= anymore, just set the -DassemblyId to either jetty or tomcat and the excludes will configure appropriately.

--jason


On May 13, 2008, at 12:23 AM, Jarek Gawor wrote:

Jason,

Can you please restore that excludeTest functionality (unless you are
planning to put in a workaround for that soon)? This is being used by
the automatic tests right now.

Jarek

On Fri, May 9, 2008 at 6:12 AM,  <[EMAIL PROTECTED]> wrote:
Author: jdillon
Date: Fri May  9 03:12:49 2008
New Revision: 654748

URL: http://svn.apache.org/viewvc?rev=654748&view=rev
Log:
Fix up some more testsuite configuration

Modified:
geronimo/server/trunk/testsuite/console-testsuite/advanced/src/ test/java/org/apache/geronimo/testsuite/console/WebServerTest.java geronimo/server/trunk/testsuite/deployment-testsuite/deployment- tests/pom.xml geronimo/server/trunk/testsuite/deployment-testsuite/jca-cms- tests/jca-cms-ear/pom.xml
   geronimo/server/trunk/testsuite/deployment-testsuite/pom.xml
   geronimo/server/trunk/testsuite/enterprise-testsuite/pom.xml
   geronimo/server/trunk/testsuite/pom.xml
   geronimo/server/trunk/testsuite/security-testsuite/pom.xml
   geronimo/server/trunk/testsuite/web-testsuite/pom.xml
geronimo/server/trunk/testsuite/web-testsuite/test-2.1-jsps/ pom.xml geronimo/server/trunk/testsuite/web-testsuite/test-2.5-servlets/ pom.xml
   geronimo/server/trunk/testsuite/web-testsuite/test-jetty/pom.xml
   geronimo/server/trunk/testsuite/web-testsuite/test-myfaces/pom.xml
   geronimo/server/trunk/testsuite/web-testsuite/test-tomcat/pom.xml
geronimo/server/trunk/testsuite/web-testsuite/test-web-forward/ web-forward-ear/pom.xml geronimo/server/trunk/testsuite/web-testsuite/test-web- references/web-references-ear/pom.xml
   geronimo/server/trunk/testsuite/webservices-testsuite/pom.xml

Modified: geronimo/server/trunk/testsuite/web-testsuite/pom.xml
URL: 
http://svn.apache.org/viewvc/geronimo/server/trunk/testsuite/web-testsuite/pom.xml?rev=654748&r1=654747&r2=654748&view=diff
= = = = = = = = = =====================================================================
--- geronimo/server/trunk/testsuite/web-testsuite/pom.xml (original)
+++ geronimo/server/trunk/testsuite/web-testsuite/pom.xml Fri May 9 03:12:49 2008
@@ -44,65 +44,8 @@
         - JSTL
    </description>

-    <properties>
-        <excludeTest>test-tomcat</excludeTest>
-    </properties>
-
    <build>
-        <pluginManagement>
-            <plugins>
-                <plugin>
-                    <groupId>org.apache.maven.plugins</groupId>
-                    <artifactId>maven-surefire-plugin</artifactId>
-                    <configuration>
-                        <suiteXmlFiles>
- <suiteXmlFile>$ {project.build.testOutputDirectory}/testng.xml</suiteXmlFile>
-                        </suiteXmlFiles>
-                        <systemProperties>
-                            <property>
-                                <name>geronimoVersion</name>
-                                <value>${version}</value>
-                            </property>
-                        </systemProperties>
-                    </configuration>
-                </plugin>
-            </plugins>
-        </pluginManagement>
-
        <plugins>
-            <!--
-            <plugin>
-                <groupId>org.codehaus.mojo</groupId>
-                <artifactId>shitty-maven-plugin</artifactId>
-                <inherited>false</inherited>
-                <configuration>
-                    <projects>
-                        <fileset>
- <directory>${project.basedir}</ directory>
-                            <includes>
-                                <include>*/pom.xml</include>
-                            </includes>
-                            <!- -
- HACK: Skip tomcat (or jetty) tests for now till we figure out how to switch it dynamically
-                            - ->
-                            <excludes>
- <exclude>${excludeTest}/pom.xml</ exclude>
-                            </excludes>
-                        </fileset>
-                    </projects>
-                </configuration>
-                <executions>
-                    <execution>
-                        <goals>
-                            <goal>clean</goal>
-                            <goal>install</goal>
-                            <goal>test</goal>
-                        </goals>
-                    </execution>
-                </executions>
-            </plugin>
-            -->
-
            <plugin>
                <groupId>org.apache.geronimo.buildsupport</groupId>
                <artifactId>testsuite-maven-plugin</artifactId>


Reply via email to