From d3b068200339c3c9e06bf1111438b3055c0069d2 Mon Sep 17 00:00:00 2001
From: Olivier Lamy <olamy@apache.org>
Date: Mon, 26 Sep 2011 22:49:57 +0000
Subject: [PATCH] [MRM-1516] missing dependency on velocity -
 ClassNotFoundException org.apache.velocity.tools.view.ToolboxManager and fis
 unit tests.

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1176112 13f79535-47bb-0310-9956-ffa450edef68
---
 .../DefaultDownloadRemoteIndexScheduler.java     |  2 +-
 .../archiva-web/archiva-webapp/pom.xml           | 16 ++++++++++++++++
 .../archiva/web/action/BrowseActionTest.java     |  3 +++
 .../AddRemoteRepositoryActionTest.java           |  2 +-
 4 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/main/java/org/apache/archiva/scheduler/indexing/DefaultDownloadRemoteIndexScheduler.java b/archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/main/java/org/apache/archiva/scheduler/indexing/DefaultDownloadRemoteIndexScheduler.java
index b0be98c0e..8c05d5788 100644
--- a/archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/main/java/org/apache/archiva/scheduler/indexing/DefaultDownloadRemoteIndexScheduler.java
+++ b/archiva-modules/archiva-scheduler/archiva-scheduler-indexing/src/main/java/org/apache/archiva/scheduler/indexing/DefaultDownloadRemoteIndexScheduler.java
@@ -145,7 +145,7 @@ public class DefaultDownloadRemoteIndexScheduler
     }
 
     @PreDestroy
-    private void shutdown()
+    public void shutdown()
         throws RepositoryAdminException, IOException
     {
         for ( RemoteRepository remoteRepository : remoteRepositoryAdmin.getRemoteRepositories() )
diff --git a/archiva-modules/archiva-web/archiva-webapp/pom.xml b/archiva-modules/archiva-web/archiva-webapp/pom.xml
index 3d2e17b93..b856b2121 100644
--- a/archiva-modules/archiva-web/archiva-webapp/pom.xml
+++ b/archiva-modules/archiva-web/archiva-webapp/pom.xml
@@ -425,6 +425,22 @@
       <artifactId>metadata-store-file</artifactId>
       <scope>runtime</scope>
     </dependency>
+    <dependency>
+      <groupId>org.apache.velocity</groupId>
+      <artifactId>velocity-tools</artifactId>
+      <version>1.3</version>
+      <scope>runtime</scope>
+      <exclusions>
+        <exclusion>
+          <groupId>commons-logging</groupId>
+          <artifactId>commons-logging</artifactId>
+        </exclusion>
+        <exclusion>
+          <groupId>velocity</groupId>
+          <artifactId>velocity-dep</artifactId>
+        </exclusion>
+      </exclusions>
+    </dependency>
     <dependency>
       <groupId>org.apache.archiva</groupId>
       <!--  FIXME: temporary coupling to plugin, should be runtime -->
diff --git a/archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/BrowseActionTest.java b/archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/BrowseActionTest.java
index ea9a976ad..97dfafd7e 100644
--- a/archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/BrowseActionTest.java
+++ b/archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/BrowseActionTest.java
@@ -25,6 +25,8 @@ import org.apache.archiva.metadata.model.ProjectVersionMetadata;
 import org.apache.archiva.metadata.repository.RepositorySession;
 import org.apache.archiva.metadata.repository.memory.TestMetadataResolver;
 import org.apache.archiva.metadata.repository.memory.TestRepositorySessionFactory;
+import org.apache.archiva.scheduler.indexing.DefaultDownloadRemoteIndexScheduler;
+import org.springframework.context.support.GenericApplicationContext;
 
 import java.util.Arrays;
 import java.util.Collections;
@@ -406,6 +408,7 @@ public class BrowseActionTest
         throws Exception
     {
         super.tearDown();
+        applicationContext.getBean( DefaultDownloadRemoteIndexScheduler.class ).shutdown();
         setObservableRepos( Lists.<String>newArrayList( "test-repo" ) );
     }
 }
\ No newline at end of file
diff --git a/archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddRemoteRepositoryActionTest.java b/archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddRemoteRepositoryActionTest.java
index 6d4b8e22c..ef3491675 100644
--- a/archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddRemoteRepositoryActionTest.java
+++ b/archiva-modules/archiva-web/archiva-webapp/src/test/java/org/apache/archiva/web/action/admin/repositories/AddRemoteRepositoryActionTest.java
@@ -62,7 +62,7 @@ public class AddRemoteRepositoryActionTest
     }
 
     public void testSecureActionBundle()
-        throws SecureActionException
+        throws Exception
     {
         archivaConfiguration.getConfiguration();
         archivaConfigurationControl.setReturnValue( new Configuration() );