[MRM-347] prevent creating /${appserver.home} directory which is not populated under...

git-svn-id: https://svn.apache.org/repos/asf/maven/archiva/trunk@562369 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/archiva-base/archiva-configuration/src/main/resources/org/apache/maven/archiva/configuration/default-archiva.xml b/archiva-base/archiva-configuration/src/main/resources/org/apache/maven/archiva/configuration/default-archiva.xml
index 99308d4d36d978ec2067022d250e0ed48a886ad9..d4b17fc3e48ff038e0bb77f4f2d836f50c3e973b 100644 (file)
--- a/archiva-base/archiva-configuration/src/main/resources/org/apache/maven/archiva/configuration/default-archiva.xml
+++ b/archiva-base/archiva-configuration/src/main/resources/org/apache/maven/archiva/configuration/default-archiva.xml
@@ -5,7 +5,7 @@
     <repository>
       <id>internal</id>
       <name>Archiva Managed Internal Repository</name>
-      <url>file://${appserver.home}/repositories/internal</url>
+      <url>file://${appserver.base}/repositories/internal</url>
       <layout>default</layout>
       <releases>true</releases>
       <snapshots>false</snapshots>
@@ -15,7 +15,7 @@
     <repository>
       <id>snapshots</id>
       <name>Archiva Managed Snapshot Repository</name>
-      <url>file://${appserver.home}/repositories/snapshots</url>
+      <url>file://${appserver.base}/repositories/snapshots</url>
       <layout>default</layout>
       <releases>false</releases>
       <snapshots>true</snapshots>
@@ -46,7 +46,7 @@
     <proxyConnector>
       <sourceRepoId>internal</sourceRepoId>
       <targetRepoId>central</targetRepoId>
-      <proxyId />
+      <proxyId/>
       <policies>
         <snapshots>disabled</snapshots>
         <releases>once</releases>
@@ -60,7 +60,7 @@
     <proxyConnector>
       <sourceRepoId>internal</sourceRepoId>
       <targetRepoId>maven2-repository.dev.java.net</targetRepoId>
-      <proxyId />
+      <proxyId/>
       <policies>
         <snapshots>disabled</snapshots>
         <releases>once</releases>

diff --git a/archiva-base/archiva-configuration/src/test/conf/repository-manager.xml b/archiva-base/archiva-configuration/src/test/conf/repository-manager.xml
index 5cb654aec2d96834c24f48fbbc0c713018659f30..5f35f5d3761ec4146ab1f4a05dea64d90d4fc023 100644 (file)
--- a/archiva-base/archiva-configuration/src/test/conf/repository-manager.xml
+++ b/archiva-base/archiva-configuration/src/test/conf/repository-manager.xml
@@ -23,7 +23,7 @@
     <repository>
       <id>internal</id>
       <name>Archiva Managed Internal Repository</name>
-      <url>file://${appserver.home}/repositories/internal</url>
+      <url>file://${appserver.base}/repositories/internal</url>
       <layout>default</layout>
       <releases>true</releases>
       <snapshots>false</snapshots>
@@ -33,7 +33,7 @@
     <repository>
       <id>snapshots</id>
       <name>Archiva Managed Snapshot Repository</name>
-      <url>file://${appserver.home}/repositories/internal</url>
+      <url>file://${appserver.base}/repositories/internal</url>
       <layout>default</layout>
       <releases>false</releases>
       <snapshots>true</snapshots>
@@ -64,7 +64,7 @@
     <proxyConnector>
       <sourceRepoId>internal</sourceRepoId>
       <targetRepoId>central</targetRepoId>
-      <proxyId />
+      <proxyId/>
       <snapshotsPolicy>disabled</snapshotsPolicy>
       <releasePolicy>never</releasePolicy>
       <failurePolicy>not-found</failurePolicy>
@@ -72,7 +72,7 @@
     <proxyConnector>
       <sourceRepoId>internal</sourceRepoId>
       <targetRepoId>maven2-repository.dev.java.net</targetRepoId>
-      <proxyId />
+      <proxyId/>
       <snapshotsPolicy>disabled</snapshotsPolicy>
       <releasePolicy>never</releasePolicy>
       <failurePolicy>not-found</failurePolicy>

