fix use of applicationContext.getBeansOfType instead of the old lookupMap.

The key is now interface#hint whereas was only hint with plexus
arghh !


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

4 files changed, 30 insertions, 7 deletions

diff --git a/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java b/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java
index 6e9861dc5..094ec86a2 100644
--- a/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java
+++ b/archiva-modules/archiva-base/archiva-proxy/src/main/java/org/apache/maven/archiva/proxy/DefaultRepositoryProxyConnectors.java
@@ -103,14 +103,14 @@ public class DefaultRepositoryProxyConnectors
      * plexus.requirement
      */
     @Inject
-    @Named(value = "repositoryContentFactory#default")
+    @Named( value = "repositoryContentFactory#default" )
     private RepositoryContentFactory repositoryFactory;
 
     /**
      * plexus.requirement
      */
     @Inject
-    @Named(value="metadataTools#default")
+    @Named( value = "metadataTools#default" )
     private MetadataTools metadataTools;
 
     @Inject
@@ -890,9 +890,12 @@ public class DefaultRepositoryProxyConnectors
     {
         for ( Entry<String, ? extends DownloadPolicy> entry : policies.entrySet() )
         {
-            String key = entry.getKey();
+            // olamy with spring rolehint is now downloadPolicy#hint
+            // so substring after last # to get the hint as with plexus
+            String key = StringUtils.substringAfterLast( entry.getKey(), "#" );
             DownloadPolicy policy = entry.getValue();
             String defaultSetting = policy.getDefaultOption();
+
             String setting = StringUtils.defaultString( settings.get( key ), defaultSetting );
 
             log.debug( "Applying [{}] policy with [{}]", key, setting );
diff --git a/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/AbstractProxyTestCase.java b/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/AbstractProxyTestCase.java
index 09815b4c4..4ab400a34 100644
--- a/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/AbstractProxyTestCase.java
+++ b/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/AbstractProxyTestCase.java
@@ -610,18 +610,33 @@ public abstract class AbstractProxyTestCase
 
     protected void setManagedNewerThanRemote( File managedFile, File remoteFile )
     {
+        setManagedNewerThanRemote(managedFile, remoteFile, 55000);
+    }
+
+    protected void setManagedNewerThanRemote( File managedFile, File remoteFile, long time )
+    {
         assertTrue( "Managed File should exist: ", managedFile.exists() );
         assertTrue( "Remote File should exist: ", remoteFile.exists() );
 
-        managedFile.setLastModified( remoteFile.lastModified() + 55000 );
+        managedFile.setLastModified( remoteFile.lastModified() + time );
+
+        assertTrue( managedFile.lastModified() > remoteFile.lastModified() );
     }
 
     protected void setManagedOlderThanRemote( File managedFile, File remoteFile )
     {
+        setManagedOlderThanRemote(managedFile, remoteFile, 55000 );
+    }
+
+    protected void setManagedOlderThanRemote( File managedFile, File remoteFile, long time )
+    {
         assertTrue( "Managed File should exist: ", managedFile.exists() );
         assertTrue( "Remote File should exist: ", remoteFile.exists() );
 
-        managedFile.setLastModified( remoteFile.lastModified() - 55000 );
+        managedFile.setLastModified( remoteFile.lastModified() - time );
+
+        assertTrue( managedFile.lastModified() < remoteFile.lastModified() );
+
     }
 
     protected void assertNotModified( File file, long expectedModificationTime )
diff --git a/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/MockConfiguration.java b/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/MockConfiguration.java
index 9a6e78380..f116e14c5 100644
--- a/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/MockConfiguration.java
+++ b/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/MockConfiguration.java
@@ -23,6 +23,7 @@ import org.apache.maven.archiva.configuration.ArchivaConfiguration;
 import org.apache.maven.archiva.configuration.Configuration;
 import org.apache.maven.archiva.configuration.ConfigurationListener;
 import org.apache.maven.archiva.configuration.FileType;
+import org.apache.maven.archiva.configuration.FileTypes;
 import org.apache.maven.archiva.configuration.ManagedRepositoryConfiguration;
 import org.apache.maven.archiva.configuration.RepositoryScanningConfiguration;
 import org.apache.maven.archiva.repository.ManagedRepositoryContent;
@@ -75,12 +76,16 @@ public class MockConfiguration
     public void initialize()
         throws Exception
     {
+
         configuration.setRepositoryScanning( new RepositoryScanningConfiguration()
         {
             @Override
             public List<FileType> getFileTypes()
             {
-                return Collections.emptyList();
+                FileType fileType = new FileType();
+                fileType.setId( FileTypes.ARTIFACTS );
+                fileType.setPatterns( Collections.singletonList( "**/*" ) );
+                return Collections.singletonList( fileType );
             }
         } );
     }
diff --git a/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/SnapshotTransferTest.java b/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/SnapshotTransferTest.java
index 0e87f8fbd..f1998971d 100644
--- a/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/SnapshotTransferTest.java
+++ b/archiva-modules/archiva-base/archiva-proxy/src/test/java/org/apache/maven/archiva/proxy/SnapshotTransferTest.java
@@ -229,7 +229,7 @@ public class SnapshotTransferTest
         File expectedFile = new File( managedDefaultDir, path );
         File remoteFile = new File( REPOPATH_PROXIED1, path );
 
-        setManagedNewerThanRemote( expectedFile, remoteFile );
+        setManagedNewerThanRemote( expectedFile, remoteFile, 12000000 );
         long expectedTimestamp = expectedFile.lastModified(); 
         
         ArtifactReference artifact = managedDefaultRepository.toArtifactReference( path );