We can create a new interface then if we want to retain the behavior of the original component (which is fine) but we need to move this toward resolving the tree and operating on that. Primarily to start making a real bridge to plug in Mercury properly.

On 24-May-09, at 6:35 AM, [email protected] wrote:

Author: bentmann
Date: Sun May 24 13:35:29 2009
New Revision: 778146

URL: http://svn.apache.org/viewvc?rev=778146&view=rev
Log:
o Fixed artifact resolver to deliver root artifact first to retain class path order (intermediate update to test, the legacy method being called shouldn't contain the root artifact at all IIRC so needs some investigation)

Modified:
maven/components/branches/MNG-2766/maven-compat/src/test/java/org/ apache/maven/artifact/resolver/ArtifactResolverTest.java

Modified: maven/components/branches/MNG-2766/maven-compat/src/test/ java/org/apache/maven/artifact/resolver/ArtifactResolverTest.java
URL: 
http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-compat/src/test/java/org/apache/maven/artifact/resolver/ArtifactResolverTest.java?rev=778146&r1=778145&r2=778146&view=diff
= = = = = = = = ====================================================================== --- maven/components/branches/MNG-2766/maven-compat/src/test/java/ org/apache/maven/artifact/resolver/ArtifactResolverTest.java (original) +++ maven/components/branches/MNG-2766/maven-compat/src/test/java/ org/apache/maven/artifact/resolver/ArtifactResolverTest.java Sun May 24 13:35:29 2009
@@ -231,6 +231,7 @@
artifactResolver.resolveTransitively( set, projectArtifact, remoteRepositories(), localRepository(), mds );

        Iterator i = result.getArtifacts().iterator();
+        i.next();
        assertEquals( "n should be first", n, i.next() );
        assertEquals( "m should be second", m, i.next() );

@@ -243,6 +244,7 @@
artifactResolver.resolveTransitively( set, projectArtifact, remoteRepositories(), localRepository(), mds );

        i = result.getArtifacts().iterator();
+        i.next();
        assertEquals( "m should be first", m, i.next() );
        assertEquals( "n should be second", n, i.next() );
    }



Thanks,

Jason

----------------------------------------------------------
Jason van Zyl
Founder,  Apache Maven
http://twitter.com/jvanzyl
http://twitter.com/SonatypeNexus
http://twitter.com/SonatypeM2E
----------------------------------------------------------

What matters is not ideas, but the people who have them. Good people can fix bad ideas, but good ideas can't save bad people.

 -- Paul Graham


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to