Index: archiva-modules/archiva-base/archiva-artifact-converter/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-artifact-converter/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-artifact-converter/pom.xml (working copy) @@ -76,8 +76,9 @@ test - org.springframework - spring-test + org.apache.archiva + archiva-testutil + ${project.version} test Index: archiva-modules/archiva-base/archiva-artifact-converter/src/test/java/org/apache/archiva/converter/artifact/LegacyToDefaultConverterTest.java =================================================================== --- archiva-modules/archiva-base/archiva-artifact-converter/src/test/java/org/apache/archiva/converter/artifact/LegacyToDefaultConverterTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-artifact-converter/src/test/java/org/apache/archiva/converter/artifact/LegacyToDefaultConverterTest.java (working copy) @@ -36,7 +36,6 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; @@ -47,13 +46,14 @@ import java.util.Locale; import java.util.Map; import java.util.regex.Matcher; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * LegacyToDefaultConverterTest * * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class LegacyToDefaultConverterTest extends TestCase Index: archiva-modules/archiva-base/archiva-checksum/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-checksum/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-checksum/pom.xml (working copy) @@ -38,10 +38,11 @@ slf4j-api - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + commons-lang commons-lang Index: archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/AbstractChecksumTestCase.java =================================================================== --- archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/AbstractChecksumTestCase.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/AbstractChecksumTestCase.java (working copy) @@ -23,12 +23,15 @@ import junit.framework.TestCase; import org.apache.archiva.common.utils.FileUtil; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.runner.RunWith; /** * AbstractChecksumTestCase * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public abstract class AbstractChecksumTestCase extends TestCase { Index: archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksumAlgorithmTest.java =================================================================== --- archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksumAlgorithmTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksumAlgorithmTest.java (working copy) @@ -22,27 +22,34 @@ import java.io.File; import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; /** * ChecksumAlgorithmTest * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class ChecksumAlgorithmTest extends TestCase { + @Test public void testGetHashByExtensionSha1() { assertEquals( ChecksumAlgorithm.SHA1, ChecksumAlgorithm.getByExtension( new File( "something.jar.sha1" ) ) ); assertEquals( ChecksumAlgorithm.SHA1, ChecksumAlgorithm.getByExtension( new File( "OTHER.JAR.SHA1" ) ) ); } - + + @Test public void testGetHashByExtensionMd5() { assertEquals( ChecksumAlgorithm.MD5, ChecksumAlgorithm.getByExtension( new File( "something.jar.md5" ) ) ); assertEquals( ChecksumAlgorithm.MD5, ChecksumAlgorithm.getByExtension( new File( "OTHER.JAR.MD5" ) ) ); } + @Test public void testGetHashByExtensionInvalid() { try Index: archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksummedFileTest.java =================================================================== --- archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksummedFileTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksummedFileTest.java (working copy) @@ -24,12 +24,14 @@ import org.apache.commons.io.FileUtils; import org.apache.commons.lang.StringUtils; +import org.junit.Test; /** * ChecksummedFileTest * * @version $Id$ */ + public class ChecksummedFileTest extends AbstractChecksumTestCase { @@ -75,6 +77,7 @@ return jarFile; } + @Test public void testCalculateChecksumMd5() throws IOException { @@ -85,6 +88,7 @@ assertEquals( expectedChecksum, actualChecksum ); } + @Test public void testCalculateChecksumSha1() throws IOException { @@ -95,6 +99,7 @@ assertEquals( expectedChecksum, actualChecksum ); } + @Test public void testCreateChecksum() throws IOException { @@ -108,6 +113,7 @@ assertEquals( "2bb14b388973351b0a4dfe11d171965f59cc61a1 redback-authz-open.jar", hashContents ); } + @Test public void testFixChecksum() throws IOException { @@ -126,12 +132,14 @@ assertTrue( "ChecksummedFile.isValid(SHA1) == true", checksummedFile.isValidChecksum( ChecksumAlgorithm.SHA1 ) ); } + @Test public void testGetChecksumFile() { ChecksummedFile checksummedFile = new ChecksummedFile( new File( "test.jar" ) ); assertEquals( "test.jar.sha1", checksummedFile.getChecksumFile( ChecksumAlgorithm.SHA1 ).getName() ); } + @Test public void testIsValidChecksum() throws IOException { @@ -141,6 +149,7 @@ assertTrue( "ChecksummedFile.isValid(SHA1)", checksummedFile.isValidChecksum( ChecksumAlgorithm.SHA1 ) ); } + @Test public void testIsValidChecksumInvalidSha1Format() throws IOException { @@ -155,6 +164,7 @@ } + @Test public void testIsValidChecksumNoChecksumFiles() throws IOException { @@ -167,6 +177,7 @@ } + @Test public void testIsValidChecksumSha1AndMd5() throws IOException { @@ -178,6 +189,7 @@ ChecksumAlgorithm.MD5 } ) ); } + @Test public void testIsValidChecksumSha1NoMd5() throws IOException { @@ -190,6 +202,7 @@ } + @Test public void testParseChecksum() throws IOException { @@ -204,6 +217,7 @@ } + @Test public void testParseChecksumAltDash1() throws IOException { @@ -215,6 +229,7 @@ assertEquals( "Checksum doesn't match", SERVLETAPI_SHA1, s ); } + @Test public void testParseChecksumAltDash2() throws IOException { @@ -226,6 +241,7 @@ assertEquals( "Checksum doesn't match", SERVLETAPI_SHA1, s ); } + @Test public void testRemoteMetadataChecksumFilePathSha1() throws IOException { @@ -246,6 +262,7 @@ } } + @Test public void testRemoteMetadataChecksumFilePathMd5() throws IOException { Index: archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksumTest.java =================================================================== --- archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksumTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-checksum/src/test/java/org/apache/archiva/checksum/ChecksumTest.java (working copy) @@ -23,6 +23,7 @@ import java.io.IOException; import java.util.ArrayList; import java.util.List; +import org.junit.Test; /** * ChecksumTest @@ -34,6 +35,7 @@ { private static final String UNSET_SHA1 = "da39a3ee5e6b4b0d3255bfef95601890afd80709"; + @Test public void testConstructSha1() { Checksum checksum = new Checksum( ChecksumAlgorithm.SHA1 ); @@ -41,6 +43,7 @@ .getAlgorithm() ); } + @Test public void testConstructMd5() { Checksum checksum = new Checksum( ChecksumAlgorithm.MD5 ); @@ -48,6 +51,7 @@ .getAlgorithm() ); } + @Test public void testUpdate() { Checksum checksum = new Checksum( ChecksumAlgorithm.SHA1 ); @@ -57,6 +61,7 @@ assertEquals( "Checksum", "e396119ae0542e85a74759602fd2f81e5d36d762", checksum.getChecksum() ); } + @Test public void testUpdateMany() throws IOException { @@ -76,6 +81,7 @@ assertEquals( "Checksum MD5", "21c2c5ca87ec018adacb2e2fb3432219", checksumMd5.getChecksum() ); } + @Test public void testUpdateWholeUpdatePartial() { Checksum checksum = new Checksum( ChecksumAlgorithm.SHA1 ); Index: archiva-modules/archiva-base/archiva-configuration/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-configuration/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-configuration/pom.xml (working copy) @@ -103,20 +103,11 @@ archiva-policies - org.springframework - spring-test + org.apache.archiva + archiva-testutil + ${project.version} test - - - xmlunit - xmlunit - test - - - org.slf4j - slf4j-simple - test - + Index: archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/ArchivaConfigurationTest.java =================================================================== --- archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/ArchivaConfigurationTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/ArchivaConfigurationTest.java (working copy) @@ -32,17 +32,18 @@ import org.slf4j.LoggerFactory; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; + import javax.inject.Inject; import java.io.File; import java.util.List; import java.util.Map; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * Test the configuration store. */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class ArchivaConfigurationTest extends TestCase Index: archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/ConfigurationTest.java =================================================================== --- archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/ConfigurationTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/ConfigurationTest.java (working copy) @@ -22,16 +22,16 @@ import junit.framework.TestCase; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; import java.util.Arrays; import java.util.HashSet; import java.util.Map; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; /** * Test the generated Configuration class from Modello. This is primarily to test the hand coded methods. */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class ConfigurationTest extends TestCase { Index: archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/FileTypesTest.java =================================================================== --- archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/FileTypesTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/FileTypesTest.java (working copy) @@ -23,11 +23,11 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml" } ) public class FileTypesTest extends TestCase Index: archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/functors/ProxyConnectorConfigurationOrderComparatorTest.java =================================================================== --- archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/functors/ProxyConnectorConfigurationOrderComparatorTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/functors/ProxyConnectorConfigurationOrderComparatorTest.java (working copy) @@ -26,15 +26,20 @@ import java.util.ArrayList; import java.util.Collections; import java.util.List; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; /** * ProxyConnectorConfigurationOrderComparatorTest * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class ProxyConnectorConfigurationOrderComparatorTest extends TestCase { + @Test public void testSortOfAllZeros() { List proxies = new ArrayList(); @@ -49,6 +54,7 @@ assertProxyOrder( new String[]{ "corporate", "snapshots", "3rdparty", "sandbox" }, proxies ); } + @Test public void testSortNormal() { List proxies = new ArrayList(); @@ -63,6 +69,7 @@ assertProxyOrder( new String[]{ "snapshots", "3rdparty", "corporate", "sandbox" }, proxies ); } + @Test public void testSortPartial() { List proxies = new ArrayList(); Index: archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/functors/RepositoryConfigurationComparatorTest.java =================================================================== --- archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/functors/RepositoryConfigurationComparatorTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/functors/RepositoryConfigurationComparatorTest.java (working copy) @@ -24,13 +24,18 @@ import org.apache.archiva.configuration.ManagedRepositoryConfiguration; import java.util.Comparator; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; /** * Test the repositry comparator. */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RepositoryConfigurationComparatorTest extends TestCase { + @Test public void testComparator() { Comparator comparator = new RepositoryConfigurationComparator(); Index: archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/LegacyArtifactPathTest.java =================================================================== --- archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/LegacyArtifactPathTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/LegacyArtifactPathTest.java (working copy) @@ -20,16 +20,16 @@ */ import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; /** * Test the generated LegacyArtifactPath class from Modello. This is primarily to test the hand coded methods. * * @since 1.1 */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class LegacyArtifactPathTest extends TestCase { Index: archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/MavenProxyPropertyLoaderTest.java =================================================================== --- archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/MavenProxyPropertyLoaderTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-configuration/src/test/java/org/apache/archiva/configuration/MavenProxyPropertyLoaderTest.java (working copy) @@ -30,10 +30,11 @@ import java.io.IOException; import java.util.Map; import java.util.Properties; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; /** */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class MavenProxyPropertyLoaderTest extends TestCase { Index: archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/pom.xml (working copy) @@ -67,8 +67,9 @@ - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test Index: archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/archiva/consumers/core/AbstractArtifactConsumerTest.java =================================================================== --- archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/archiva/consumers/core/AbstractArtifactConsumerTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/archiva/consumers/core/AbstractArtifactConsumerTest.java (working copy) @@ -35,12 +35,12 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public abstract class AbstractArtifactConsumerTest extends TestCase Index: archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/archiva/consumers/core/repository/AbstractRepositoryPurgeTest.java =================================================================== --- archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/archiva/consumers/core/repository/AbstractRepositoryPurgeTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-consumers/archiva-core-consumers/src/test/java/org/apache/archiva/consumers/core/repository/AbstractRepositoryPurgeTest.java (working copy) @@ -21,10 +21,7 @@ import junit.framework.TestCase; import org.apache.archiva.admin.model.beans.ManagedRepository; -import org.apache.archiva.admin.model.managed.ManagedRepositoryAdmin; -import org.apache.archiva.admin.repository.managed.DefaultManagedRepositoryAdmin; import org.apache.archiva.common.plexusbridge.PlexusSisuBridge; -import org.apache.archiva.configuration.ArchivaConfiguration; import org.apache.archiva.metadata.repository.MetadataRepository; import org.apache.archiva.metadata.repository.RepositorySession; import org.apache.archiva.repository.ManagedRepositoryContent; @@ -39,18 +36,17 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; -import java.io.IOException; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; /** */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public abstract class AbstractRepositoryPurgeTest extends TestCase Index: archiva-modules/archiva-base/archiva-consumers/archiva-lucene-consumers/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-consumers/archiva-lucene-consumers/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-consumers/archiva-lucene-consumers/pom.xml (working copy) @@ -74,8 +74,9 @@ archiva-repository-admin-api - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test Index: archiva-modules/archiva-base/archiva-consumers/archiva-lucene-consumers/src/test/java/org/apache/archiva/consumers/lucene/NexusIndexerConsumerTest.java =================================================================== --- archiva-modules/archiva-base/archiva-consumers/archiva-lucene-consumers/src/test/java/org/apache/archiva/consumers/lucene/NexusIndexerConsumerTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-consumers/archiva-lucene-consumers/src/test/java/org/apache/archiva/consumers/lucene/NexusIndexerConsumerTest.java (working copy) @@ -36,7 +36,6 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; @@ -46,11 +45,12 @@ import java.util.HashSet; import java.util.List; import java.util.Set; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * NexusIndexerConsumerTest */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class NexusIndexerConsumerTest extends TestCase Index: archiva-modules/archiva-base/archiva-converter/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-converter/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-converter/pom.xml (working copy) @@ -68,10 +68,11 @@ provided - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + Index: archiva-modules/archiva-base/archiva-converter/src/test/java/org/apache/archiva/converter/RepositoryConverterTest.java =================================================================== --- archiva-modules/archiva-base/archiva-converter/src/test/java/org/apache/archiva/converter/RepositoryConverterTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-converter/src/test/java/org/apache/archiva/converter/RepositoryConverterTest.java (working copy) @@ -31,7 +31,6 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.inject.Named; @@ -39,6 +38,7 @@ import java.io.IOException; import java.util.ArrayList; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * Test the repository converter. @@ -48,7 +48,7 @@ * @todo should reject if dependencies are missing - rely on reporting? * @todo group metadata */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = {"classpath*:/META-INF/spring-context.xml","classpath:/spring-context.xml"} ) public class RepositoryConverterTest extends TestCase Index: archiva-modules/archiva-base/archiva-indexer/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-indexer/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-indexer/pom.xml (working copy) @@ -113,8 +113,9 @@ test - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test Index: archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/AbstractNexusRepositorySearch.java =================================================================== --- archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/AbstractNexusRepositorySearch.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/AbstractNexusRepositorySearch.java (working copy) @@ -48,11 +48,12 @@ import javax.inject.Inject; import java.io.File; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * @author Olivier Lamy */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public abstract class AbstractNexusRepositorySearch extends TestCase Index: archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/NexusRepositorySearchPaginateTest.java =================================================================== --- archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/NexusRepositorySearchPaginateTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/NexusRepositorySearchPaginateTest.java (working copy) @@ -22,14 +22,14 @@ import org.apache.archiva.indexer.util.SearchUtil; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; import java.util.Arrays; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; /** * @author Olivier Lamy */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class NexusRepositorySearchPaginateTest extends TestCase { Index: archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/NexusRepositorySearchTest.java =================================================================== --- archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/NexusRepositorySearchTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-indexer/src/test/java/org/apache/archiva/indexer/search/NexusRepositorySearchTest.java (working copy) @@ -24,16 +24,16 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import java.io.File; import java.util.ArrayList; import java.util.Arrays; import java.util.Collection; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class NexusRepositorySearchTest extends AbstractNexusRepositorySearch Index: archiva-modules/archiva-base/archiva-model/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-model/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-model/pom.xml (working copy) @@ -42,8 +42,9 @@ archiva-common - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test Index: archiva-modules/archiva-base/archiva-model/src/test/java/org/apache/archiva/model/RepositoryURLTest.java =================================================================== --- archiva-modules/archiva-base/archiva-model/src/test/java/org/apache/archiva/model/RepositoryURLTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-model/src/test/java/org/apache/archiva/model/RepositoryURLTest.java (working copy) @@ -20,12 +20,16 @@ */ import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; /** * RepositoryURLTest * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RepositoryURLTest extends TestCase { @@ -49,43 +53,51 @@ assertEquals( "Password", expectedPassword, rurl.getPassword() ); } + @Test public void testFileUrlNormal() { assertURL( "file:///home/joakim/code/test/this/", "file", NO_HOST, NO_PORT, "/home/joakim/code/test/this/", NO_USER, NO_PASS ); } + @Test public void testFileUrlShort() { assertURL( "file:/home/joakim/code/test/this/", "file", NO_HOST, NO_PORT, "/home/joakim/code/test/this/", NO_USER, NO_PASS ); } + @Test public void testHttpUrlPathless() { assertURL( "http://machine", "http", "machine", NO_PORT, "/", NO_USER, NO_PASS ); } + @Test public void testHttpUrlWithPort() { assertURL( "http://machine:8080/", "http", "machine", "8080", "/", NO_USER, NO_PASS ); } + @Test public void testHttpUrlWithUsernamePassword() { assertURL( "http://user:pass@machine/secured/", "http", "machine", NO_PORT, "/secured/", "user", "pass" ); } + @Test public void testHttpUrlWithUsernameNoPassword() { assertURL( "http://user@machine/secured/", "http", "machine", NO_PORT, "/secured/", "user", NO_PASS ); } + @Test public void testHttpUrlWithUsernamePasswordAndPort() { assertURL( "http://user:pass@machine:9090/secured/", "http", "machine", "9090", "/secured/", "user", "pass" ); } + @Test public void testBogusWithPath() { // This should not fail. The intent of RepositoryURL is to have it support oddball protocols that Index: archiva-modules/archiva-base/archiva-policies/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-policies/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-policies/pom.xml (working copy) @@ -81,10 +81,11 @@ - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + org.slf4j jcl-over-slf4j Index: archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/CachedFailuresPolicyTest.java =================================================================== --- archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/CachedFailuresPolicyTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/CachedFailuresPolicyTest.java (working copy) @@ -24,19 +24,19 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.inject.Named; import java.io.File; import java.util.Properties; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * CachedFailuresPolicyTest * * @version $Id$ */ -@RunWith( value = SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class CachedFailuresPolicyTest extends TestCase Index: archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/ChecksumPolicyTest.java =================================================================== --- archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/ChecksumPolicyTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/ChecksumPolicyTest.java (working copy) @@ -26,7 +26,6 @@ import org.junit.rules.TestName; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import java.io.BufferedReader; import java.io.File; @@ -34,6 +33,7 @@ import java.util.Properties; import javax.inject.Inject; import javax.inject.Named; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.junit.Assert.*; @@ -42,7 +42,7 @@ * * @version $Id$ */ -@RunWith( value = SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = {"classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml"} ) public class ChecksumPolicyTest { Index: archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/ReleasePolicyTest.java =================================================================== --- archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/ReleasePolicyTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/ReleasePolicyTest.java (working copy) @@ -25,19 +25,19 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.inject.Named; import java.io.File; import java.util.Properties; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * ReleasePolicyTest * * @version $Id$ */ -@RunWith( value = SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class ReleasePolicyTest extends TestCase Index: archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/SnapshotsPolicyTest.java =================================================================== --- archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/SnapshotsPolicyTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-policies/src/test/java/org/apache/archiva/policies/SnapshotsPolicyTest.java (working copy) @@ -25,19 +25,19 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.inject.Named; import java.io.File; import java.util.Properties; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * SnapshotsPolicyTest * * @version $Id$ */ -@RunWith( value = SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class SnapshotsPolicyTest extends TestCase Index: archiva-modules/archiva-base/archiva-proxy-common/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-proxy-common/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-proxy-common/pom.xml (working copy) @@ -62,10 +62,11 @@ commons-logging-api - org.springframework - spring-test + org.apache.archiva + archiva-testutil + ${project.version} test - + Index: archiva-modules/archiva-base/archiva-proxy-common/src/test/java/org/apache/archiva/proxy/common/WagonFactoryTest.java =================================================================== --- archiva-modules/archiva-base/archiva-proxy-common/src/test/java/org/apache/archiva/proxy/common/WagonFactoryTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-proxy-common/src/test/java/org/apache/archiva/proxy/common/WagonFactoryTest.java (working copy) @@ -24,15 +24,15 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * Test the WagonFactory works through Spring to be bound into the RepositoryProxyConnectors implementation. * */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml" } ) public class WagonFactoryTest extends TestCase Index: archiva-modules/archiva-base/archiva-proxy/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-proxy/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-proxy/pom.xml (working copy) @@ -129,10 +129,11 @@ provided - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + org.apache.archiva archiva-repository-admin-default Index: archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/archiva/proxy/AbstractProxyTestCase.java =================================================================== --- archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/archiva/proxy/AbstractProxyTestCase.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/archiva/proxy/AbstractProxyTestCase.java (working copy) @@ -63,6 +63,7 @@ import java.util.Collection; import java.util.Date; import java.util.Locale; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.junit.Assert.*; @@ -71,7 +72,7 @@ * * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public abstract class AbstractProxyTestCase { Index: archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/archiva/proxy/HttpProxyTransferTest.java =================================================================== --- archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/archiva/proxy/HttpProxyTransferTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/archiva/proxy/HttpProxyTransferTest.java (working copy) @@ -45,7 +45,6 @@ import org.mortbay.jetty.handler.AbstractHandler; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.servlet.ServletException; @@ -53,6 +52,7 @@ import javax.servlet.http.HttpServletResponse; import java.io.File; import java.io.IOException; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.junit.Assert.*; @@ -61,7 +61,7 @@ * * @version $Id: ManagedDefaultTransferTest.java 677852 2008-07-18 08:16:24Z brett $ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class HttpProxyTransferTest { Index: archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/pom.xml (working copy) @@ -93,6 +93,12 @@ + org.apache.archiva + archiva-testutil + ${project.version} + test + + org.springframework spring-context-support ${spring.version} Index: archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/src/test/java/org/apache/archiva/admin/repository/AbstractRepositoryAdminTest.java =================================================================== --- archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/src/test/java/org/apache/archiva/admin/repository/AbstractRepositoryAdminTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/src/test/java/org/apache/archiva/admin/repository/AbstractRepositoryAdminTest.java (working copy) @@ -35,16 +35,16 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * @author Olivier Lamy */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public abstract class AbstractRepositoryAdminTest extends TestCase Index: archiva-modules/archiva-base/archiva-repository-layer/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-repository-layer/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-layer/pom.xml (working copy) @@ -67,10 +67,11 @@ test - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + org.apache.archiva metadata-model Index: archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/AbstractRepositoryLayerTestCase.java =================================================================== --- archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/AbstractRepositoryLayerTestCase.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/AbstractRepositoryLayerTestCase.java (working copy) @@ -28,17 +28,17 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import java.io.File; import javax.inject.Inject; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * AbstractRepositoryLayerTestCase * * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public abstract class AbstractRepositoryLayerTestCase { Index: archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/content/FilenameParserTest.java =================================================================== --- archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/content/FilenameParserTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/content/FilenameParserTest.java (working copy) @@ -20,16 +20,16 @@ */ import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; /** * FilenameParserTest * * @version $Id$ */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class FilenameParserTest extends TestCase { Index: archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/metadata/RepositoryMetadataReaderTest.java =================================================================== --- archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/metadata/RepositoryMetadataReaderTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/metadata/RepositoryMetadataReaderTest.java (working copy) @@ -25,16 +25,16 @@ import org.apache.archiva.xml.XMLException; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; import java.io.File; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; /** * RepositoryMetadataReaderTest * * @version $Id$ */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RepositoryMetadataReaderTest extends TestCase { Index: archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/metadata/RepositoryMetadataWriterTest.java =================================================================== --- archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/metadata/RepositoryMetadataWriterTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/metadata/RepositoryMetadataWriterTest.java (working copy) @@ -25,17 +25,17 @@ import org.custommonkey.xmlunit.XMLAssert; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; import java.io.File; import java.io.StringWriter; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; /** * RepositoryMetadataWriterTest * * @version $Id$ */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RepositoryMetadataWriterTest extends TestCase { Index: archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/RepositoryURLTest.java =================================================================== --- archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/RepositoryURLTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-layer/src/test/java/org/apache/archiva/repository/RepositoryURLTest.java (working copy) @@ -25,12 +25,16 @@ import java.net.MalformedURLException; import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; /** * RepositoryURLTest * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RepositoryURLTest extends TestCase { @@ -46,7 +50,8 @@ assertEquals( port, url.getPort() ); assertEquals( path, url.getPath() ); } - + + @Test public void testProtocolHttp() throws MalformedURLException { @@ -54,6 +59,7 @@ "/path/to/resource.txt" ); } + @Test public void testProtocolWagonWebdav() throws MalformedURLException { @@ -61,6 +67,7 @@ "/path/to/resource.txt" ); } + @Test public void testProtocolHttpWithPort() throws MalformedURLException { @@ -68,6 +75,7 @@ "/path/to/resource.txt" ); } + @Test public void testProtocolHttpWithUsername() throws MalformedURLException { @@ -75,6 +83,7 @@ "/path/to/resource.txt" ); } + @Test public void testProtocolHttpWithUsernamePassword() throws MalformedURLException { @@ -82,6 +91,7 @@ "/path/to/resource.txt" ); } + @Test public void testProtocolHttpWithUsernamePasswordPort() throws MalformedURLException { Index: archiva-modules/archiva-base/archiva-repository-scanner/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-repository-scanner/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-scanner/pom.xml (working copy) @@ -58,8 +58,9 @@ --> - org.springframework - spring-test + org.apache.archiva + archiva-testutil + ${project.version} test Index: archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/RepositoryContentConsumersTest.java =================================================================== --- archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/RepositoryContentConsumersTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/RepositoryContentConsumersTest.java (working copy) @@ -40,7 +40,6 @@ import org.springframework.core.env.Environment; import org.springframework.core.io.Resource; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; @@ -53,13 +52,14 @@ import java.util.List; import java.util.Locale; import java.util.Map; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * RepositoryContentConsumersTest * * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class RepositoryContentConsumersTest extends TestCase Index: archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/RepositoryScannerTest.java =================================================================== --- archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/RepositoryScannerTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-repository-scanner/src/test/java/org/apache/archiva/repository/scanner/RepositoryScannerTest.java (working copy) @@ -29,7 +29,6 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; @@ -41,13 +40,14 @@ import java.util.List; import java.util.Locale; import java.util.TimeZone; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * RepositoryScannerTest * * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml" } ) public class RepositoryScannerTest extends TestCase Index: archiva-modules/archiva-base/archiva-transaction/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-transaction/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-transaction/pom.xml (working copy) @@ -52,8 +52,9 @@ plexus-utils - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test Index: archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/AbstractFileEventTest.java =================================================================== --- archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/AbstractFileEventTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/AbstractFileEventTest.java (working copy) @@ -29,16 +29,22 @@ import java.io.IOException; import java.util.Arrays; import java.util.List; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.runner.RunWith; /** * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public abstract class AbstractFileEventTest extends TestCase { protected List digesters; @SuppressWarnings( "unchecked" ) + @Before + @Override public void setUp() throws Exception { Index: archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/CopyFileEventTest.java =================================================================== --- archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/CopyFileEventTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/CopyFileEventTest.java (working copy) @@ -24,6 +24,9 @@ import java.io.File; import java.io.IOException; +import org.junit.After; +import org.junit.Before; +import org.junit.Test; /** */ @@ -40,6 +43,8 @@ private String source, oldChecksum; + @Override + @Before public void setUp() throws Exception { @@ -67,7 +72,8 @@ oldChecksum = readFile( testDestChecksum ); } - + + @Test public void testCopyCommitRollback() throws Exception { @@ -92,6 +98,7 @@ assertChecksumRollback( testDest ); } + @Test public void testCopyCommitRollbackWithBackup() throws Exception { @@ -126,6 +133,7 @@ assertChecksumRollback( testDest ); } + @Test public void testCreateRollbackCommit() throws Exception { @@ -148,7 +156,9 @@ assertTrue( "Test that the destination contents are copied correctly", source.equals( target ) ); } - protected void tearDown() + @After + @Override + public void tearDown() throws Exception { super.tearDown(); @@ -156,6 +166,7 @@ FileUtils.deleteDirectory( new File( FileUtil.getBasedir(), "target/transaction-tests" ) ); } + @Override protected void assertChecksumCommit( File file ) throws IOException { @@ -166,6 +177,7 @@ assertFalse( "Test that the destination checksum contents are created correctly", oldChecksum.equals( target ) ); } + @Override protected void assertChecksumRollback( File file ) throws IOException { Index: archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/CreateFileEventTest.java =================================================================== --- archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/CreateFileEventTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-transaction/src/test/java/org/apache/archiva/transaction/CreateFileEventTest.java (working copy) @@ -23,6 +23,8 @@ import org.apache.commons.io.FileUtils; import org.apache.archiva.common.utils.FileUtil; +import org.junit.After; +import org.junit.Test; /** */ @@ -31,6 +33,7 @@ { private File testDir = new File( FileUtil.getBasedir(), "target/transaction-tests/create-file" ); + @Test public void testCreateCommitRollback() throws Exception { @@ -56,6 +59,7 @@ assertTrue( "target directory still exists", new File( FileUtil.getBasedir(), "target" ).exists() ); } + @Test public void testCreateCommitRollbackWithBackup() throws Exception { @@ -90,6 +94,7 @@ assertChecksumRollback( testFile ); } + @Test public void testCreateRollbackCommit() throws Exception { @@ -110,7 +115,9 @@ assertChecksumCommit( testFile ); } - protected void tearDown() + @Override + @After + public void tearDown() throws Exception { super.tearDown(); Index: archiva-modules/archiva-base/archiva-xml-tools/pom.xml =================================================================== --- archiva-modules/archiva-base/archiva-xml-tools/pom.xml (revision 1355676) +++ archiva-modules/archiva-base/archiva-xml-tools/pom.xml (working copy) @@ -53,6 +53,12 @@ org.codehaus.plexus plexus-utils + + org.apache.archiva + archiva-testutil + ${project.version} + test + Index: archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/AbstractArchivaXmlTestCase.java =================================================================== --- archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/AbstractArchivaXmlTestCase.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/AbstractArchivaXmlTestCase.java (working copy) @@ -22,12 +22,15 @@ import java.io.File; import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.runner.RunWith; /** * AbstractArchivaXmlTestCase * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public abstract class AbstractArchivaXmlTestCase extends TestCase { Index: archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/LatinEntitiesTest.java =================================================================== --- archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/LatinEntitiesTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/LatinEntitiesTest.java (working copy) @@ -20,15 +20,20 @@ */ import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; /** * LatinEntitiesTest * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class LatinEntitiesTest extends TestCase { + @Test public void testResolveEntity() { // Good Entities. Index: archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/LatinEntityResolutionReaderTest.java =================================================================== --- archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/LatinEntityResolutionReaderTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/LatinEntityResolutionReaderTest.java (working copy) @@ -32,6 +32,7 @@ import java.io.Reader; import java.io.StringWriter; import java.net.URL; +import org.junit.Test; /** * LatinEntityResolutionReaderTest @@ -139,7 +140,8 @@ fail( "IOException: " + e.getMessage() ); } } - + + @Test public void testReaderNormalBufsize() throws IOException { @@ -155,6 +157,7 @@ assertProperRead( expected, "no-prolog-with-entities.xml", 4096 ); } + @Test public void testReaderSmallBufsize() throws IOException { @@ -170,6 +173,7 @@ assertProperRead( expected, "no-prolog-with-entities.xml", 1024 ); } + @Test public void testReaderRediculouslyTinyBufsize() throws IOException { @@ -185,6 +189,7 @@ assertProperRead( expected, "no-prolog-with-entities.xml", 32 ); } + @Test public void testReaderHugeBufsize() throws IOException { @@ -200,7 +205,7 @@ assertProperRead( expected, "no-prolog-with-entities.xml", 409600 ); } - + @Test public void testReaderLeftOver() throws IOException { @@ -225,7 +230,7 @@ } } - + @Test public void testNoLatinEntitiesHugeLine() { assertProperRead( "commons-codec-1.2.pom", "commons-codec-1.2.pom", 4096 ); Index: archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/XMLReaderTest.java =================================================================== --- archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/XMLReaderTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/XMLReaderTest.java (working copy) @@ -24,6 +24,7 @@ import java.util.List; import org.dom4j.Element; +import org.junit.Test; /** * XMLReaderTest @@ -49,7 +50,8 @@ assertTrue( "Contains [" + expectedText + "]", texts.contains( expectedText ) ); } } - + + @Test public void testNoPrologBasicRead() throws XMLException { @@ -60,6 +62,7 @@ assertElementTexts( fruits, new String[] { "apple", "cherry", "pear", "peach" } ); } + @Test public void testNoPrologEntitiesRead() throws XMLException { @@ -70,6 +73,7 @@ assertElementTexts( names, new String[] { TRYGVIS, INFINITE_ARCHIVA } ); } + @Test public void testNoPrologUtf8Read() throws XMLException { @@ -80,6 +84,7 @@ assertElementTexts( names, new String[] { TRYGVIS, INFINITE_ARCHIVA } ); } + @Test public void testPrologUtf8Read() throws XMLException { @@ -91,6 +96,7 @@ } // MRM-1136 + @Test public void testProxiedMetadataRead() throws XMLException { Index: archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/XMLWriterTest.java =================================================================== --- archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/XMLWriterTest.java (revision 1355676) +++ archiva-modules/archiva-base/archiva-xml-tools/src/test/java/org/apache/archiva/xml/XMLWriterTest.java (working copy) @@ -24,6 +24,7 @@ import org.dom4j.Element; import java.io.StringWriter; +import org.junit.Test; /** * XMLWriterTest @@ -33,6 +34,7 @@ public class XMLWriterTest extends AbstractArchivaXmlTestCase { + @Test public void testWrite() throws Exception { Index: archiva-modules/archiva-scheduler/archiva-scheduler-indexing/pom.xml =================================================================== --- archiva-modules/archiva-scheduler/archiva-scheduler-indexing/pom.xml (revision 1355676) +++ archiva-modules/archiva-scheduler/archiva-scheduler-indexing/pom.xml (working copy) @@ -91,9 +91,15 @@ slf4j-log4j12 test - org.apache.archiva + archiva-testutil + ${project.version} + test + + + + org.apache.archiva archiva-repository-admin-default test Index: archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/test/java/org/apache/archiva/scheduler/indexing/ArchivaIndexingTaskExecutorTest.java =================================================================== --- archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/test/java/org/apache/archiva/scheduler/indexing/ArchivaIndexingTaskExecutorTest.java (revision 1355676) +++ archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/test/java/org/apache/archiva/scheduler/indexing/ArchivaIndexingTaskExecutorTest.java (working copy) @@ -42,7 +42,6 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.BufferedInputStream; @@ -55,11 +54,12 @@ import java.util.Set; import java.util.zip.ZipEntry; import java.util.zip.ZipInputStream; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * ArchivaIndexingTaskExecutorTest */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class ArchivaIndexingTaskExecutorTest extends TestCase Index: archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/test/java/org/apache/archiva/scheduler/indexing/DownloadRemoteIndexTaskTest.java =================================================================== --- archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/test/java/org/apache/archiva/scheduler/indexing/DownloadRemoteIndexTaskTest.java (revision 1355676) +++ archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/test/java/org/apache/archiva/scheduler/indexing/DownloadRemoteIndexTaskTest.java (working copy) @@ -43,18 +43,18 @@ import org.slf4j.LoggerFactory; import org.springframework.scheduling.concurrent.ThreadPoolTaskScheduler; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; import java.io.IOException; import java.util.Arrays; import java.util.concurrent.TimeUnit; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * @author Olivier Lamy */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class DownloadRemoteIndexTaskTest extends TestCase Index: archiva-modules/archiva-scheduler/archiva-scheduler-repository/pom.xml =================================================================== --- archiva-modules/archiva-scheduler/archiva-scheduler-repository/pom.xml (revision 1355676) +++ archiva-modules/archiva-scheduler/archiva-scheduler-repository/pom.xml (working copy) @@ -86,10 +86,11 @@ slf4j-api - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + org.mockito mockito-all Index: archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorAbstractTest.java =================================================================== --- archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorAbstractTest.java (revision 1355676) +++ archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorAbstractTest.java (working copy) @@ -31,22 +31,22 @@ import org.junit.Before; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.inject.Named; import java.io.File; import java.util.Calendar; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.mockito.Mockito.mock; /** * ArchivaRepositoryScanningTaskExecutorPhase1Test * - * @version $Id: ArchivaRepositoryScanningTaskExecutorPhase1Test.java 1214303 2011-12-14 15:37:51Z olamy $ + * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public abstract class ArchivaRepositoryScanningTaskExecutorAbstractTest extends TestCase @@ -78,6 +78,7 @@ protected MetadataRepository metadataRepository; @Before + @Override public void setUp() throws Exception { @@ -123,6 +124,7 @@ } @After + @Override public void tearDown() throws Exception { Index: archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorPhase1Test.java =================================================================== --- archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorPhase1Test.java (revision 1355676) +++ archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorPhase1Test.java (working copy) @@ -30,9 +30,8 @@ /** * ArchivaRepositoryScanningTaskExecutorPhase1Test * - * @version $Id: ArchivaRepositoryScanningTaskExecutorPhase1Test.java 1214303 2011-12-14 15:37:51Z olamy $ + * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class ArchivaRepositoryScanningTaskExecutorPhase1Test extends ArchivaRepositoryScanningTaskExecutorAbstractTest Index: archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorPhase2Test.java =================================================================== --- archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorPhase2Test.java (revision 1355676) +++ archiva-modules/archiva-scheduler/archiva-scheduler-repository/src/test/java/org/apache/archiva/scheduler/repository/ArchivaRepositoryScanningTaskExecutorPhase2Test.java (working copy) @@ -36,9 +36,9 @@ /** * ArchivaRepositoryScanningTaskExecutorPhase2Test * - * @version $Id: ArchivaRepositoryScanningTaskExecutorTest.java 1214303 2011-12-14 15:37:51Z olamy $ + * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) + @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class ArchivaRepositoryScanningTaskExecutorPhase2Test extends ArchivaRepositoryScanningTaskExecutorAbstractTest Index: archiva-modules/archiva-web/archiva-rest/archiva-rest-services/pom.xml =================================================================== --- archiva-modules/archiva-web/archiva-rest/archiva-rest-services/pom.xml (revision 1355676) +++ archiva-modules/archiva-web/archiva-rest/archiva-rest-services/pom.xml (working copy) @@ -225,8 +225,13 @@ spring-test test - + org.apache.archiva + archiva-testutil + ${project.version} + test + + org.springframework spring-web test Index: archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/AbstractArchivaRestTest.java =================================================================== --- archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/AbstractArchivaRestTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-rest/archiva-rest-services/src/test/java/org/apache/archiva/rest/services/AbstractArchivaRestTest.java (working copy) @@ -45,10 +45,13 @@ import java.io.File; import java.util.Collections; import java.util.Date; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.runner.RunWith; /** * @author Olivier Lamy */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public abstract class AbstractArchivaRestTest extends AbstractRestServicesTest { Index: archiva-modules/archiva-web/archiva-rss/pom.xml =================================================================== --- archiva-modules/archiva-web/archiva-rss/pom.xml (revision 1355676) +++ archiva-modules/archiva-web/archiva-rss/pom.xml (working copy) @@ -57,6 +57,12 @@ test + org.apache.archiva + archiva-testutil + ${project.version} + test + + org.slf4j jcl-over-slf4j test Index: archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/processor/NewArtifactsRssFeedProcessorTest.java =================================================================== --- archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/processor/NewArtifactsRssFeedProcessorTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/processor/NewArtifactsRssFeedProcessorTest.java (working copy) @@ -34,7 +34,6 @@ import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; import java.util.ArrayList; import java.util.Calendar; @@ -44,8 +43,9 @@ import java.util.List; import java.util.Map; import java.util.TimeZone; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class NewArtifactsRssFeedProcessorTest extends TestCase { Index: archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/processor/NewVersionsOfArtifactRssFeedProcessorTest.java =================================================================== --- archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/processor/NewVersionsOfArtifactRssFeedProcessorTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/processor/NewVersionsOfArtifactRssFeedProcessorTest.java (working copy) @@ -29,7 +29,6 @@ import org.junit.Before; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; import java.util.Arrays; import java.util.Collections; @@ -37,8 +36,9 @@ import java.util.HashMap; import java.util.List; import java.util.Map; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class NewVersionsOfArtifactRssFeedProcessorTest extends TestCase { Index: archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/RssFeedGeneratorTest.java =================================================================== --- archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/RssFeedGeneratorTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-rss/src/test/java/org/apache/archiva/rss/RssFeedGeneratorTest.java (working copy) @@ -26,17 +26,17 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.util.ArrayList; import java.util.Date; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * @version */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = {"classpath*:/META-INF/spring-context.xml"} ) public class RssFeedGeneratorTest extends TestCase Index: archiva-modules/archiva-web/archiva-security/pom.xml =================================================================== --- archiva-modules/archiva-web/archiva-security/pom.xml (revision 1355676) +++ archiva-modules/archiva-web/archiva-security/pom.xml (working copy) @@ -147,6 +147,12 @@ test + org.apache.archiva + archiva-testutil + ${project.version} + test + + org.apache.derby derby test Index: archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/archiva/security/AbstractSecurityTest.java =================================================================== --- archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/archiva/security/AbstractSecurityTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/archiva/security/AbstractSecurityTest.java (working copy) @@ -38,18 +38,18 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.inject.Named; import java.io.File; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * AbstractSecurityTest * * @version $Id: AbstractSecurityTest */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public abstract class AbstractSecurityTest extends TestCase Index: archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/archiva/security/RoleManagerTest.java =================================================================== --- archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/archiva/security/RoleManagerTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-security/src/test/java/org/apache/archiva/security/RoleManagerTest.java (working copy) @@ -24,16 +24,16 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * RoleProfilesTest * * @version $Id: RoleManagerTest.java 4330 2007-05-10 17:28:56Z jmcconnell $ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class RoleManagerTest extends TestCase Index: archiva-modules/archiva-web/archiva-web-common/pom.xml =================================================================== --- archiva-modules/archiva-web/archiva-web-common/pom.xml (revision 1355676) +++ archiva-modules/archiva-web/archiva-web-common/pom.xml (working copy) @@ -140,6 +140,12 @@ test + org.apache.archiva + archiva-testutil + ${project.version} + test + + org.apache.maven.wagon wagon-http test Index: archiva-modules/archiva-web/archiva-web-common/src/test/java/org/apache/archiva/web/rss/RssFeedServletTest.java =================================================================== --- archiva-modules/archiva-web/archiva-web-common/src/test/java/org/apache/archiva/web/rss/RssFeedServletTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-web-common/src/test/java/org/apache/archiva/web/rss/RssFeedServletTest.java (working copy) @@ -35,12 +35,12 @@ import org.junit.Ignore; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.BlockJUnit4ClassRunner; import javax.servlet.http.HttpServletResponse; import java.io.File; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; -@RunWith( BlockJUnit4ClassRunner.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RssFeedServletTest extends TestCase { Index: archiva-modules/archiva-web/archiva-webapp-js/pom.xml =================================================================== --- archiva-modules/archiva-web/archiva-webapp-js/pom.xml (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp-js/pom.xml (working copy) @@ -650,7 +650,12 @@ mail test - + + org.apache.archiva + archiva-testutil + ${project.version} + test + Index: archiva-modules/archiva-web/archiva-webapp-js/src/test/java/org/apache/archiva/webapp/ui/services/api/RuntimeInfoServiceTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp-js/src/test/java/org/apache/archiva/webapp/ui/services/api/RuntimeInfoServiceTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp-js/src/test/java/org/apache/archiva/webapp/ui/services/api/RuntimeInfoServiceTest.java (working copy) @@ -29,10 +29,13 @@ import java.io.File; import java.util.Collections; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.runner.RunWith; /** * @author Olivier Lamy */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RuntimeInfoServiceTest extends AbstractRestServicesTest { Index: archiva-modules/archiva-web/archiva-webapp/pom.xml =================================================================== --- archiva-modules/archiva-web/archiva-webapp/pom.xml (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/pom.xml (working copy) @@ -875,7 +875,13 @@ test ${project.version} - + + + org.apache.archiva + archiva-testutil + ${project.version} + test + Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/AbstractDownloadTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/AbstractDownloadTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/AbstractDownloadTest.java (working copy) @@ -49,10 +49,13 @@ import org.springframework.web.context.ContextLoaderListener; import java.util.Collections; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.runner.RunWith; /** * @author Olivier Lamy */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public abstract class AbstractDownloadTest extends TestCase { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/DownloadArtifactsTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/DownloadArtifactsTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/DownloadArtifactsTest.java (working copy) @@ -34,7 +34,6 @@ import org.junit.BeforeClass; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -51,11 +50,12 @@ import java.util.List; import java.util.zip.ZipEntry; import java.util.zip.ZipFile; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; /** * @author Olivier Lamy */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class DownloadArtifactsTest extends AbstractDownloadTest { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/DownloadMergedIndexTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/DownloadMergedIndexTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/DownloadMergedIndexTest.java (working copy) @@ -37,16 +37,16 @@ import org.junit.BeforeClass; import org.junit.Test; import org.junit.runner.RunWith; -import org.junit.runners.JUnit4; import java.io.File; import java.util.Arrays; import java.util.List; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; /** * @author Olivier Lamy */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class DownloadMergedIndexTest extends AbstractDownloadTest { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/AbstractActionTestCase.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/AbstractActionTestCase.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/AbstractActionTestCase.java (working copy) @@ -43,7 +43,11 @@ import java.util.HashMap; import java.util.List; import java.util.Map; - +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.After; +import org.junit.Before; +import org.junit.runner.RunWith; +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public abstract class AbstractActionTestCase extends StrutsSpringTestCase { @@ -207,7 +211,8 @@ } @Override - protected void setUp( ) + @Before + public void setUp( ) throws Exception { super.setUp( ); @@ -223,7 +228,8 @@ } @Override - protected void tearDown( ) + @After + public void tearDown( ) throws Exception { super.tearDown( ); Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/AbstractWebworkTestCase.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/AbstractWebworkTestCase.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/AbstractWebworkTestCase.java (working copy) @@ -26,12 +26,14 @@ import java.lang.reflect.Method; import java.util.Collection; import java.util.List; - +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.runner.RunWith; /** * AbstractWebworkTestCase * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public abstract class AbstractWebworkTestCase extends StrutsSpringTestCase { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/AbstractOrganizationInfoActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/AbstractOrganizationInfoActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/AbstractOrganizationInfoActionTest.java (working copy) @@ -24,6 +24,7 @@ import org.apache.archiva.configuration.Configuration; import org.apache.archiva.web.action.AbstractWebworkTestCase; import org.easymock.MockControl; +import org.junit.Before; /** */ @@ -40,8 +41,9 @@ protected abstract AbstractAppearanceAction getAction(); + @Before @Override - protected void setUp() + public void setUp() throws Exception { super.setUp(); Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/EditOrganizationInfoActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/EditOrganizationInfoActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/EditOrganizationInfoActionTest.java (working copy) @@ -28,6 +28,8 @@ import java.util.HashMap; import java.util.List; import java.util.Map; +import org.junit.Before; +import org.junit.Test; /** */ @@ -54,6 +56,7 @@ private ActionValidatorManager actionValidatorManager; @Override + @Before public void setUp() throws Exception { @@ -64,6 +67,7 @@ actionValidatorManager = factory.createDefaultActionValidatorManager(); } + @Test public void testOrganisationInfoSaves() throws Exception { @@ -96,6 +100,7 @@ assertEquals( "URL1", orginfo.getUrl() ); } + @Test public void testStruts2ValidationFrameworkWithNullInputs() throws Exception { @@ -122,6 +127,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithBlankInputs() throws Exception { @@ -148,6 +154,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithInvalidInputs() throws Exception { @@ -184,6 +191,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithValidInputs() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/OrganizationInfoActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/OrganizationInfoActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/appearance/OrganizationInfoActionTest.java (working copy) @@ -20,12 +20,14 @@ */ import org.apache.archiva.configuration.OrganisationInformation; +import org.junit.Test; /** */ public class OrganizationInfoActionTest extends AbstractOrganizationInfoActionTest { + @Test public void testOrganisationInfoLoads() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/AddProxyConnectorActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/AddProxyConnectorActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/AddProxyConnectorActionTest.java (working copy) @@ -42,6 +42,8 @@ import java.util.List; import java.util.Map; +import org.junit.Before; +import org.junit.Test; /** * AddProxyConnectorActionTest @@ -58,7 +60,8 @@ private ArchivaConfiguration archivaConfiguration; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -75,6 +78,7 @@ archivaConfiguration ); } + @Test public void testAddBlackListPattern() throws Exception { @@ -106,6 +110,7 @@ assertEquals( 1, connector.getBlackListPatterns().size() ); } + @Test public void testAddProperty() throws Exception { @@ -139,6 +144,7 @@ } @SuppressWarnings( "unchecked" ) + @Test public void testAddProxyConnectorCommit() throws Exception { @@ -173,6 +179,7 @@ assertEquals( "gramov-a-bits", actualConnector.getProperties().get( "eat-a" ) ); } + @Test public void testAddProxyConnectorInitialPage() throws Exception { @@ -194,6 +201,7 @@ assertEquals( Action.INPUT, status ); } + @Test public void testAddWhiteListPattern() throws Exception { @@ -225,6 +233,7 @@ assertEquals( 1, connector.getWhiteListPatterns().size() ); } + @Test public void testRemoveBlackListPattern() throws Exception { @@ -271,6 +280,7 @@ assertEquals( "Should have left 1 blacklist pattern", "**/*.war", connector.getBlackListPatterns().get( 0 ) ); } + @Test public void testRemoveProperty() throws Exception { @@ -317,6 +327,7 @@ assertEquals( "Should have left 1 property", "general-tso", connector.getProperties().get( "username" ) ); } + @Test public void testRemoveWhiteListPattern() throws Exception { @@ -363,6 +374,7 @@ assertEquals( "Should have left 1 whitelist pattern", "javax/**/*", connector.getWhiteListPatterns().get( 0 ) ); } + @Test public void testSecureActionBundle() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/DeleteProxyConnectorActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/DeleteProxyConnectorActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/DeleteProxyConnectorActionTest.java (working copy) @@ -32,7 +32,8 @@ import org.apache.archiva.web.action.AbstractWebworkTestCase; import org.apache.archiva.redback.integration.interceptor.SecureActionBundle; import org.easymock.MockControl; - +import org.junit.Before; +import org.junit.Test; /** * DeleteProxyConnectorActionTest * @@ -52,7 +53,8 @@ private ArchivaConfiguration archivaConfiguration; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -68,7 +70,8 @@ ( (DefaultProxyConnectorAdmin) action.getProxyConnectorAdmin() ).setArchivaConfiguration( archivaConfiguration ); } - + + @Test public void testConfirmDelete() throws Exception { @@ -84,6 +87,7 @@ assertNoErrors( action ); } + @Test public void testConfirmDeleteBadSourceOrTarget() throws Exception { @@ -118,6 +122,7 @@ assertHasErrors( action ); } + @Test public void testConfirmDeleteNoSourceOrTarget() throws Exception { @@ -152,6 +157,7 @@ assertHasErrors( action ); } + @Test public void testDelete() throws Exception { @@ -177,6 +183,7 @@ assertEquals( 0, archivaConfiguration.getConfiguration().getProxyConnectors().size() ); } + @Test public void testSecureActionBundle() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/DisableProxyConnectorActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/DisableProxyConnectorActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/DisableProxyConnectorActionTest.java (working copy) @@ -31,6 +31,8 @@ import org.apache.archiva.redback.components.registry.RegistryException; import org.apache.archiva.redback.integration.interceptor.SecureActionBundle; import org.easymock.MockControl; +import org.junit.Before; +import org.junit.Test; public class DisableProxyConnectorActionTest extends AbstractWebworkTestCase @@ -46,7 +48,8 @@ private ArchivaConfiguration archivaConfiguration; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -62,6 +65,7 @@ ( (DefaultProxyConnectorAdmin) action.getProxyConnectorAdmin() ).setArchivaConfiguration( archivaConfiguration ); } + @Test public void testConfirmDisableBadSourceOrTarget() throws Exception { @@ -96,6 +100,7 @@ assertHasErrors( action ); } + @Test public void testConfirmDisableNoSourceOrTarget() throws Exception { @@ -130,6 +135,7 @@ assertHasErrors( action ); } + @Test public void testDelete() throws Exception { @@ -158,6 +164,7 @@ assertTrue( config.isDisabled() ); } + @Test public void testSecureActionBundle() throws Exception { @@ -169,6 +176,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testConfirmEnable() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/EditProxyConnectorActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/EditProxyConnectorActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/EditProxyConnectorActionTest.java (working copy) @@ -43,6 +43,8 @@ import java.util.List; import java.util.Map; +import org.junit.Before; +import org.junit.Test; /** * EditProxyConnectorActionTest @@ -63,7 +65,8 @@ private ArchivaConfiguration archivaConfiguration; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -112,6 +115,7 @@ archivaConfiguration ); } + @Test public void testAddBlackListPattern() throws Exception { @@ -145,6 +149,7 @@ assertEquals( 1, connector.getBlackListPatterns().size() ); } + @Test public void testAddProperty() throws Exception { @@ -180,6 +185,7 @@ assertEquals( "gramov-a-bits", connector.getProperties().get( "eat-a" ) ); } + @Test public void testAddWhiteListPattern() throws Exception { @@ -214,6 +220,7 @@ } @SuppressWarnings( "unchecked" ) + @Test public void testEditProxyConnectorCommit() throws Exception { @@ -250,6 +257,7 @@ } + @Test public void testEditProxyConnectorInitialPage() throws Exception { @@ -266,6 +274,7 @@ assertEquals( Action.INPUT, status ); } + @Test public void testRemoveBlackListPattern() throws Exception { @@ -314,6 +323,7 @@ assertEquals( "Should have left 1 blacklist pattern", "**/*.war", connector.getBlackListPatterns().get( 0 ) ); } + @Test public void testRemoveProperty() throws Exception { @@ -362,6 +372,7 @@ assertEquals( "Should have left 1 property", "general-tso", connector.getProperties().get( "username" ) ); } + @Test public void testRemoveWhiteListPattern() throws Exception { @@ -410,6 +421,7 @@ assertEquals( "Should have left 1 whitelist pattern", "javax/**/*", connector.getWhiteListPatterns().get( 0 ) ); } + @Test public void testSecureActionBundle() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/EnableProxyConnectorActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/EnableProxyConnectorActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/EnableProxyConnectorActionTest.java (working copy) @@ -30,6 +30,8 @@ import org.apache.archiva.web.action.AbstractWebworkTestCase; import org.apache.archiva.redback.integration.interceptor.SecureActionBundle; import org.easymock.MockControl; +import org.junit.Before; +import org.junit.Test; public class EnableProxyConnectorActionTest extends AbstractWebworkTestCase @@ -45,7 +47,8 @@ private ArchivaConfiguration archivaConfiguration; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -69,6 +72,7 @@ archivaConfiguration.save( config ); } + @Test public void testConfirmDeleteBadSourceOrTarget() throws Exception { @@ -103,6 +107,7 @@ assertHasErrors( action ); } + @Test public void testConfirmEnableNoSourceOrTarget() throws Exception { @@ -137,6 +142,7 @@ assertHasErrors( action ); } + @Test public void testEnable() throws Exception { @@ -165,6 +171,7 @@ assertFalse( config.isDisabled() ); } + @Test public void testSecureActionBundle() throws Exception { @@ -176,6 +183,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testConfirmEnable() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/ProxyConnectorsActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/ProxyConnectorsActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/ProxyConnectorsActionTest.java (working copy) @@ -33,6 +33,8 @@ import org.apache.archiva.redback.components.registry.RegistryException; import org.apache.archiva.redback.integration.interceptor.SecureActionBundle; import org.easymock.MockControl; +import org.junit.Before; +import org.junit.Test; /** * ProxyConnectorsActionTest @@ -55,7 +57,8 @@ private ArchivaConfiguration archivaConfiguration; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -72,6 +75,7 @@ archivaConfiguration ); } + @Test public void testSecureActionBundle() throws Exception { @@ -84,6 +88,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testExecute() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/SortProxyConnectorsActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/SortProxyConnectorsActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/connectors/proxy/SortProxyConnectorsActionTest.java (working copy) @@ -38,6 +38,8 @@ import java.util.Collections; import java.util.List; +import org.junit.Before; +import org.junit.Test; /** * SortProxyConnectorsActionTest @@ -62,7 +64,8 @@ private ArchivaConfiguration archivaConfiguration; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -90,6 +93,7 @@ archivaConfiguration.save( config ); } + @Test public void testSecureActionBundle() throws Exception { @@ -101,6 +105,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testSortDown() throws Exception { @@ -115,6 +120,7 @@ assertOrder( new String[]{ JAVAX, CENTRAL, CODEHAUS } ); } + @Test public void testSortDownPastEnd() throws Exception { @@ -131,6 +137,7 @@ assertOrder( new String[]{ CENTRAL, JAVAX, CODEHAUS } ); } + @Test public void testSortUp() throws Exception { @@ -145,6 +152,7 @@ assertOrder( new String[]{ CENTRAL, CODEHAUS, JAVAX } ); } + @Test public void testSortUpPastBeginning() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/legacy/AddLegacyArtifactPathActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/legacy/AddLegacyArtifactPathActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/legacy/AddLegacyArtifactPathActionTest.java (working copy) @@ -29,7 +29,11 @@ import java.util.HashMap; import java.util.List; import java.util.Map; - +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class AddLegacyArtifactPathActionTest extends TestCase { @@ -67,6 +71,7 @@ private ActionValidatorManager actionValidatorManager; @Override + @Before public void setUp() throws Exception { @@ -77,6 +82,7 @@ actionValidatorManager = factory.createDefaultActionValidatorManager(); } + @Test public void testStruts2ValidationFrameworkWithNullInputs() throws Exception { @@ -120,6 +126,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithBlankInputs() throws Exception { @@ -163,6 +170,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithInvalidInputs() throws Exception { @@ -218,6 +226,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithValidInputs() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/networkproxies/ConfigureNetworkProxyActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/networkproxies/ConfigureNetworkProxyActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/networkproxies/ConfigureNetworkProxyActionTest.java (working copy) @@ -29,7 +29,11 @@ import java.util.HashMap; import java.util.List; import java.util.Map; - +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class ConfigureNetworkProxyActionTest extends TestCase { private static final String EMPTY_STRING = ""; @@ -64,6 +68,7 @@ private ActionValidatorManager actionValidatorManager; @Override + @Before public void setUp() throws Exception { @@ -74,6 +79,7 @@ actionValidatorManager = factory.createDefaultActionValidatorManager(); } + @Test public void testStruts2ValidationFrameworkWithNullInputs() throws Exception { // prep @@ -107,6 +113,7 @@ ValidatorUtil.assertFieldErrors(expectedFieldErrors, fieldErrors); } + @Test public void testStruts2ValidationFrameworkWithBlankInputs() throws Exception { // prep @@ -140,6 +147,7 @@ ValidatorUtil.assertFieldErrors(expectedFieldErrors, fieldErrors); } + @Test public void testStruts2ValidationFrameworkWithInvalidInputs() throws Exception { // prep @@ -183,6 +191,7 @@ ValidatorUtil.assertFieldErrors(expectedFieldErrors, fieldErrors); } + @Test public void testStruts2ValidationFrameworkWithValidInputs() throws Exception { // prep Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AbstractManagedRepositoryActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AbstractManagedRepositoryActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AbstractManagedRepositoryActionTest.java (working copy) @@ -25,7 +25,11 @@ import org.apache.struts2.StrutsSpringTestCase; import java.io.File; - +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public abstract class AbstractManagedRepositoryActionTest extends StrutsSpringTestCase { @@ -71,7 +75,8 @@ } @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddManagedRepositoryActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddManagedRepositoryActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddManagedRepositoryActionTest.java (working copy) @@ -42,6 +42,8 @@ import java.util.HashMap; import java.util.List; import java.util.Map; +import org.junit.Before; +import org.junit.Test; /** * AddManagedRepositoryActionTest @@ -71,7 +73,8 @@ @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -111,6 +114,7 @@ } + @Test public void testSecureActionBundle() throws SecureActionException { @@ -124,6 +128,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testAddRepositoryInitialPage() throws Exception { @@ -153,6 +158,7 @@ assertFalse( configuration.isSnapshots() ); } + @Test public void testAddRepository() throws Exception { @@ -216,6 +222,7 @@ } + @Test public void testAddRepositoryExistingLocation() throws Exception { @@ -242,6 +249,7 @@ registryControl.verify(); } + @Test public void testStruts2ValidationFrameworkWithNullInputs() throws Exception { @@ -277,6 +285,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithBlankInputs() throws Exception { @@ -313,6 +322,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithInvalidInputs() throws Exception { @@ -366,6 +376,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithValidInputs() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddRemoteRepositoryActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddRemoteRepositoryActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddRemoteRepositoryActionTest.java (working copy) @@ -29,6 +29,8 @@ import org.easymock.MockControl; import java.util.Collections; +import org.junit.Before; +import org.junit.Test; /** * AddRemoteRepositoryActionTest @@ -46,7 +48,9 @@ private static final String REPO_ID = "remote-repo-ident"; - protected void setUp() + @Override + @Before + public void setUp() throws Exception { super.setUp(); @@ -60,6 +64,7 @@ archivaConfiguration ); } + @Test public void testSecureActionBundle() throws Exception { @@ -73,6 +78,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testAddRemoteRepositoryInitialPage() throws Exception { @@ -89,6 +95,7 @@ assertEquals( Action.INPUT, status ); } + @Test public void testAddRemoteRepository() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteManagedRepositoryActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteManagedRepositoryActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteManagedRepositoryActionTest.java (working copy) @@ -53,6 +53,8 @@ import java.util.Arrays; import java.util.Collections; import java.util.List; +import org.junit.Before; +import org.junit.Test; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; @@ -89,7 +91,9 @@ private MockControl metadataRepositoryControl; - protected void setUp() + @Override + @Before + public void setUp() throws Exception { super.setUp(); @@ -129,6 +133,7 @@ metadataRepositoryControl.replay(); } + @Test public void testSecureActionBundle() throws SecureActionException, RepositoryAdminException { @@ -142,6 +147,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testDeleteRepositoryAndReposUnderRepoGroup() throws Exception { @@ -172,6 +178,7 @@ metadataRepositoryControl.verify(); } + @Test public void testDeleteRepositoryConfirmation() throws Exception { @@ -207,6 +214,7 @@ action.getManagedRepositoryAdmin().getManagedRepositories() ); } + @Test public void testDeleteRepositoryKeepContent() throws Exception { @@ -248,6 +256,7 @@ return control; } + @Test public void testDeleteRepositoryDeleteContent() throws Exception { @@ -275,6 +284,7 @@ metadataRepositoryControl.verify(); } + @Test public void testDeleteRepositoryAndAssociatedProxyConnectors() throws Exception { @@ -306,6 +316,7 @@ metadataRepositoryControl.verify(); } + @Test public void testDeleteRepositoryCancelled() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteRemoteRepositoryActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteRemoteRepositoryActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteRemoteRepositoryActionTest.java (working copy) @@ -33,6 +33,8 @@ import org.easymock.MockControl; import java.util.Collections; +import org.junit.Before; +import org.junit.Test; /** * DeleteRemoteRepositoryActionTest @@ -50,7 +52,9 @@ private ArchivaConfiguration archivaConfiguration; - protected void setUp() + @Override + @Before + public void setUp() throws Exception { super.setUp(); @@ -65,6 +69,7 @@ archivaConfiguration ); } + @Test public void testDeleteRemoteRepositoryConfirmation() throws Exception { @@ -92,6 +97,7 @@ action.getRemoteRepositoryAdmin().getRemoteRepositories() ); } + @Test public void testDeleteRemoteRepository() throws org.apache.archiva.redback.components.registry.RegistryException, IndeterminateConfigurationException, RepositoryAdminException { @@ -123,6 +129,7 @@ assertEquals( 0, configuration.getProxyConnectors().size() ); } + @Test public void testDeleteRemoteRepositoryCancelled() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteRepositoryGroupActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteRepositoryGroupActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/DeleteRepositoryGroupActionTest.java (working copy) @@ -32,6 +32,8 @@ import org.easymock.MockControl; import java.util.Collections; +import org.junit.Before; +import org.junit.Test; /** * DeleteRepositoryGroupActionTest @@ -47,7 +49,9 @@ private ArchivaConfiguration archivaConfiguration; - protected void setUp() + @Override + @Before + public void setUp() throws Exception { super.setUp(); @@ -61,6 +65,7 @@ archivaConfiguration ); } + @Test public void testSecureActionBundle() throws SecureActionException, RepositoryAdminException { @@ -74,6 +79,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testDeleteRepositoryGroupConfirmation() throws Exception { @@ -94,6 +100,7 @@ assertEquals( Collections.singletonList( origRepoGroup ), configuration.getRepositoryGroups() ); } + @Test public void testDeleteRepositoryGroup() throws Exception { @@ -118,6 +125,7 @@ assertTrue( configuration.getRepositoryGroups().isEmpty() ); } + @Test public void testDeleteRepositoryGroupCancelled() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/EditManagedRepositoryActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/EditManagedRepositoryActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/EditManagedRepositoryActionTest.java (working copy) @@ -53,11 +53,17 @@ import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; + /** * EditManagedRepositoryActionTest * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class EditManagedRepositoryActionTest extends AbstractManagedRepositoryActionTest { @@ -82,7 +88,8 @@ private RepositoryArchivaTaskScheduler repositoryTaskScheduler; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -133,6 +140,7 @@ } + @Test public void testSecureActionBundle() throws SecureActionException, RepositoryAdminException { @@ -146,6 +154,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testEditRepositoryInitialPage() throws Exception { @@ -176,6 +185,7 @@ assertRepositoryEquals( repository, createRepository() ); } + @Test public void testEditRepository() throws Exception { @@ -285,6 +295,7 @@ registryControl.verify(); } + @Test public void testEditRepositoryLocationChanged() throws Exception { @@ -387,6 +398,7 @@ registryControl.verify(); } + @Test public void testStruts2ValidationFrameworkWithNullInputs() throws Exception { @@ -422,6 +434,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithBlankInputs() throws Exception { @@ -458,6 +471,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithInvalidInputs() throws Exception { @@ -511,6 +525,7 @@ ValidatorUtil.assertFieldErrors( expectedFieldErrors, fieldErrors ); } + @Test public void testStruts2ValidationFrameworkWithValidInputs() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/EditRemoteRepositoryActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/EditRemoteRepositoryActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/EditRemoteRepositoryActionTest.java (working copy) @@ -33,11 +33,16 @@ import java.util.Collections; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; /** * EditRemoteRepositoryActionTest * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class EditRemoteRepositoryActionTest extends StrutsSpringTestCase { @@ -55,7 +60,9 @@ return new String[]{ "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" }; } - protected void setUp() + @Before + @Override + public void setUp() throws Exception { super.setUp(); @@ -69,6 +76,7 @@ archivaConfiguration ); } + @Test public void testEditRemoteRepository() throws Exception { @@ -98,6 +106,7 @@ archivaConfigurationControl.verify(); } + @Test public void testEditRemoteRepositoryInitialPage() throws Exception { @@ -121,6 +130,7 @@ assertRepositoryEquals( repository, createRepository() ); } + @Test public void testSecureActionBundle() throws SecureActionException, RepositoryAdminException { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/RepositoriesActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/RepositoriesActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/RepositoriesActionTest.java (working copy) @@ -41,10 +41,15 @@ import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; - +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.After; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; /** * Test the repositories action returns the correct data. */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RepositoriesActionTest extends StrutsSpringTestCase { @@ -54,7 +59,9 @@ ArchivaConfiguration originalArchivaConfiguration; - protected void setUp() + @Override + @Before + public void setUp() throws Exception { @@ -73,7 +80,8 @@ @Override - protected void tearDown() + @After + public void tearDown() throws Exception { super.tearDown(); @@ -91,6 +99,7 @@ return new String[]{ "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" }; } + @Test public void testGetRepositories() throws Exception { @@ -145,6 +154,7 @@ } } + @Test public void testSecureActionBundle() throws SecureActionException { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/RepositoryGroupsActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/RepositoryGroupsActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/RepositoryGroupsActionTest.java (working copy) @@ -37,6 +37,7 @@ import java.util.Collections; import java.util.List; +import org.junit.Test; /** * RepositoryGroupsActionTest @@ -56,7 +57,8 @@ private ArchivaConfiguration archivaConfiguration; - protected void setUp() + @Override + public void setUp() throws Exception { super.setUp(); @@ -72,6 +74,7 @@ archivaConfiguration ); } + @Test public void testSecureActionBundle() throws SecureActionException, RepositoryAdminException { @@ -85,6 +88,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testAddRepositoryGroup() throws Exception { @@ -108,6 +112,7 @@ archivaConfigurationControl.verify(); } + @Test public void testAddEmptyRepositoryGroup() throws Exception { @@ -127,6 +132,7 @@ assertEquals( 0, configuration.getRepositoryGroups().size() ); } + @Test public void testAddDuplicateRepositoryGroup() throws Exception { @@ -156,6 +162,7 @@ action.getRepositoryGroupAdmin().getRepositoriesGroups() ); } + @Test public void testGetRepositoryGroups() throws Exception { @@ -191,6 +198,7 @@ assertEquals( REPO2_ID, repos.get( 0 ) ); } + @Test public void testAddRepositoryToGroup() throws Exception { @@ -240,6 +248,7 @@ assertNull( action.getGroupToRepositoryMap().get( repoGroup.getId() ) ); } + @Test public void testRemoveRepositoryFromGroup() throws Exception { @@ -291,6 +300,7 @@ assertEquals( REPO2_ID, repos.get( 1 ) ); } + @Test public void testAddDuplicateRepositoryToGroup() throws Exception { @@ -327,6 +337,7 @@ assertEquals( Action.ERROR, result ); } + @Test public void testRemoveRepositoryNotInGroup() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/SortRepositoriesActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/SortRepositoriesActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/SortRepositoriesActionTest.java (working copy) @@ -28,9 +28,15 @@ import org.apache.archiva.redback.integration.interceptor.SecureActionException; import org.easymock.MockControl; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.After; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; /** * SortRepositoriesActionTest */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class SortRepositoriesActionTest extends StrutsSpringTestCase { @@ -56,7 +62,9 @@ return new String[]{ "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" }; } - protected void setUp() + @Before + @Override + public void setUp() throws Exception { super.setUp(); @@ -68,13 +76,15 @@ } @Override - protected void tearDown() + @After + public void tearDown() throws Exception { super.tearDown(); action.archivaConfiguration = originalArchivaConfiguration; } + @Test public void testSecureActionBundle() throws SecureActionException { @@ -87,6 +97,7 @@ assertEquals( 1, bundle.getAuthorizationTuples().size() ); } + @Test public void testSortDownFirstRepository() throws Exception { @@ -121,6 +132,7 @@ assertEquals( REPO3_ID, repositories.get( 2 ) ); } + @Test public void testSortDownLastRepository() throws Exception { @@ -155,6 +167,7 @@ assertEquals( REPO3_ID, repositories.get( 2 ) ); } + @Test public void testSortUpLastRepository() throws Exception { @@ -189,6 +202,7 @@ assertEquals( REPO2_ID, repositories.get( 2 ) ); } + @Test public void testSortUpFirstRepository() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/scanning/RepositoryScanningActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/scanning/RepositoryScanningActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/scanning/RepositoryScanningActionTest.java (working copy) @@ -9,6 +9,8 @@ import java.util.ArrayList; import java.util.List; +import org.junit.Before; +import org.junit.Test; /* * Licensed to the Apache Software Foundation (ASF) under one @@ -40,7 +42,9 @@ private Configuration config; - protected void setUp() + @Override + @Before + public void setUp() throws Exception { @@ -65,6 +69,7 @@ } + @Test public void testUpdateKnownConsumers() throws Exception { @@ -90,6 +95,7 @@ assertEquals( "results " + results, 8, results.size() ); } + @Test public void testDisableAllKnownConsumers() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/BrowseActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/BrowseActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/BrowseActionTest.java (working copy) @@ -30,6 +30,7 @@ import java.util.Arrays; import java.util.Collections; import java.util.List; +import org.junit.Test; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; @@ -47,7 +48,8 @@ private static final String OTHER_TEST_REPO = "other-repo"; - protected void setUp() + @Override + public void setUp() throws Exception { super.setUp(); @@ -61,7 +63,8 @@ factory.setRepositorySession( repositorySession ); } - protected void tearDown() + @Override + public void tearDown() throws Exception { super.tearDown(); @@ -69,11 +72,13 @@ setObservableRepos( Lists.newArrayList( "test-repo" ) ); } + @Test public void testInstantiation() { assertFalse( action == getActionProxy( "/browse.action" ).getAction() ); } + @Test public void testBrowse() throws Exception { @@ -92,6 +97,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseNoObservableRepos() throws Exception { @@ -103,6 +109,7 @@ assertNoOutputVariables(); } + @Test public void testBrowseGroupNoObservableRepos() throws Exception { @@ -122,6 +129,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseArtifactNoObservableRepos() throws Exception { @@ -143,6 +151,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseGroupNoGroupId() throws Exception { @@ -151,6 +160,7 @@ assertNoOutputVariables(); } + @Test public void testBrowseGroupNoArtifacts() throws Exception { @@ -172,6 +182,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseGroupWithArtifacts() throws Exception { @@ -195,6 +206,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseWithCollapsedGroupsAndArtifacts() throws Exception { @@ -217,6 +229,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseWithCollapsedGroupsAndArtifactsAcrossRepositories() throws Exception { @@ -241,6 +254,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseGroupWithCollapsedGroupsAndArtifacts() throws Exception { @@ -266,6 +280,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseArtifactNoGroupId() throws Exception { @@ -284,6 +299,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseArtifactNoArtifactId() throws Exception { @@ -302,6 +318,7 @@ assertNull( action.getSharedModel() ); } + @Test public void testBrowseArtifact() throws Exception @@ -336,6 +353,7 @@ assertDefaultModel( model, selectedGroupId, selectedArtifactId, null ); } + @Test public void testBrowseArtifactWithSnapshots() throws Exception Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/DeleteArtifactActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/DeleteArtifactActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/DeleteArtifactActionTest.java (working copy) @@ -44,7 +44,12 @@ import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; - +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.After; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class DeleteArtifactActionTest extends StrutsSpringTestCase { @@ -79,7 +84,8 @@ } @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -111,7 +117,8 @@ } @Override - protected void tearDown() + @After + public void tearDown() throws Exception { action = null; @@ -120,6 +127,7 @@ } + @Test public void testNPEInDeleteArtifact() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/reports/GenerateReportActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/reports/GenerateReportActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/reports/GenerateReportActionTest.java (working copy) @@ -44,6 +44,7 @@ import java.util.HashMap; import java.util.List; import java.util.Map; +import org.junit.Test; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; @@ -76,7 +77,7 @@ @Override - protected void setUp() + public void setUp() throws Exception { super.setUp(); @@ -101,6 +102,8 @@ factory.setRepositorySession( repositorySession ); } + + @Override @After public void tearDown() throws Exception @@ -149,6 +152,7 @@ assertEquals( availableRepositories, action.getAvailableRepositories() ); } + @Test public void testGenerateStatisticsInvalidRowCount() throws Exception { @@ -162,6 +166,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsInvalidEndDate() throws Exception { @@ -176,6 +181,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsMalformedEndDate() throws Exception { @@ -191,6 +197,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsInvalidEndDateMultiRepo() throws Exception { @@ -205,6 +212,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsMalformedEndDateMultiRepo() throws Exception { @@ -220,6 +228,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsNoRepos() throws Exception { @@ -232,6 +241,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsSingleRepo() throws Exception { @@ -247,6 +257,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsSingleRepoNoStats() throws Exception @@ -264,6 +275,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsOvershotPages() throws Exception @@ -281,6 +293,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsMultipleRepoNoResults() throws Exception @@ -303,6 +316,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testGenerateStatisticsMultipleRepo() throws Exception @@ -322,6 +336,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsSingleRepo() throws Exception { @@ -344,6 +359,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsMultipleRepos() throws Exception { @@ -365,6 +381,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsMalformedEndDateMultiRepo() throws Exception { @@ -380,6 +397,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsMalformedEndDateSingleRepo() throws Exception { @@ -395,6 +413,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsInvalidEndDateMultiRepo() throws Exception { @@ -409,6 +428,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsInvalidEndDateSingleRepo() throws Exception { @@ -423,6 +443,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsSingleRepoNoStats() throws Exception @@ -439,6 +460,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsNoRepos() throws Exception { @@ -451,6 +473,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsMultipleRepoNoResults() throws Exception @@ -472,6 +495,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testDownloadStatisticsMultipleRepoInStrutsFormat() throws Exception { @@ -494,6 +518,7 @@ repositoryStatisticsManagerControl.verify(); } + @Test public void testHealthReportSingleRepo() throws Exception { @@ -524,6 +549,7 @@ metadataRepositoryControl.verify(); } + @Test public void testHealthReportInvalidRowCount() throws Exception { @@ -542,6 +568,7 @@ metadataRepositoryControl.verify(); } + @Test public void testHealthReportAllRepos() throws Exception { @@ -576,6 +603,7 @@ metadataRepositoryControl.verify(); } + @Test public void testHealthReportSingleRepoByCorrectGroupId() throws Exception { @@ -606,6 +634,7 @@ metadataRepositoryControl.verify(); } + @Test public void testHealthReportSingleRepoByCorrectGroupIdAllRepositories() throws Exception { @@ -641,6 +670,7 @@ metadataRepositoryControl.verify(); } + @Test public void testHealthReportSingleRepoByIncorrectGroupId() throws Exception { @@ -681,6 +711,7 @@ return problem; } + @Test public void testHealthReportNoRepositoryId() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/SearchActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/SearchActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/SearchActionTest.java (working copy) @@ -39,6 +39,7 @@ import java.util.Arrays; import java.util.Collections; import java.util.List; +import org.junit.Test; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; @@ -72,7 +73,7 @@ private RepositorySession session; @Override - protected void setUp() + public void setUp() throws Exception { super.setUp(); @@ -107,6 +108,7 @@ // quick search... + @Test public void testQuickSearch() throws Exception { @@ -153,6 +155,7 @@ searchControl.verify(); } + @Test public void testSearchWithinSearchResults() throws Exception { @@ -203,6 +206,7 @@ searchControl.verify(); } + @Test public void testQuickSearchUserHasNoAccessToAnyRepository() throws Exception { @@ -223,6 +227,7 @@ userReposControl.verify(); } + @Test public void testQuickSearchNoSearchHits() throws Exception { @@ -258,6 +263,7 @@ // advanced/filtered search... + @Test public void testAdvancedSearchOneRepository() throws Exception { @@ -305,6 +311,7 @@ searchControl.verify(); } + @Test public void testAdvancedSearchAllRepositories() throws Exception { @@ -356,6 +363,7 @@ userReposControl.verify(); } + @Test public void testAdvancedSearchNoSearchHits() throws Exception { @@ -393,6 +401,7 @@ searchControl.verify(); } + @Test public void testAdvancedSearchUserHasNoAccessToAnyRepository() throws Exception { @@ -406,6 +415,7 @@ assertEquals(GlobalResults.ACCESS_TO_NO_REPOS, result); } + @Test public void testAdvancedSearchNoSpecifiedCriteria() throws Exception { @@ -422,6 +432,7 @@ } // find artifact.. + @Test public void testFindArtifactWithOneHit() throws Exception { @@ -450,6 +461,7 @@ userReposControl.verify(); } + @Test public void testFindArtifactWithMultipleHits() throws Exception { @@ -478,6 +490,7 @@ userReposControl.verify(); } + @Test public void testFindArtifactNoChecksumSpecified() throws Exception { @@ -488,6 +501,7 @@ assertEquals("Unable to search for a blank checksum", (String) action.getActionErrors().iterator().next()); } + @Test public void testFindArtifactNoResults() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/ShowArtifactActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/ShowArtifactActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/ShowArtifactActionTest.java (working copy) @@ -47,6 +47,7 @@ import java.util.Collections; import java.util.List; import java.util.Map; +import org.junit.Test; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; @@ -77,7 +78,8 @@ private static final String TEST_TYPE = "jar"; - protected void setUp() + @Override + public void setUp() throws Exception { super.setUp(); @@ -115,11 +117,13 @@ } + @Test public void testInstantiation() { assertFalse( action == getActionProxy( "/showArtifact.action" ).getAction() ); } + @Test public void testGetArtifactUniqueRelease() { metadataResolver.setProjectVersion( TEST_REPO, TEST_GROUP_ID, TEST_ARTIFACT_ID, @@ -143,6 +147,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testGetArtifactUniqueSnapshot() { metadataResolver.setProjectVersion( TEST_REPO, TEST_GROUP_ID, TEST_ARTIFACT_ID, @@ -173,6 +178,7 @@ assertNull( action.getMailingLists() ); } + @Test public void testGetArtifactUniqueSnapshotTimestamped() { metadataResolver.setProjectVersion( TEST_REPO, TEST_GROUP_ID, TEST_ARTIFACT_ID, @@ -189,6 +195,7 @@ assertNoOutputFields(); } + @Test public void testGetMissingProject() { setActionParameters(); @@ -200,6 +207,7 @@ assertNoOutputFields(); } + @Test public void testGetArtifactNoObservableRepos() { setObservableRepos( Collections.emptyList() ); @@ -214,6 +222,7 @@ assertNoOutputFields(); } + @Test public void testGetArtifactNotInObservableRepos() { metadataResolver.setProjectVersion( OTHER_TEST_REPO, TEST_GROUP_ID, TEST_ARTIFACT_ID, @@ -228,6 +237,7 @@ assertNoOutputFields(); } + @Test public void testGetArtifactOnlySeenInSecondObservableRepo() { setObservableRepos( Arrays.asList( OTHER_TEST_REPO, TEST_REPO ) ); @@ -252,6 +262,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testGetArtifactSeenInBothObservableRepo() { setObservableRepos( Arrays.asList( TEST_REPO, OTHER_TEST_REPO ) ); @@ -278,6 +289,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testGetArtifactCanOnlyObserveInOneOfTwoRepos() { setObservableRepos( Arrays.asList( TEST_REPO ) ); @@ -304,6 +316,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testGetArtifactNoMavenFacet() { ProjectVersionMetadata versionMetadata = new ProjectVersionMetadata(); @@ -335,6 +348,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testMetadataHasRepositoryFacetProblem() { String errMsg = "Error in resolving artifact's parent POM file: Sample Parent POM not found"; @@ -360,6 +374,7 @@ assertEquals( "Artifact metadata is incomplete: " + errMsg, action.getActionErrors().toArray()[0].toString() ); } + @Test public void testMetadataIncomplete() { ProjectVersionMetadata metaData = createProjectModel( TEST_SNAPSHOT_VERSION ); @@ -385,6 +400,7 @@ assertEquals( "Artifact metadata is incomplete.", action.getActionErrors().toArray()[0].toString() ); } + @Test public void testGetMailingLists() { ProjectVersionMetadata versionMetadata = createProjectModel( TEST_VERSION ); @@ -413,6 +429,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testGetDependencies() { ProjectVersionMetadata versionMetadata = createProjectModel( TEST_VERSION ); @@ -441,6 +458,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testGetDependees() throws Exception { @@ -471,6 +489,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testGetProjectMetadata() { ProjectVersionMetadata versionMetadata = createProjectModel( TEST_VERSION ); @@ -498,6 +517,7 @@ assertTrue( action.getArtifacts().isEmpty() ); } + @Test public void testAddAndDeleteMetadataProperty() { ProjectVersionMetadata versionMetadata = createProjectModel( TEST_VERSION ); Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/UploadActionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/UploadActionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/UploadActionTest.java (working copy) @@ -52,6 +52,7 @@ import java.util.Date; import java.util.List; import java.util.TimeZone; +import org.junit.Test; /** * UploadActionTest @@ -78,6 +79,7 @@ private ManagedRepository managedRepository; + @Override public void setUp() throws Exception { @@ -116,6 +118,7 @@ } + @Override public void tearDown() throws Exception { @@ -286,6 +289,7 @@ return control; } + @Test public void testArtifactUploadWithPomSuccessful() throws Exception { @@ -327,6 +331,7 @@ verifyProjectMetadataChecksums( repoLocation ); } + @Test public void testArtifactUploadWithClassifier() throws Exception { @@ -387,6 +392,7 @@ verifyProjectMetadataChecksums( repoLocation ); } + @Test public void testArtifactUploadGeneratePomSuccessful() throws Exception { @@ -428,6 +434,7 @@ verifyProjectMetadataChecksums( repoLocation ); } + @Test public void testArtifactUploadNoPomSuccessful() throws Exception { @@ -511,6 +518,7 @@ assertTrue( StringUtils.contains( contents, md5 ) ); } + @Test public void testArtifactUploadFailedRepositoryNotFound() throws Exception { @@ -548,6 +556,7 @@ new File( repoLocation, "/org/apache/archiva/artifact-upload/" + MetadataTools.MAVEN_METADATA ).exists() ); } + @Test public void testArtifactUploadSnapshots() throws Exception { @@ -680,6 +689,7 @@ assertEquals( "Incorrect build number in filename.", "2", buildnumber ); } + @Test public void testChecksumIsCorrectWhenArtifactIsReUploaded() throws Exception { @@ -748,6 +758,7 @@ verifyProjectMetadataChecksums( repoLocation ); } + @Test public void testUploadArtifactAlreadyExistingRedeploymentsBlocked() throws Exception { @@ -794,6 +805,7 @@ verifyProjectMetadataChecksums( repoLocation ); } + @Test public void testUploadArtifactAlreadyExistingRedeploymentsAllowed() throws Exception { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/startup/BannerTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/startup/BannerTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/startup/BannerTest.java (working copy) @@ -27,12 +27,15 @@ import org.apache.commons.io.IOUtils; import junit.framework.TestCase; - +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; /** * BannerTest * * @version $Id$ */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class BannerTest extends TestCase { @@ -43,14 +46,16 @@ assertEquals( "Encoding: ", encoded, Banner.encode( decoded ) ); assertEquals( "Decoding: ", decoded, Banner.decode( encoded ) ); } - + + @Test public void testEncodeDecode() { assertEncodeDecode( "[$10 ]", "[ ]" ); assertEncodeDecode( "$$$5_$n$5_", "$_____" + eol + "_____" ); assertEncodeDecode( "$${Refgjuvyr}", "${Erstwhile}" ); } - + + @Test public void testInjectVersion() { assertEquals( "[ 1.0 ]", Banner.injectVersion( "[#####]", "1.0" ) ); @@ -60,6 +65,7 @@ Banner.injectVersion( "Archiva:\"+eol+\" (##############)", "1.0-alpha-1" ) ); } + @Test public void testGetBanner() throws IOException { Index: archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/tags/DependencyTreeTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/tags/DependencyTreeTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/tags/DependencyTreeTest.java (working copy) @@ -42,15 +42,15 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import java.util.List; import javax.inject.Inject; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = {"classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml", "classpath:/spring-context-DependencyTreeTest.xml"} ) public class DependencyTreeTest Index: archiva-modules/archiva-web/archiva-webdav/pom.xml =================================================================== --- archiva-modules/archiva-web/archiva-webdav/pom.xml (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/pom.xml (working copy) @@ -185,6 +185,12 @@ org.apache.archiva + archiva-testutil + ${project.version} + test + + + org.apache.archiva archiva-repository-admin-default test Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/AbstractRepositoryServletTestCase.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/AbstractRepositoryServletTestCase.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/AbstractRepositoryServletTestCase.java (working copy) @@ -40,19 +40,19 @@ import org.slf4j.LoggerFactory; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.servlet.http.HttpServletResponse; import java.io.File; import java.io.IOException; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * AbstractRepositoryServletTestCase * * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/repository-servlet-simple.xml" } ) public abstract class AbstractRepositoryServletTestCase extends TestCase Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavResourceFactoryTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavResourceFactoryTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavResourceFactoryTest.java (working copy) @@ -50,18 +50,18 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; import java.io.IOException; import java.util.ArrayList; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * ArchivaDavResourceFactoryTest */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class ArchivaDavResourceFactoryTest extends TestCase Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavResourceLocatorTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavResourceLocatorTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavResourceLocatorTest.java (working copy) @@ -20,22 +20,29 @@ */ import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; /** */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class ArchivaDavResourceLocatorTest extends TestCase { ArchivaDavLocatorFactory factory; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); factory = new ArchivaDavLocatorFactory(); } + @Test public void testAvoidDoubleSlashInHref() throws Exception { @@ -53,6 +60,7 @@ assertEquals( "/repository/internal", locator.getRepositoryPath() ); } + @Test public void testLocatorWithPrefixHref() throws Exception { @@ -70,6 +78,7 @@ assertEquals( "/repository/internal", locator.getRepositoryPath() ); } + @Test public void testLocatorWithHrefThatContainsPrefix() throws Exception { @@ -87,6 +96,7 @@ assertEquals( "/repository/internal", locator.getRepositoryPath() ); } + @Test public void testLocatorWithRootHref() throws Exception { Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavSessionProviderTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavSessionProviderTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavSessionProviderTest.java (working copy) @@ -48,7 +48,12 @@ import java.util.Enumeration; import java.util.Locale; import java.util.Map; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class ArchivaDavSessionProviderTest extends TestCase { @@ -57,7 +62,8 @@ private WebdavRequest request; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -65,6 +71,7 @@ request = new WebdavRequestImpl( new HttpServletRequestMock(), null ); } + @Test public void testAttachSession() throws Exception { @@ -73,6 +80,7 @@ assertNotNull( request.getDavSession() ); } + @Test public void testReleaseSession() throws Exception { Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavSessionTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavSessionTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/ArchivaDavSessionTest.java (working copy) @@ -21,8 +21,14 @@ import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; + +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class ArchivaDavSessionTest extends TestCase { + @Test public void testTokens() { ArchivaDavSession session = new ArchivaDavSession(); @@ -36,6 +42,7 @@ assertEquals(0, session.getLockTokens().length); } + @Test public void testAddReferencesThrowsUnsupportedOperationException() { ArchivaDavSession session = new ArchivaDavSession(); @@ -50,6 +57,7 @@ } } + @Test public void testRemoveReferencesThrowsUnsupportedOperationException() { ArchivaDavSession session = new ArchivaDavSession(); Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/DavResourceTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/DavResourceTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/DavResourceTest.java (working copy) @@ -42,13 +42,13 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; import java.util.Collections; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class DavResourceTest extends TestCase Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/MimeTypesLoaderTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/MimeTypesLoaderTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/MimeTypesLoaderTest.java (working copy) @@ -24,16 +24,16 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * ArchivaMimeTypesTest * * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class MimeTypesLoaderTest extends TestCase Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/RepositoryServletSecurityTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/RepositoryServletSecurityTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/RepositoryServletSecurityTest.java (working copy) @@ -54,13 +54,13 @@ import org.junit.runner.RunWith; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.servlet.http.HttpServletResponse; import java.io.File; import java.io.IOException; import java.io.InputStream; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * RepositoryServletSecurityTest Test the flow of the authentication and authorization checks. This does not necessarily @@ -68,7 +68,7 @@ * * @version $Id$ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class RepositoryServletSecurityTest extends TestCase Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/util/MimeTypesTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/util/MimeTypesTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/util/MimeTypesTest.java (working copy) @@ -23,16 +23,16 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; /** * MimeTypesTest * * @version $Id: MimeTypesTest.java 6556 2007-06-20 20:44:46Z joakime $ */ -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class MimeTypesTest extends TestCase Index: archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/util/RepositoryPathUtilTest.java =================================================================== --- archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/util/RepositoryPathUtilTest.java (revision 1355676) +++ archiva-modules/archiva-web/archiva-webdav/src/test/java/org/apache/archiva/webdav/util/RepositoryPathUtilTest.java (working copy) @@ -20,12 +20,17 @@ */ import junit.framework.TestCase; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Test; +import org.junit.runner.RunWith; /** */ +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RepositoryPathUtilTest extends TestCase { + @Test public void testGetRepositoryId() { String href = "/path/to/my/resource"; @@ -38,6 +43,7 @@ assertEquals( "/", RepositoryPathUtil.getLogicalResource( href ) ); } + @Test public void testGetLogicalPath() { String href = "/repository/internal/org/apache/maven/someartifact.jar"; Index: archiva-modules/metadata/metadata-repository-api/pom.xml =================================================================== --- archiva-modules/metadata/metadata-repository-api/pom.xml (revision 1355676) +++ archiva-modules/metadata/metadata-repository-api/pom.xml (working copy) @@ -49,10 +49,11 @@ archiva-repository-admin-api - org.springframework - spring-test + org.apache.archiva + archiva-testutil + ${project.version} test - + org.slf4j slf4j-api Index: archiva-modules/metadata/metadata-repository-api/src/test/java/org/apache/archiva/metadata/repository/AbstractMetadataRepositoryTest.java =================================================================== --- archiva-modules/metadata/metadata-repository-api/src/test/java/org/apache/archiva/metadata/repository/AbstractMetadataRepositoryTest.java (revision 1355676) +++ archiva-modules/metadata/metadata-repository-api/src/test/java/org/apache/archiva/metadata/repository/AbstractMetadataRepositoryTest.java (working copy) @@ -38,7 +38,6 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import java.util.ArrayList; import java.util.Arrays; @@ -49,8 +48,9 @@ import java.util.HashMap; import java.util.List; import java.util.Map; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public abstract class AbstractMetadataRepositoryTest extends TestCase Index: archiva-modules/plugins/audit/pom.xml =================================================================== --- archiva-modules/plugins/audit/pom.xml (revision 1355676) +++ archiva-modules/plugins/audit/pom.xml (working copy) @@ -37,10 +37,11 @@ metadata-repository-api - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + Index: archiva-modules/plugins/audit/src/test/java/org/apache/archiva/audit/AuditManagerTest.java =================================================================== --- archiva-modules/plugins/audit/src/test/java/org/apache/archiva/audit/AuditManagerTest.java (revision 1355676) +++ archiva-modules/plugins/audit/src/test/java/org/apache/archiva/audit/AuditManagerTest.java (working copy) @@ -35,7 +35,12 @@ import java.util.List; import java.util.Map; import java.util.TimeZone; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class AuditManagerTest extends TestCase { @@ -69,7 +74,8 @@ } @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -83,7 +89,8 @@ repository.setId( TEST_REPO_ID ); repository.setLocation( "" ); } - + + @Test public void testGetMostRecentEvents() throws Exception { @@ -122,6 +129,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetMostRecentEventsLessThan10() throws Exception { @@ -159,6 +167,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetMostRecentEventsInterleavedRepositories() throws Exception { @@ -207,6 +216,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetMostRecentEventsWhenEmpty() throws Exception @@ -222,6 +232,7 @@ metadataRepositoryControl.verify(); } + @Test public void testAddAuditEvent() throws Exception @@ -237,6 +248,7 @@ metadataRepositoryControl.verify(); } + @Test public void testAddAuditEventNoRepositoryId() throws Exception { @@ -252,6 +264,7 @@ metadataRepositoryControl.verify(); } + @Test public void testDeleteStats() throws Exception @@ -265,6 +278,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetEventsRangeInside() throws Exception @@ -298,6 +312,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetEventsRangeUpperOutside() throws Exception { @@ -335,6 +350,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetEventsRangeLowerOutside() throws Exception { @@ -372,6 +388,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetEventsRangeLowerAndUpperOutside() throws Exception { @@ -415,6 +432,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetEventsWithResource() throws Exception { @@ -458,6 +476,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetEventsWithNonExistantResource() throws Exception { @@ -497,6 +516,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetEventsRangeMultipleRepositories() throws Exception { @@ -546,6 +566,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetEventsRangeNotInside() throws Exception { Index: archiva-modules/plugins/maven2-repository/pom.xml =================================================================== --- archiva-modules/plugins/maven2-repository/pom.xml (revision 1355676) +++ archiva-modules/plugins/maven2-repository/pom.xml (working copy) @@ -129,8 +129,9 @@ test - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test Index: archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/dependency/tree/maven2/DependencyTreeBuilderTest.java =================================================================== --- archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/dependency/tree/maven2/DependencyTreeBuilderTest.java (revision 1355676) +++ archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/dependency/tree/maven2/DependencyTreeBuilderTest.java (working copy) @@ -34,14 +34,14 @@ import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.inject.Named; import java.io.File; import java.util.Collections; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = {"classpath*:/META-INF/spring-context.xml","classpath:/spring-context.xml"} ) public class DependencyTreeBuilderTest extends TestCase Index: archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/metadata/repository/storage/maven2/Maven2RepositoryMetadataResolverTest.java =================================================================== --- archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/metadata/repository/storage/maven2/Maven2RepositoryMetadataResolverTest.java (revision 1355676) +++ archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/metadata/repository/storage/maven2/Maven2RepositoryMetadataResolverTest.java (working copy) @@ -55,11 +55,12 @@ import java.util.Collections; import java.util.Comparator; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class Maven2RepositoryMetadataResolverTest extends TestCase Index: archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/metadata/repository/storage/maven2/MavenRepositoryMetadataReaderTest.java =================================================================== --- archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/metadata/repository/storage/maven2/MavenRepositoryMetadataReaderTest.java (revision 1355676) +++ archiva-modules/plugins/maven2-repository/src/test/java/org/apache/archiva/metadata/repository/storage/maven2/MavenRepositoryMetadataReaderTest.java (working copy) @@ -31,13 +31,14 @@ import java.io.File; import java.util.Arrays; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; /** * RepositoryMetadataReaderTest * * @version $Id$ */ -@RunWith( JUnit4.class ) +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class MavenRepositoryMetadataReaderTest extends TestCase { Index: archiva-modules/plugins/metadata-store-file/pom.xml =================================================================== --- archiva-modules/plugins/metadata-store-file/pom.xml (revision 1355676) +++ archiva-modules/plugins/metadata-store-file/pom.xml (working copy) @@ -56,6 +56,12 @@ test + org.apache.archiva + archiva-testutil + ${project.version} + test + + commons-logging commons-logging-api test Index: archiva-modules/plugins/metadata-store-jcr/pom.xml =================================================================== --- archiva-modules/plugins/metadata-store-jcr/pom.xml (revision 1355676) +++ archiva-modules/plugins/metadata-store-jcr/pom.xml (working copy) @@ -44,10 +44,11 @@ test - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + javax.jcr jcr Index: archiva-modules/plugins/problem-reports/pom.xml =================================================================== --- archiva-modules/plugins/problem-reports/pom.xml (revision 1355676) +++ archiva-modules/plugins/problem-reports/pom.xml (working copy) @@ -54,10 +54,11 @@ archiva-consumer-api - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + org.mockito mockito-all Index: archiva-modules/plugins/problem-reports/src/test/java/org/apache/archiva/reports/consumers/DuplicateArtifactsConsumerTest.java =================================================================== --- archiva-modules/plugins/problem-reports/src/test/java/org/apache/archiva/reports/consumers/DuplicateArtifactsConsumerTest.java (revision 1355676) +++ archiva-modules/plugins/problem-reports/src/test/java/org/apache/archiva/reports/consumers/DuplicateArtifactsConsumerTest.java (working copy) @@ -36,7 +36,6 @@ import org.mockito.Matchers; import org.springframework.context.ApplicationContext; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import javax.inject.Named; @@ -44,11 +43,12 @@ import java.io.FileNotFoundException; import java.util.Arrays; import java.util.Date; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.mockito.Mockito.*; @SuppressWarnings( { "ThrowableInstanceNeverThrown" } ) -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath:/spring-context.xml" } ) public class DuplicateArtifactsConsumerTest extends TestCase Index: archiva-modules/plugins/repository-statistics/pom.xml =================================================================== --- archiva-modules/plugins/repository-statistics/pom.xml (revision 1355676) +++ archiva-modules/plugins/repository-statistics/pom.xml (working copy) @@ -50,10 +50,11 @@ jackrabbit-jcr-commons - org.slf4j - slf4j-simple + org.apache.archiva + archiva-testutil + ${project.version} test - + org.slf4j jcl-over-slf4j Index: archiva-modules/plugins/repository-statistics/src/test/java/org/apache/archiva/metadata/repository/stats/JcrRepositoryStatisticsGatheringTest.java =================================================================== --- archiva-modules/plugins/repository-statistics/src/test/java/org/apache/archiva/metadata/repository/stats/JcrRepositoryStatisticsGatheringTest.java (revision 1355676) +++ archiva-modules/plugins/repository-statistics/src/test/java/org/apache/archiva/metadata/repository/stats/JcrRepositoryStatisticsGatheringTest.java (working copy) @@ -39,9 +39,14 @@ import java.util.Calendar; import java.util.Date; import java.util.zip.GZIPInputStream; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.After; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; import static org.mockito.Mockito.*; - +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class JcrRepositoryStatisticsGatheringTest extends TestCase { @@ -58,7 +63,8 @@ private Session session; @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -100,7 +106,8 @@ } @Override - protected void tearDown() + @After + public void tearDown() throws Exception { session.logout(); @@ -108,6 +115,7 @@ super.tearDown(); } + @Test public void testJcrStatisticsQuery() throws Exception { Index: archiva-modules/plugins/repository-statistics/src/test/java/org/apache/archiva/metadata/repository/stats/RepositoryStatisticsManagerTest.java =================================================================== --- archiva-modules/plugins/repository-statistics/src/test/java/org/apache/archiva/metadata/repository/stats/RepositoryStatisticsManagerTest.java (revision 1355676) +++ archiva-modules/plugins/repository-statistics/src/test/java/org/apache/archiva/metadata/repository/stats/RepositoryStatisticsManagerTest.java (working copy) @@ -35,7 +35,12 @@ import java.util.Map; import java.util.TimeZone; import javax.jcr.Session; +import org.apache.archiva.test.ArchivaBlockJUnit4ClassRunner; +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +@RunWith( ArchivaBlockJUnit4ClassRunner.class ) public class RepositoryStatisticsManagerTest extends TestCase { @@ -63,7 +68,8 @@ } @Override - protected void setUp() + @Before + public void setUp() throws Exception { super.setUp(); @@ -74,6 +80,7 @@ metadataRepository = (MetadataRepository) metadataRepositoryControl.getMock(); } + @Test public void testGetLatestStats() throws Exception { @@ -113,6 +120,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetLatestStatsWhenEmpty() throws Exception { @@ -127,6 +135,7 @@ metadataRepositoryControl.verify(); } + @Test public void testAddNewStats() throws Exception { @@ -165,6 +174,7 @@ metadataRepositoryControl.verify(); } + @Test public void testDeleteStats() throws Exception { @@ -210,6 +220,7 @@ metadataRepositoryControl.verify(); } + @Test public void testDeleteStatsWhenEmpty() throws Exception { @@ -229,6 +240,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetStatsRangeInside() throws Exception { @@ -272,6 +284,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetStatsRangeUpperOutside() throws Exception { @@ -320,6 +333,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetStatsRangeLowerOutside() throws Exception { @@ -368,6 +382,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetStatsRangeLowerAndUpperOutside() throws Exception { @@ -422,6 +437,7 @@ metadataRepositoryControl.verify(); } + @Test public void testGetStatsRangeNotInside() throws Exception { Index: archiva-modules/plugins/stage-repository-merge/pom.xml =================================================================== --- archiva-modules/plugins/stage-repository-merge/pom.xml (revision 1355676) +++ archiva-modules/plugins/stage-repository-merge/pom.xml (working copy) @@ -86,6 +86,12 @@ wagon-http-lightweight test + + org.apache.archiva + archiva-testutil + ${project.version} + test + Index: archiva-modules/plugins/stage-repository-merge/src/test/java/org/apache/archiva/stagerepository/merge/Maven2RepositoryMergerTest.java =================================================================== --- archiva-modules/plugins/stage-repository-merge/src/test/java/org/apache/archiva/stagerepository/merge/Maven2RepositoryMergerTest.java (revision 1355676) +++ archiva-modules/plugins/stage-repository-merge/src/test/java/org/apache/archiva/stagerepository/merge/Maven2RepositoryMergerTest.java (working copy) @@ -31,16 +31,16 @@ import org.junit.runner.RunWith; import org.mockito.MockitoAnnotations; import org.springframework.test.context.ContextConfiguration; -import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; import javax.inject.Inject; import java.io.File; import java.util.ArrayList; import java.util.List; +import org.apache.archiva.test.ArchivaSpringJUnit4ClassRunner; import static org.mockito.Mockito.*; -@RunWith( SpringJUnit4ClassRunner.class ) +@RunWith( ArchivaSpringJUnit4ClassRunner.class ) @ContextConfiguration( locations = { "classpath*:/META-INF/spring-context.xml", "classpath*:/spring-context.xml" } ) public class Maven2RepositoryMergerTest extends TestCase @@ -57,6 +57,7 @@ private MetadataRepository metadataRepository; @Before + @Override public void setUp() throws Exception { Index: archiva-testutil/pom.xml =================================================================== --- archiva-testutil/pom.xml (revision 0) +++ archiva-testutil/pom.xml (working copy) @@ -0,0 +1,51 @@ + + + + + + org.apache.archiva + archiva + 1.4-M3-SNAPSHOT + + 4.0.0 + archiva-testutil + Archiva :: Test Utility + + + org.springframework + spring-test + + + junit + junit + compile + + + xmlunit + xmlunit + compile + + + org.slf4j + slf4j-simple + compile + + + Index: archiva-testutil/src/main/java/org/apache/archiva/test/ArchivaBlockJUnit4ClassRunner.java =================================================================== --- archiva-testutil/src/main/java/org/apache/archiva/test/ArchivaBlockJUnit4ClassRunner.java (revision 0) +++ archiva-testutil/src/main/java/org/apache/archiva/test/ArchivaBlockJUnit4ClassRunner.java (working copy) @@ -0,0 +1,38 @@ +/* + * Copyright 2012 The Apache Software Foundation. + * + * Licensed 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.archiva.test; + +import java.util.List; +import org.junit.runners.BlockJUnit4ClassRunner; +import org.junit.runners.model.FrameworkMethod; +import org.junit.runners.model.InitializationError; + +/** + * + * @author Eric + */ +public class ArchivaBlockJUnit4ClassRunner extends BlockJUnit4ClassRunner { + + public ArchivaBlockJUnit4ClassRunner(Class klass) throws InitializationError { + super(klass); + } + + @Override + protected List computeTestMethods() + { + return ListGenerator.getShuffleList(super.computeTestMethods()); + } +} Index: archiva-testutil/src/main/java/org/apache/archiva/test/ArchivaSpringJUnit4ClassRunner.java =================================================================== --- archiva-testutil/src/main/java/org/apache/archiva/test/ArchivaSpringJUnit4ClassRunner.java (revision 0) +++ archiva-testutil/src/main/java/org/apache/archiva/test/ArchivaSpringJUnit4ClassRunner.java (working copy) @@ -0,0 +1,40 @@ +/* + * Copyright 2012 The Apache Software Foundation. + * + * Licensed 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.archiva.test; + +import java.util.List; +import org.junit.runners.model.FrameworkMethod; +import org.junit.runners.model.InitializationError; +import org.springframework.test.context.junit4.SpringJUnit4ClassRunner; + +/** + * + * @author Eric + */ +public class ArchivaSpringJUnit4ClassRunner extends SpringJUnit4ClassRunner { + + public ArchivaSpringJUnit4ClassRunner(Class clazz) throws InitializationError { + super(clazz); + } + + @Override + protected List computeTestMethods() + { + return ListGenerator.getShuffleList(super.computeTestMethods()); + } + + +} Index: archiva-testutil/src/main/java/org/apache/archiva/test/ListGenerator.java =================================================================== --- archiva-testutil/src/main/java/org/apache/archiva/test/ListGenerator.java (revision 0) +++ archiva-testutil/src/main/java/org/apache/archiva/test/ListGenerator.java (working copy) @@ -0,0 +1,60 @@ +/* + * Copyright 2012 The Apache Software Foundation. + * + * Licensed 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.archiva.test; + +import java.util.ArrayList; +import java.util.Collections; +import java.util.List; +import org.junit.runners.model.FrameworkMethod; + +/** + * + * @author Eric + */ +public class ListGenerator +{ + private static int MAXROUND=2; + private ListGenerator() + { + } + + static List getShuffleList(List computeTestMethods) + { + List generated = new ArrayList(); + String javaSpecVersion = System.getProperty("java.specification.version"); + // 1.6 1.5 version not shuffled to allow build + if (javaSpecVersion.equals("1.6") || javaSpecVersion.equals("1.5")) + { + generated = computeTestMethods; + } else + { // 1.7 and more generated shuffled list + // double test method to have more change of failure + for (int i = 0;ihttp://archiva.apache.org + archiva-testutil archiva-cli archiva-modules archiva-docs