You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Olivier Lamy (JIRA)" <ji...@codehaus.org> on 2007/11/08 00:06:35 UTC

[jira] Commented: (CONTINUUM-1542) Surefire Report in continuum contains excess/superfluous test methods.

    [ http://jira.codehaus.org/browse/CONTINUUM-1542?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_113133 ] 

Olivier Lamy commented on CONTINUUM-1542:
-----------------------------------------

someone with karma on maven.zones can attach the xml tests output files ?
They are in ${buildDirectory}/296/32827/surefire-reports/*.xml
And for this builds too : http://maven.zones.apache.org/continuum/surefireReport.action?buildId=33424&projectId=441.
The trouble looks to happends when there are test in an AbstractTestCase class this looks to generate inappropriate 
Just build archiva and vi vi archiva-web/archiva-webapp/target/surefire-reports/TEST-org.apache.maven.archiva.web.repository.RepositoryServletProxiedPluginSnapshotPolicyTest.xml

At the end of the file I see :
{noformat} 
  <testcase time="2.292" name="testGetVersionMetadataDefaultLayout"/>
  <testcase time="0.311" name="testGetProjectMetadataDefaultLayout"/>
  <testcase time="0.287" name="testGetSnapshotVersionMetadataDefaultLayout"/>
  <testcase time="0.109" name="testArchivaTypes"/>
  <testcase time="1.481" name="testGetProxiedSnapshotsArtifactPolicyAlwaysManagedNewer"/>
  <testcase time="0.839" name="testGetProxiedSnapshotsArtifactPolicyAlwaysManagedOlder"/>
  <testcase time="0.883" name="testGetProxiedSnapshotsArtifactPolicyAlwaysNoManagedContent"/>
  <testcase time="0.79" name="testGetProxiedSnapshotsArtifactPolicyDailyFail"/>
  <testcase time="0.832" name="testGetProxiedSnapshotsArtifactPolicyDailyNoManagedContent"/>
  <testcase time="1.395" name="testGetProxiedSnapshotsArtifactPolicyDailyPass"/>
  <testcase time="0.773" name="testGetProxiedSnapshotsArtifactPolicyRejectFail"/>
  <testcase time="0.752" name="testGetProxiedSnapshotsArtifactPolicyRejectNoManagedContentFail"/>
  <testcase time="0.771" name="testGetProxiedSnapshotsArtifactPolicyRejectPass"/>
  <testcase time="0.919" name="testGetProxiedSnapshotsArtifactPolicyHourlyFail"/>
  <testcase time="0.757" name="testGetProxiedSnapshotsArtifactPolicyHourlyNoManagedContent"/>
  <testcase time="0.776" name="testGetProxiedSnapshotsArtifactPolicyHourlyPass"/>
  <testcase time="0.756" name="testGetProxiedSnapshotsArtifactPolicyOnceFail"/>
  <testcase time="0.765" name="testGetProxiedSnapshotsArtifactPolicyOnceNoManagedContent"/>
  <testcase time="0.77" name="testGetProxiedSnapshotsArtifactPolicyOncePass"/>
{noformat} 
That's why all of this are in the report.

> Surefire Report in continuum contains excess/superfluous test methods.
> ----------------------------------------------------------------------
>
>                 Key: CONTINUUM-1542
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-1542
>             Project: Continuum
>          Issue Type: Bug
>          Components: Testing
>            Reporter: Joakim Erdfelt
>            Assignee: Olivier Lamy
>             Fix For: 1.1
>
>
> When looking at the surefire test results within a continuum build that reported a failure, the list of test methods for the tested class contains extra methods from other tested classes.
> Continuum on maven.zones.apache.org [surefire report example with problem|http://maven.zones.apache.org/continuum/surefireReport.action;jsessionid=10vfs6g7jbl9f?buildId=32827&projectId=296#org.apache.maven.archiva.web.repository.RepositoryServletProxiedReleasePolicyTest]
> If you look at the list of test methods for RepositoryServletProxiedReleasePolicyTest you can see that the report shows the correct # of tests at the top with 15, but when you goto the details for that test, you see well over 15 tests.
> In fact, when you look at the section that starts with "Test Cases" you can see that the list of tests for each Test class  is actually the list of tests from the previous Test class + the tests for the current Test class.
> Example:
> +RepositoryServletNoProxyMetadataTest+
> 	
> (/) testGetVersionMetadataDefaultLayout 	
> (/) testGetProjectMetadataDefaultLayout 	
> (/) testGetSnapshotVersionMetadataDefaultLayout 	
> +ArchivaMimeTypeLoaderTest+
> 	
> (/) -testGetVersionMetadataDefaultLayout-
> (/) -testGetProjectMetadataDefaultLayout- 	
> (/) -testGetSnapshotVersionMetadataDefaultLayout- 	
> (/) testArchivaTypes 	
> +RepositoryServletProxiedPluginSnapshotPolicyTest+
> 	
> (/) -testGetVersionMetadataDefaultLayout- 	
> (/) -testGetProjectMetadataDefaultLayout- 	
> (/) -testGetSnapshotVersionMetadataDefaultLayout- 	
> (/) -testArchivaTypes- 	
> (/) testGetProxiedSnapshotsArtifactPolicyAlwaysManagedNewer 	
> (/) testGetProxiedSnapshotsArtifactPolicyAlwaysManagedOlder 	
> (/) testGetProxiedSnapshotsArtifactPolicyAlwaysNoManagedContent 	
> (/) testGetProxiedSnapshotsArtifactPolicyDailyFail 	
> ...

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