diff --git a/archiva-base/archiva-configuration/src/test/java/org/apache/maven/archiva/configuration/ArchivaConfigurationTest.java b/archiva-base/archiva-configuration/src/test/java/org/apache/maven/archiva/configuration/ArchivaConfigurationTest.java
index 96816674584370ea391bfc69689c86d0272c4342..21aaf1b5905a00fcedc840edcef02b03bd4596b7 100644 (file)
--- a/archiva-base/archiva-configuration/src/test/java/org/apache/maven/archiva/configuration/ArchivaConfigurationTest.java
+++ b/archiva-base/archiva-configuration/src/test/java/org/apache/maven/archiva/configuration/ArchivaConfigurationTest.java
@@ -30,13 +30,15 @@ import java.util.List;
  *
  * @author <a href="mailto:brett@apache.org">Brett Porter</a>
  */
-public class ArchivaConfigurationTest extends PlexusTestCase
+public class ArchivaConfigurationTest
+    extends PlexusTestCase
 {
-    public void testDefaults() throws Exception
+    public void testDefaults()
+        throws Exception
     {
         ArchivaConfiguration archivaConfiguration =
             (ArchivaConfiguration) lookup( ArchivaConfiguration.class, "test-defaults" );
-        
+
         Configuration configuration = archivaConfiguration.getConfiguration();
 
         // check default configuration
@@ -44,7 +46,8 @@ public class ArchivaConfigurationTest extends PlexusTestCase
         assertTrue( "check configuration has default elements", configuration.getRepositories().isEmpty() );
     }
 
-    public void testGetConfiguration() throws Exception
+    public void testGetConfiguration()
+        throws Exception
     {
         ArchivaConfiguration archivaConfiguration =
             (ArchivaConfiguration) lookup( ArchivaConfiguration.class.getName(), "test-configuration" );
@@ -75,23 +78,24 @@ public class ArchivaConfigurationTest extends PlexusTestCase
         RepositoryConfiguration repository =
             (RepositoryConfiguration) configuration.getRepositories().iterator().next();
 
-        assertEquals( "check managed repositories", "file://${appserver.home}/repositories/internal",
+        assertEquals( "check managed repositories", "file://${appserver.base}/repositories/internal",
                       repository.getUrl() );
         assertEquals( "check managed repositories", "Archiva Managed Internal Repository", repository.getName() );
         assertEquals( "check managed repositories", "internal", repository.getId() );
         assertEquals( "check managed repositories", "default", repository.getLayout() );
         assertTrue( "check managed repositories", repository.isIndexed() );
-        
+
         WebappConfiguration webapp = (WebappConfiguration) configuration.getWebapp();
         assertNotNull( "check webapp", webapp );
-        
+
         UserInterfaceOptions ui = (UserInterfaceOptions) webapp.getUi();
-        assertNotNull( "check webapp ui", ui  );
+        assertNotNull( "check webapp ui", ui );
         assertTrue( "check showFindArtifacts", ui.isShowFindArtifacts() );
         assertTrue( "check appletFindEnabled", ui.isAppletFindEnabled() );
     }
 
-    public void testGetConfigurationSystemOverride() throws Exception
+    public void testGetConfigurationSystemOverride()
+        throws Exception
     {
         ArchivaConfiguration archivaConfiguration =
             (ArchivaConfiguration) lookup( ArchivaConfiguration.class.getName(), "test-configuration" );
@@ -104,7 +108,8 @@ public class ArchivaConfigurationTest extends PlexusTestCase
         //        assertEquals( "check indexPath", ".index", configuration.getIndexPath() );
     }
 
-    public void testStoreConfiguration() throws Exception
+    public void testStoreConfiguration()
+        throws Exception
     {
         File file = getTestFile( "target/test/test-file.xml" );
         file.delete();
@@ -134,7 +139,8 @@ public class ArchivaConfigurationTest extends PlexusTestCase
         //        assertEquals( "check value", "index-path", configuration.getIndexPath() );
     }
 
-    public void testStoreConfigurationUser() throws Exception
+    public void testStoreConfigurationUser()
+        throws Exception
     {
         File baseFile = getTestFile( "target/test/test-file.xml" );
         baseFile.delete();
@@ -164,7 +170,8 @@ public class ArchivaConfigurationTest extends PlexusTestCase
         //        assertEquals( "check value", "index-path", configuration.getIndexPath() );
     }
 
-    public void testStoreConfigurationFallback() throws Exception
+    public void testStoreConfigurationFallback()
+        throws Exception
     {
         File baseFile = getTestFile( "target/test/test-file.xml" );
         baseFile.delete();
@@ -194,7 +201,8 @@ public class ArchivaConfigurationTest extends PlexusTestCase
         //        assertEquals( "check value", "index-path", configuration.getIndexPath() );
     }
 
-    public void testRemoveProxiedRepositoryAndStoreConfiguration() throws Exception
+    public void testRemoveProxiedRepositoryAndStoreConfiguration()
+        throws Exception
     {
         // MRM-300