You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ta...@apache.org on 2020/06/30 15:49:30 UTC

[myfaces] branch master updated: removed very outdated cargo support

This is an automated email from the ASF dual-hosted git repository.

tandraschko pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/myfaces.git


The following commit(s) were added to refs/heads/master by this push:
     new 3b7cbd2  removed very outdated cargo support
3b7cbd2 is described below

commit 3b7cbd297743404d1c0e8daa4d04023172a2b5b9
Author: Thomas Andraschko <ta...@apache.org>
AuthorDate: Tue Jun 30 17:49:22 2020 +0200

    removed very outdated cargo support
---
 test/pom.xml                                       | 283 ++++++++++-----------
 .../apache/myfaces/test/cargo/CargoTestSetup.java  | 179 -------------
 2 files changed, 135 insertions(+), 327 deletions(-)

diff --git a/test/pom.xml b/test/pom.xml
index 2472cb2..179fd6b 100644
--- a/test/pom.xml
+++ b/test/pom.xml
@@ -1,148 +1,135 @@
-<!--
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to you under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *
- * $Id: pom.xml 1326869 2012-04-17 01:30:30Z lu4242 $
- */
--->
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
-    <modelVersion>4.0.0</modelVersion>
-
-    <parent>
-        <groupId>org.apache.myfaces.core</groupId>
-        <artifactId>myfaces-core-project</artifactId>
-        <version>2.3-next-SNAPSHOT</version>
-        <relativePath>../parent/pom.xml</relativePath>
-    </parent>
-
-    <groupId>org.apache.myfaces.core</groupId>
-    <artifactId>myfaces-test</artifactId>
-    <packaging>jar</packaging>
-    <name>Apache MyFaces Core 2.3-next - Test</name>
-
-    <dependencies>
-        <dependency>
-            <groupId>org.apache.myfaces.core</groupId>
-            <artifactId>myfaces-api</artifactId>
-            <version>${project.version}</version>
-            <scope>provided</scope>
-        </dependency>
-
-        <dependency>
-            <groupId>javax.portlet</groupId>
-            <artifactId>portlet-api</artifactId>
-            <version>1.0</version>
-            <scope>provided</scope>
-        </dependency>
-
-        <!-- Java EE APIs -->
-        <dependency>
-            <groupId>org.apache.tomcat</groupId>
-            <artifactId>tomcat-servlet-api</artifactId>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.tomcat</groupId>
-            <artifactId>tomcat-el-api</artifactId>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.geronimo.specs</groupId>
-            <artifactId>geronimo-jsp_2.2_spec</artifactId>
-        </dependency>
-
-        <dependency>
-            <groupId>net.sourceforge.htmlunit</groupId>
-            <artifactId>htmlunit</artifactId>
-            <version>2.33</version>
-            <optional>true</optional>
-        </dependency>
-
-        <dependency>
-            <groupId>jmock</groupId>
-            <artifactId>jmock</artifactId>
-            <scope>compile</scope>
-        </dependency>
-        <dependency>
-            <groupId>jmock</groupId>
-            <artifactId>jmock-cglib</artifactId>
-            <scope>compile</scope>
-        </dependency>
-
-        <!-- For the "org.apache.shale.test.cargo" package, we need to have  -->
-        <!-- JUnit as a compile time dependency, not just scope="test".      -->
-        <dependency>
-            <groupId>junit</groupId>
-            <artifactId>junit</artifactId>
-            <scope>compile</scope>
-        </dependency>
-
-        <dependency>
-            <groupId>org.codehaus.cargo</groupId>
-            <artifactId>cargo-core-uberjar</artifactId>
-            <version>0.8</version>
-            <optional>true</optional>
-        </dependency>
-        <dependency>
-            <groupId>org.codehaus.cargo</groupId>
-            <artifactId>cargo-ant</artifactId>
-            <version>0.8</version>
-            <optional>true</optional>
-        </dependency>
-    </dependencies>
-
-    <build>
-        <plugins>
-            <plugin>
-                <artifactId>maven-source-plugin</artifactId>
-                <executions>
-                    <execution>
-                        <id>attach-source</id>
-                        <goals>
-                            <goal>jar</goal>
-                        </goals>
-                    </execution>
-                </executions>
-            </plugin>
-        </plugins>
-    </build>
-    
-    <profiles>
-        <profile>
-            <id>generate-assembly</id>
-            <activation>
-                <property>
-                    <name>performRelease</name>
-                    <value>true</value>
-                </property>
-            </activation>
-            <build>
-                <plugins>
-                    <plugin>
-                        <artifactId>maven-javadoc-plugin</artifactId>
-                        <executions>
-                            <execution>
-                                <id>attach-javadoc</id>
-                                <goals>
-                                    <goal>jar</goal>
-                                </goals>
-                            </execution>
-                        </executions>
-                    </plugin>
-                </plugins>
-            </build>
-        </profile>
-    </profiles>   
-
-</project>
+<!--
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to you under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ * $Id: pom.xml 1326869 2012-04-17 01:30:30Z lu4242 $
+ */
+-->
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
+    <modelVersion>4.0.0</modelVersion>
+
+    <parent>
+        <groupId>org.apache.myfaces.core</groupId>
+        <artifactId>myfaces-core-project</artifactId>
+        <version>2.3-next-SNAPSHOT</version>
+        <relativePath>../parent/pom.xml</relativePath>
+    </parent>
+
+    <groupId>org.apache.myfaces.core</groupId>
+    <artifactId>myfaces-test</artifactId>
+    <packaging>jar</packaging>
+    <name>Apache MyFaces Core 2.3-next - Test</name>
+
+    <dependencies>
+        <dependency>
+            <groupId>org.apache.myfaces.core</groupId>
+            <artifactId>myfaces-api</artifactId>
+            <version>${project.version}</version>
+            <scope>provided</scope>
+        </dependency>
+
+        <dependency>
+            <groupId>javax.portlet</groupId>
+            <artifactId>portlet-api</artifactId>
+            <version>1.0</version>
+            <scope>provided</scope>
+        </dependency>
+
+        <!-- Java EE APIs -->
+        <dependency>
+            <groupId>org.apache.tomcat</groupId>
+            <artifactId>tomcat-servlet-api</artifactId>
+        </dependency>
+        <dependency>
+            <groupId>org.apache.tomcat</groupId>
+            <artifactId>tomcat-el-api</artifactId>
+        </dependency>
+        <dependency>
+            <groupId>org.apache.geronimo.specs</groupId>
+            <artifactId>geronimo-jsp_2.2_spec</artifactId>
+        </dependency>
+
+        <dependency>
+            <groupId>net.sourceforge.htmlunit</groupId>
+            <artifactId>htmlunit</artifactId>
+            <version>2.33</version>
+            <optional>true</optional>
+        </dependency>
+
+        <dependency>
+            <groupId>jmock</groupId>
+            <artifactId>jmock</artifactId>
+            <scope>compile</scope>
+        </dependency>
+        <dependency>
+            <groupId>jmock</groupId>
+            <artifactId>jmock-cglib</artifactId>
+            <scope>compile</scope>
+        </dependency>
+
+        <!-- For the "org.apache.shale.test.cargo" package, we need to have  -->
+        <!-- JUnit as a compile time dependency, not just scope="test".      -->
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+            <scope>compile</scope>
+        </dependency>
+    </dependencies>
+
+    <build>
+        <plugins>
+            <plugin>
+                <artifactId>maven-source-plugin</artifactId>
+                <executions>
+                    <execution>
+                        <id>attach-source</id>
+                        <goals>
+                            <goal>jar</goal>
+                        </goals>
+                    </execution>
+                </executions>
+            </plugin>
+        </plugins>
+    </build>
+
+    <profiles>
+        <profile>
+            <id>generate-assembly</id>
+            <activation>
+                <property>
+                    <name>performRelease</name>
+                    <value>true</value>
+                </property>
+            </activation>
+            <build>
+                <plugins>
+                    <plugin>
+                        <artifactId>maven-javadoc-plugin</artifactId>
+                        <executions>
+                            <execution>
+                                <id>attach-javadoc</id>
+                                <goals>
+                                    <goal>jar</goal>
+                                </goals>
+                            </execution>
+                        </executions>
+                    </plugin>
+                </plugins>
+            </build>
+        </profile>
+    </profiles>
+
+</project>
diff --git a/test/src/main/java/org/apache/myfaces/test/cargo/CargoTestSetup.java b/test/src/main/java/org/apache/myfaces/test/cargo/CargoTestSetup.java
deleted file mode 100644
index 103fe4f..0000000
--- a/test/src/main/java/org/apache/myfaces/test/cargo/CargoTestSetup.java
+++ /dev/null
@@ -1,179 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License.  You may obtain a copy of the License at
- *
- *   http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-package org.apache.myfaces.test.cargo;
-
-import java.io.File;
-
-import junit.extensions.TestSetup;
-import junit.framework.Test;
-
-import org.codehaus.cargo.container.InstalledLocalContainer;
-import org.codehaus.cargo.container.ContainerType;
-import org.codehaus.cargo.container.tomcat.Tomcat5xInstalledLocalContainer;
-import org.codehaus.cargo.container.deployable.Deployable;
-import org.codehaus.cargo.container.deployable.DeployableType;
-import org.codehaus.cargo.container.configuration.LocalConfiguration;
-import org.codehaus.cargo.container.configuration.ConfigurationType;
-import org.codehaus.cargo.generic.deployable.DefaultDeployableFactory;
-import org.codehaus.cargo.generic.configuration.ConfigurationFactory;
-import org.codehaus.cargo.generic.configuration.DefaultConfigurationFactory;
-import org.codehaus.cargo.generic.DefaultContainerFactory;
-import org.codehaus.cargo.util.log.FileLogger;
-
-/**
- * <p>Convenience <code>TestSetup</code> class which uses Cargo to start
- * and stop a Servlet container.</p>
- * 
- * @since 1.0.0
- */
-public class CargoTestSetup extends TestSetup
-{
-
-    // ------------------------------------------------------------ Constructors
-
-    /**
-     * <p>Construct a new instance of this test setup.</p>
-     *
-     * @param test Tests to be run within this test setup.
-     */
-    public CargoTestSetup(Test test)
-    {
-        super(test);
-    }
-
-    // ------------------------------------------------------ Instance Variables
-
-    /**
-     * <p>The installed local container for this test setup.</p>
-     */
-    private InstalledLocalContainer container;
-
-    // ------------------------------------------------------ Test Setup Methods
-
-    /**
-     * <p>Start the container prior to running the tests.</p>
-     * <p>The following System properties are used:
-     * <ul>
-     * <li>cargo.container.id - ID of the container to use. [tomcat5x]</li>
-     * <li>cargo.container.home - Full path to a local installation of the container.
-     * If not set, uses the value of the TOMCAT_HOME environment variable.
-     * One of cargo.container.home or TOMCAT_HOME is REQUIRED.</li>
-     * <li>cargo.deployable - Full path to the war file to deploy. REQUIRED.</li>
-     * <li>cargo.container.output - Full path to a file to use for output. [none]</li>
-     * <li>cargo.container.log - Full path to a file to use for logging. [none]</li>
-     * <li>cargo.servlet.port - The port on which the container should listen. [8080]</li>
-     * </ul>
-     * </p>
-     *
-     * @throws Exception if an error occurs.
-     */
-    protected void setUp() throws Exception
-    {
-
-        super.setUp();
-
-        // If there is no container id, default to Tomcat 5x
-        String containerId = System.getProperty("cargo.container.id");
-        if (containerId == null)
-        {
-            containerId = Tomcat5xInstalledLocalContainer.ID;
-        }
-        System.out.println("[INFO] container id: " + containerId);
-
-        // Construct the war, using the container id and the path to the war file
-        String deployablePath = System.getProperty("cargo.deployable");
-        System.out.println("[INFO] deployable: " + deployablePath);
-        Deployable war = new DefaultDeployableFactory().createDeployable(
-                containerId, deployablePath, DeployableType.WAR);
-
-        // Container configuration
-        ConfigurationFactory configurationFactory = new DefaultConfigurationFactory();
-
-        LocalConfiguration configuration = (LocalConfiguration) configurationFactory
-                .createConfiguration(containerId, ConfigurationType.STANDALONE);
-
-        // Find and (if provided) set the port to use for the container.
-        String servletPort = System.getProperty("cargo.servlet.port");
-        if (servletPort != null)
-        {
-            configuration.setProperty("cargo.servlet.port", servletPort);
-            System.out.println("[INFO] servlet port: " + servletPort);
-        }
-
-        configuration.addDeployable(war);
-
-        container = (InstalledLocalContainer) new DefaultContainerFactory()
-                .createContainer(containerId, ContainerType.INSTALLED,
-                        configuration);
-
-        // If 'cargo.container.home' is not set, or if an expression was
-        // passed through, try to use the TOMCAT_HOME environment variable.
-        String containerHome = System.getProperty("cargo.container.home");
-        if (containerHome == null || containerHome.startsWith("$"))
-        {
-            containerHome = System.getenv("TOMCAT_HOME");
-        }
-        System.out.println("[INFO] container home: " + containerHome);
-        container.setHome(new File(containerHome));
-
-        // Find and (if provided) set the path to a log file
-        String containerLog = System.getProperty("cargo.container.log");
-        if (containerLog != null)
-        {
-            System.out.println("[INFO] container log: " + containerLog);
-            container.setLogger(new FileLogger(containerLog, false));
-        }
-
-        // Find and (if provided) set the path to an output file
-        String containerOutput = System.getProperty("cargo.container.output");
-        if (containerOutput != null)
-        {
-            System.out.println("[INFO] container output: " + containerOutput);
-            container.setOutput(new File(containerOutput));
-        }
-
-        container.start();
-    }
-
-    /**
-     * Stop the container after running the tests.
-     *
-     * @throws Exception if an error occurs.
-     */
-    protected void tearDown() throws Exception
-    {
-        container.stop();
-        super.tearDown();
-    }
-
-    /**
-     * Return the name of the test setup.
-     * (Temporarily required due to MSUREFIRE-119.)
-     *
-     * @return the name of the test setup.
-     * @deprecated No replacement.
-     */
-
-    public String getName()
-    {
-        return "CargoTestSetup";
-    }
-
-}