With little adjustment to test no need to have 2 contructors
git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1748892 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/main/java/org/apache/maven/plugins/assembly/artifact/DefaultDependencyResolver.java b/src/main/java/org/apache/maven/plugins/assembly/artifact/DefaultDependencyResolver.java
index d80b798..7eda4ca 100644
--- a/src/main/java/org/apache/maven/plugins/assembly/artifact/DefaultDependencyResolver.java
+++ b/src/main/java/org/apache/maven/plugins/assembly/artifact/DefaultDependencyResolver.java
@@ -19,6 +19,15 @@
* under the License.
*/
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.LinkedHashMap;
+import java.util.LinkedHashSet;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+
import org.apache.maven.artifact.Artifact;
import org.apache.maven.artifact.factory.ArtifactFactory;
import org.apache.maven.artifact.repository.ArtifactRepository;
@@ -44,18 +53,8 @@
import org.codehaus.plexus.component.annotations.Component;
import org.codehaus.plexus.component.annotations.Requirement;
import org.codehaus.plexus.logging.AbstractLogEnabled;
-import org.codehaus.plexus.logging.Logger;
import org.codehaus.plexus.util.StringUtils;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.HashSet;
-import java.util.LinkedHashMap;
-import java.util.LinkedHashSet;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-
/**
* @author jdcasey
* @version $Id$
@@ -65,28 +64,12 @@
extends AbstractLogEnabled
implements DependencyResolver
{
-
@Requirement
private RepositorySystem resolver;
-
@Requirement
private ArtifactFactory factory;
- @SuppressWarnings( "UnusedDeclaration" )
- public DefaultDependencyResolver()
- {
- // for plexus init
- }
-
- protected DefaultDependencyResolver( final RepositorySystem resolver, final ArtifactFactory factory,
- final Logger logger )
- {
- this.resolver = resolver;
- this.factory = factory;
- enableLogging( logger );
- }
-
@Override
public Map<DependencySet, Set<Artifact>> resolveDependencySets( final Assembly assembly, ModuleSet moduleSet,
final AssemblerConfigurationSource configSource,
diff --git a/src/test/java/org/apache/maven/plugins/assembly/artifact/DefaultDependencyResolverTest.java b/src/test/java/org/apache/maven/plugins/assembly/artifact/DefaultDependencyResolverTest.java
index 28be57a..32f5622 100644
--- a/src/test/java/org/apache/maven/plugins/assembly/artifact/DefaultDependencyResolverTest.java
+++ b/src/test/java/org/apache/maven/plugins/assembly/artifact/DefaultDependencyResolverTest.java
@@ -41,7 +41,6 @@
import org.apache.maven.plugins.assembly.model.Repository;
import org.apache.maven.plugins.assembly.resolved.AssemblyId;
import org.apache.maven.project.MavenProject;
-import org.apache.maven.repository.RepositorySystem;
import org.codehaus.plexus.PlexusTestCase;
import org.codehaus.plexus.logging.Logger;
import org.codehaus.plexus.logging.console.ConsoleLogger;
@@ -57,9 +56,7 @@
private ArtifactRepositoryLayout layout;
- private RepositorySystem resolver;
-
- private ConsoleLogger logger;
+ private DefaultDependencyResolver resolver;
@Override
public void setUp()
@@ -67,11 +64,11 @@
{
super.setUp();
- resolver = lookup( RepositorySystem.class );
+ resolver = (DefaultDependencyResolver) lookup( DependencyResolver.class );
+
factory = lookup( ArtifactFactory.class );
repoFactory = lookup( ArtifactRepositoryFactory.class );
layout = lookup( ArtifactRepositoryLayout.class, "default" );
- logger = new ConsoleLogger( Logger.LEVEL_DEBUG, "test" );
}
public void test_getDependencySetResolutionRequirements()
@@ -90,7 +87,7 @@
final ResolutionManagementInfo info = new ResolutionManagementInfo( project );
final Assembly assembly = new Assembly();
- new DefaultDependencyResolver( resolver, factory, logger ).updateDependencySetResolutionRequirements( ds1, info,
+ resolver.updateDependencySetResolutionRequirements( ds1, info,
AssemblyId.createAssemblyId(
assembly ),
project );
@@ -185,7 +182,6 @@
mm.replayAll();
- final DefaultDependencyResolver resolver = new DefaultDependencyResolver( this.resolver, factory, logger );
resolver.enableLogging( new ConsoleLogger( Logger.LEVEL_DEBUG, "test" ) );
final Assembly assembly = new Assembly();
@@ -241,7 +237,7 @@
assembly.setRepositories( repositories );
final ResolutionManagementInfo info = new ResolutionManagementInfo( project );
- new DefaultDependencyResolver( resolver, factory, logger ).updateRepositoryResolutionRequirements( assembly,
+ resolver.updateRepositoryResolutionRequirements( assembly,
info );
assertTrue( info.isResolutionRequired() );
@@ -281,8 +277,7 @@
project.setRemoteArtifactRepositories( projectRepos );
final List<ArtifactRepository> aggregated =
- new DefaultDependencyResolver( resolver, factory, logger ).aggregateRemoteArtifactRepositories(
- externalRepos, Collections.singleton( project ) );
+ resolver.aggregateRemoteArtifactRepositories( externalRepos, Collections.singleton( project ) );
assertRepositoryWithId( er1.getId(), aggregated, true );
assertRepositoryWithId( er2.getId(), aggregated, true );