Author: olamy
Date: Thu Dec  6 23:01:51 2012
New Revision: 1418123

URL: http://svn.apache.org/viewvc?rev=1418123&view=rev
Log:
fix unit tests

Modified:
    
archiva/trunk/archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/src/main/java/org/apache/archiva/admin/repository/runtime/DefaultArchivaRuntimeConfigurationAdmin.java
    
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-merge-index-download.xml
    
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-rss-servlet.xml
    
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-test-common.xml
    
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-with-jcr.xml

Modified: 
archiva/trunk/archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/src/main/java/org/apache/archiva/admin/repository/runtime/DefaultArchivaRuntimeConfigurationAdmin.java
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/src/main/java/org/apache/archiva/admin/repository/runtime/DefaultArchivaRuntimeConfigurationAdmin.java?rev=1418123&r1=1418122&r2=1418123&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/src/main/java/org/apache/archiva/admin/repository/runtime/DefaultArchivaRuntimeConfigurationAdmin.java
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-base/archiva-repository-admin/archiva-repository-admin-default/src/main/java/org/apache/archiva/admin/repository/runtime/DefaultArchivaRuntimeConfigurationAdmin.java
 Thu Dec  6 23:01:51 2012
@@ -24,6 +24,7 @@ import org.apache.archiva.admin.model.be
 import org.apache.archiva.admin.model.beans.ArchivaRuntimeConfiguration;
 import org.apache.archiva.admin.model.runtime.ArchivaRuntimeConfigurationAdmin;
 import org.apache.archiva.admin.repository.AbstractRepositoryAdmin;
+import org.apache.archiva.configuration.ArchivaConfiguration;
 import org.apache.archiva.configuration.Configuration;
 import org.apache.archiva.configuration.IndeterminateConfigurationException;
 import org.apache.archiva.configuration.RedbackRuntimeConfiguration;
@@ -32,6 +33,8 @@ import org.apache.archiva.redback.config
 import org.apache.archiva.redback.configuration.UserConfigurationException;
 import org.apache.archiva.redback.configuration.UserConfigurationKeys;
 import org.apache.commons.lang.StringUtils;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.stereotype.Service;
 
 import javax.annotation.PostConstruct;
@@ -45,10 +48,14 @@ import java.util.List;
  */
 @Service( "userConfiguration#archiva" )
 public class DefaultArchivaRuntimeConfigurationAdmin
