You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by Brett Porter <br...@apache.org> on 2008/07/04 05:24:26 UTC

Re: svn commit: r673896 - /maven/artifact/trunk/src/test/java/org/apache/maven/artifact/repository/DefaultArtifactRepositoryFactoryTest.java

I commented on the JIRA - I'm pretty sure this is a problem with the  
active collections.

- Brett

On 04/07/2008, at 1:15 PM, jvanzyl@apache.org wrote:

> Author: jvanzyl
> Date: Thu Jul  3 20:15:24 2008
> New Revision: 673896
>
> URL: http://svn.apache.org/viewvc?rev=673896&view=rev
> Log:
> Checking MNG-3422, all the layouts are there. The descriptors are  
> generated and are available in tests so they are getting punched  
> elsewhere.
>
> Modified:
>    maven/artifact/trunk/src/test/java/org/apache/maven/artifact/ 
> repository/DefaultArtifactRepositoryFactoryTest.java
>
> Modified: maven/artifact/trunk/src/test/java/org/apache/maven/ 
> artifact/repository/DefaultArtifactRepositoryFactoryTest.java
> URL: http://svn.apache.org/viewvc/maven/artifact/trunk/src/test/java/org/apache/maven/artifact/repository/DefaultArtifactRepositoryFactoryTest.java?rev=673896&r1=673895&r2=673896&view=diff
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- maven/artifact/trunk/src/test/java/org/apache/maven/artifact/ 
> repository/DefaultArtifactRepositoryFactoryTest.java (original)
> +++ maven/artifact/trunk/src/test/java/org/apache/maven/artifact/ 
> repository/DefaultArtifactRepositoryFactoryTest.java Thu Jul  3  
> 20:15:24 2008
> @@ -69,4 +69,17 @@
>
>         assertTrue( layout instanceof DefaultRepositoryLayout );
>     }
> +
> +    public void testRetrievalOfKnownRepositoryLayouts()
> +        throws Exception
> +    {
> +        ArtifactRepositoryLayout defaultLayout =  
> repoFactory.getLayout( "default" );
> +        assertNotNull( defaultLayout );
> +
> +        ArtifactRepositoryLayout legacyLayout =  
> repoFactory.getLayout( "legacy" );
> +        assertNotNull( legacyLayout );
> +
> +        ArtifactRepositoryLayout flatLayout =  
> repoFactory.getLayout( "flat" );
> +        assertNotNull( flatLayout );
> +    }
> }
>
>

--
Brett Porter
brett@apache.org
http://blogs.exist.com/bporter/


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org


Re: svn commit: r673896 - /maven/artifact/trunk/src/test/java/org/apache/maven/artifact/repository/DefaultArtifactRepositoryFactoryTest.java

Posted by Jason van Zyl <ja...@maven.org>.
The test uses the active collections so it's something else (maybe in  
conjunction with the active collections).

On 3-Jul-08, at 11:24 PM, Brett Porter wrote:

> I commented on the JIRA - I'm pretty sure this is a problem with the  
> active collections.
>
> - Brett
>
> On 04/07/2008, at 1:15 PM, jvanzyl@apache.org wrote:
>
>> Author: jvanzyl
>> Date: Thu Jul  3 20:15:24 2008
>> New Revision: 673896
>>
>> URL: http://svn.apache.org/viewvc?rev=673896&view=rev
>> Log:
>> Checking MNG-3422, all the layouts are there. The descriptors are  
>> generated and are available in tests so they are getting punched  
>> elsewhere.
>>
>> Modified:
>>   maven/artifact/trunk/src/test/java/org/apache/maven/artifact/ 
>> repository/DefaultArtifactRepositoryFactoryTest.java
>>
>> Modified: maven/artifact/trunk/src/test/java/org/apache/maven/ 
>> artifact/repository/DefaultArtifactRepositoryFactoryTest.java
>> URL: http://svn.apache.org/viewvc/maven/artifact/trunk/src/test/java/org/apache/maven/artifact/repository/DefaultArtifactRepositoryFactoryTest.java?rev=673896&r1=673895&r2=673896&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- maven/artifact/trunk/src/test/java/org/apache/maven/artifact/ 
>> repository/DefaultArtifactRepositoryFactoryTest.java (original)
>> +++ maven/artifact/trunk/src/test/java/org/apache/maven/artifact/ 
>> repository/DefaultArtifactRepositoryFactoryTest.java Thu Jul  3  
>> 20:15:24 2008
>> @@ -69,4 +69,17 @@
>>
>>        assertTrue( layout instanceof DefaultRepositoryLayout );
>>    }
>> +
>> +    public void testRetrievalOfKnownRepositoryLayouts()
>> +        throws Exception
>> +    {
>> +        ArtifactRepositoryLayout defaultLayout =  
>> repoFactory.getLayout( "default" );
>> +        assertNotNull( defaultLayout );
>> +
>> +        ArtifactRepositoryLayout legacyLayout =  
>> repoFactory.getLayout( "legacy" );
>> +        assertNotNull( legacyLayout );
>> +
>> +        ArtifactRepositoryLayout flatLayout =  
>> repoFactory.getLayout( "flat" );
>> +        assertNotNull( flatLayout );
>> +    }
>> }
>>
>>
>
> --
> Brett Porter
> brett@apache.org
> http://blogs.exist.com/bporter/
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
> For additional commands, e-mail: dev-help@maven.apache.org
>

Thanks,

Jason

----------------------------------------------------------
Jason van Zyl
Founder,  Apache Maven
jason at sonatype dot com
----------------------------------------------------------

In short, man creates for himself a new religion of a rational
and technical order to justify his work and to be justified in it.

   -- Jacques Ellul, The Technological Society


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org