Re: svn commit: r778146 - /maven/components/branches/MNG-2766/maven-compat/src/test/java/org/apache/maven/artifact/resolver/ArtifactResolverTest.java

[Jason van Zyl <jv...@sonatype.com>]

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, bentmann@apache.org 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: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org