-    extends AbstractRepositoryAdmin
     implements ArchivaRuntimeConfigurationAdmin, UserConfiguration
 {
 
+    protected Logger log = LoggerFactory.getLogger( getClass() );
+
+    @Inject
+    private ArchivaConfiguration archivaConfiguration;
+
     @Inject
     @Named( value = "userConfiguration#redback" )
     UserConfiguration userConfiguration;
@@ -110,18 +117,18 @@ public class DefaultArchivaRuntimeConfig
 
     public ArchivaRuntimeConfiguration getArchivaRuntimeConfiguration()
     {
-        return build( 
getArchivaConfiguration().getConfiguration().getRedbackRuntimeConfiguration() );
+        return build( 
archivaConfiguration.getConfiguration().getRedbackRuntimeConfiguration() );
     }
 
     public void updateArchivaRuntimeConfiguration( ArchivaRuntimeConfiguration 
archivaRuntimeConfiguration )
         throws RepositoryAdminException
     {
         RedbackRuntimeConfiguration runtimeConfiguration = build( 
archivaRuntimeConfiguration );
-        Configuration configuration = 
getArchivaConfiguration().getConfiguration();
+        Configuration configuration = archivaConfiguration.getConfiguration();
         configuration.setRedbackRuntimeConfiguration( runtimeConfiguration );
         try
         {
-            getArchivaConfiguration().save( configuration );
+            archivaConfiguration.save( configuration );
         }
         catch ( RegistryException e )
         {
@@ -135,7 +142,15 @@ public class DefaultArchivaRuntimeConfig
 
     private ArchivaRuntimeConfiguration build( RedbackRuntimeConfiguration 
runtimeConfiguration )
     {
-        return new BeanReplicator().replicateBean( runtimeConfiguration, 
ArchivaRuntimeConfiguration.class );
+        ArchivaRuntimeConfiguration archivaRuntimeConfiguration = new 
BeanReplicator().replicateBean( runtimeConfiguration, 
ArchivaRuntimeConfiguration.class );
+
+        if (archivaRuntimeConfiguration.getArchivaLdapConfiguration() == null)
+        {
+            // prevent NPE
+            archivaRuntimeConfiguration.setArchivaLdapConfiguration( new 
ArchivaLdapConfiguration() );
+        }
+
+        return archivaRuntimeConfiguration;
     }
 
     private RedbackRuntimeConfiguration build( ArchivaRuntimeConfiguration 
archivaRuntimeConfiguration )

Modified: 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-merge-index-download.xml
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-merge-index-download.xml?rev=1418123&r1=1418122&r2=1418123&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-merge-index-download.xml
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-merge-index-download.xml
 Thu Dec  6 23:01:51 2012
@@ -52,5 +52,5 @@
   </bean>
 
   <bean name="repositorySessionFactory" 
class="org.apache.archiva.webtest.memory.TestRepositorySessionFactory"/>
-
+  <alias name="userConfiguration#archiva" alias="userConfiguration#default"/>
 </beans>

Modified: 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-rss-servlet.xml
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-rss-servlet.xml?rev=1418123&r1=1418122&r2=1418123&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-rss-servlet.xml
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-rss-servlet.xml
 Thu Dec  6 23:01:51 2012
@@ -65,5 +65,5 @@
   </bean>
   
   <bean name="RoleManagerStub" 
class="org.apache.archiva.web.rss.RoleManagerStub"/>
-
+  <alias name="userConfiguration#archiva" alias="userConfiguration#default"/>
 </beans>
\ No newline at end of file

Modified: 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-test-common.xml
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-test-common.xml?rev=1418123&r1=1418122&r2=1418123&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-test-common.xml
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-test-common.xml
 Thu Dec  6 23:01:51 2012
@@ -21,13 +21,11 @@
 <beans xmlns="http://www.springframework.org/schema/beans";
        xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
        xmlns:context="http://www.springframework.org/schema/context";
-       xmlns:jaxrs="http://cxf.apache.org/jaxrs";
        xsi:schemaLocation="http://www.springframework.org/schema/beans
            http://www.springframework.org/schema/beans/spring-beans-3.0.xsd
            http://www.springframework.org/schema/context
-           http://www.springframework.org/schema/context/spring-context-3.0.xsd
-           http://cxf.apache.org/jaxrs
-           http://cxf.apache.org/schemas/jaxrs.xsd"; default-lazy-init="true">
+           
http://www.springframework.org/schema/context/spring-context-3.0.xsd";
+       default-lazy-init="true">
 
   <context:annotation-config/>
   <context:component-scan
@@ -83,5 +81,5 @@
   </bean>
 
   <alias name="repositorySessionFactory#jcr" alias="repositorySessionFactory"/>
-
+  <alias name="userConfiguration#archiva" alias="userConfiguration#default"/>
 </beans>
\ No newline at end of file

Modified: 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-with-jcr.xml
URL: 
http://svn.apache.org/viewvc/archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-with-jcr.xml?rev=1418123&r1=1418122&r2=1418123&view=diff
==============================================================================
--- 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-with-jcr.xml
 (original)
+++ 
archiva/trunk/archiva-modules/archiva-web/archiva-web-common/src/test/resources/spring-context-with-jcr.xml
 Thu Dec  6 23:01:51 2012
@@ -49,5 +49,5 @@
       </props>
     </property>
   </bean>
-
+  <alias name="userConfiguration#archiva" alias="userConfiguration#default"/>
 </beans>


Reply via email to