[ 
http://jira.codehaus.org/browse/WAGON-73?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_104145
 ] 

Brian Fox edited comment on WAGON-73 at 8/6/07 8:33 PM:
--------------------------------------------------------

I ran the ITs using Maven 2.0.7 and I get one additional failure with the patch 
applied:
{noformat}
  
testSnapshotLocalMetadataUpdatedOnInstall(org.apache.maven.integrationtests.MavenIT0108SnapshotUpdateTest)
{noformat}

The ones that fail regardless of the patch or not are:

{noformat}
Failed tests:
  testit0002(org.apache.maven.integrationtests.MavenIT0002Test)
  testit0008(org.apache.maven.integrationtests.MavenIT0008Test)
  testit0009(org.apache.maven.integrationtests.MavenIT0009Test)
  testit0012(org.apache.maven.integrationtests.MavenIT0012Test)
  testit0022(org.apache.maven.integrationtests.MavenIT0022Test)
  testit0023(org.apache.maven.integrationtests.MavenIT0023Test)
  testit0025(org.apache.maven.integrationtests.MavenIT0025Test)
  testit0026(org.apache.maven.integrationtests.MavenIT0026Test)
  testit0027(org.apache.maven.integrationtests.MavenIT0027Test)
  testit0040(org.apache.maven.integrationtests.MavenIT0040Test)
  testit0041(org.apache.maven.integrationtests.MavenIT0041Test)
  testit0044(org.apache.maven.integrationtests.MavenIT0044Test)
  testit0045(org.apache.maven.integrationtests.MavenIT0045Test)
  testit0046(org.apache.maven.integrationtests.MavenIT0046Test)
  testit0049(org.apache.maven.integrationtests.MavenIT0049Test)
  testit0064(org.apache.maven.integrationtests.MavenIT0064Test)
  testit0071(org.apache.maven.integrationtests.MavenIT0071Test)
  testit0073(org.apache.maven.integrationtests.MavenIT0073Test)
  testit0082(org.apache.maven.integrationtests.MavenIT0082Test)
  testit0088(org.apache.maven.integrationtests.MavenIT0088Test)
  testit0090(org.apache.maven.integrationtests.MavenIT0090Test)

Tests in error:
  testit0086(org.apache.maven.integrationtests.MavenIT0086Test)
  testit0087(org.apache.maven.integrationtests.MavenIT0087Test)
  testit0104(org.apache.maven.integrationtests.MavenIT0104Test)

Tests run: 115, Failures: 21, Errors: 3, Skipped: 0
{noformat}

The error for this test is:
{noformat}
testSnapshotLocalMetadataUpdatedOnInstall(org.apache.maven.integrationtests.MavenIT0108SnapshotUpdateTest)
  Time elapsed: 3.5 sec  <<< FAILURE!
junit.framework.ComparisonFailure: expected:<...6...> but was:<...7...>
        at junit.framework.Assert.assertEquals(Assert.java:81)
        at junit.framework.Assert.assertEquals(Assert.java:87)
        at 
org.apache.maven.integrationtests.MavenIT0108SnapshotUpdateTest.assertLocalMetadataIsToday(MavenIT0108SnapshotUpdateTest.java:243)
        at 
org.apache.maven.integrationtests.MavenIT0108SnapshotUpdateTest.testSnapshotLocalMetadataUpdatedOnInstall(MavenIT0108SnapshotUpdateTest.java:215)
        at 
org.apache.maven.integrationtests.MavenIT0108SnapshotUpdateTest.testSnapshotLocalMetadataUpdatedOnInstall(MavenIT0108SnapshotUpdateTest.java:215)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
        at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:585)
        at junit.framework.TestCase.runTest(TestCase.java:154)

{noformat}


 was:
I ran the ITs using Maven 2.0.7 and I get one additional failure with the patch 
applied:
{noformat}
  
testSnapshotLocalMetadataUpdatedOnInstall(org.apache.maven.integrationtests.MavenIT0108SnapshotUpdateTest)
{noformat}

The ones that fail regardless of the patch or not are:

{noformat}
Failed tests:
  testit0002(org.apache.maven.integrationtests.MavenIT0002Test)
  testit0008(org.apache.maven.integrationtests.MavenIT0008Test)
  testit0009(org.apache.maven.integrationtests.MavenIT0009Test)
  testit0012(org.apache.maven.integrationtests.MavenIT0012Test)
  testit0022(org.apache.maven.integrationtests.MavenIT0022Test)
  testit0023(org.apache.maven.integrationtests.MavenIT0023Test)
  testit0025(org.apache.maven.integrationtests.MavenIT0025Test)
  testit0026(org.apache.maven.integrationtests.MavenIT0026Test)
  testit0027(org.apache.maven.integrationtests.MavenIT0027Test)
  testit0040(org.apache.maven.integrationtests.MavenIT0040Test)
  testit0041(org.apache.maven.integrationtests.MavenIT0041Test)
  testit0044(org.apache.maven.integrationtests.MavenIT0044Test)
  testit0045(org.apache.maven.integrationtests.MavenIT0045Test)
  testit0046(org.apache.maven.integrationtests.MavenIT0046Test)
  testit0049(org.apache.maven.integrationtests.MavenIT0049Test)
  testit0064(org.apache.maven.integrationtests.MavenIT0064Test)
  testit0071(org.apache.maven.integrationtests.MavenIT0071Test)
  testit0073(org.apache.maven.integrationtests.MavenIT0073Test)
  testit0082(org.apache.maven.integrationtests.MavenIT0082Test)
  testit0088(org.apache.maven.integrationtests.MavenIT0088Test)
  testit0090(org.apache.maven.integrationtests.MavenIT0090Test)

Tests in error:
  testit0086(org.apache.maven.integrationtests.MavenIT0086Test)
  testit0087(org.apache.maven.integrationtests.MavenIT0087Test)
  testit0104(org.apache.maven.integrationtests.MavenIT0104Test)

Tests run: 115, Failures: 21, Errors: 3, Skipped: 0
{noformat}

> MirroredWagon infinite loop
> ---------------------------
>
>                 Key: WAGON-73
>                 URL: http://jira.codehaus.org/browse/WAGON-73
>             Project: wagon
>          Issue Type: Bug
>            Reporter: Phillip Webb
>            Priority: Critical
>             Fix For: 2.0
>
>         Attachments: returnsonmirroredwagon.patch, 
> WAGON-73-ConnectAndGetTest.patch, WagonManagerTest.java.patch
>
>
> The MirroredWagon class includes a get method that runs into an infinite loop.
> I think a return is required after this.impl.get( resource, destination );
>     public void get( String resource, File destination )
>         throws TransferFailedException, ResourceDoesNotExistException, 
> AuthorizationException
>     {
>         try
>         {
>             while ( true )
>             {
>                 try
>                 {
>                     this.impl.get( resource, destination );
>                 }
>                 catch ( TransferFailedException e )
>                 {
>                     nextMirror();
>                 }
>             }
>         }
>         catch ( ExhaustedMirrorsException e )
>         {
>         }
>     }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to