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()


Reply via email to