Author: jdcasey Date: Wed Jul 26 14:56:35 2006 New Revision: 425863 URL: http://svn.apache.org/viewvc?rev=425863&view=rev Log: [MNG-1562] Added unit test confirming the necessity of groupId:artifactId:type for non-jar dependency references. Will update site documentation as well.
Modified: maven/components/trunk/maven-project/src/test/java/org/apache/maven/project/injection/DefaultModelDefaultsInjectorTest.java Modified: maven/components/trunk/maven-project/src/test/java/org/apache/maven/project/injection/DefaultModelDefaultsInjectorTest.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/test/java/org/apache/maven/project/injection/DefaultModelDefaultsInjectorTest.java?rev=425863&r1=425862&r2=425863&view=diff ============================================================================== --- maven/components/trunk/maven-project/src/test/java/org/apache/maven/project/injection/DefaultModelDefaultsInjectorTest.java (original) +++ maven/components/trunk/maven-project/src/test/java/org/apache/maven/project/injection/DefaultModelDefaultsInjectorTest.java Wed Jul 26 14:56:35 2006 @@ -16,6 +16,8 @@ * limitations under the License. */ +import java.util.List; + import junit.framework.TestCase; import org.apache.maven.model.Dependency; @@ -23,8 +25,6 @@ import org.apache.maven.model.Exclusion; import org.apache.maven.model.Model; -import java.util.List; - /** * @author jdcasey */ @@ -34,6 +34,45 @@ public void testShouldConstructWithNoParams() { new DefaultModelDefaultsInjector(); + } + + public void testShouldMergeManagedDependencyOfTypeEJBToDependencyList() + { + Model model = new Model(); + + Dependency managedDep = new Dependency(); + + managedDep.setGroupId( "group" ); + managedDep.setArtifactId( "artifact" ); + managedDep.setVersion( "1.0" ); + managedDep.setType( "ejb" ); + + DependencyManagement depMgmt = new DependencyManagement(); + + depMgmt.addDependency( managedDep ); + + model.setDependencyManagement( depMgmt ); + + Dependency dep = new Dependency(); + + dep.setGroupId( "group" ); + dep.setArtifactId( "artifact" ); + + // looks like groupId:artifactId:type is the minimum for identification, where + // type is defaulted to "jar". + dep.setType( "ejb" ); + + model.addDependency( dep ); + + new DefaultModelDefaultsInjector().injectDefaults( model ); + + List resultingDeps = model.getDependencies(); + + assertEquals( 1, resultingDeps.size() ); + + Dependency result = (Dependency) resultingDeps.get( 0 ); + + assertEquals( "1.0", result.getVersion() ); } public void testShouldSucceedInMergingDependencyWithDependency()