[ 
http://jira.codehaus.org/browse/SUREFIRE-691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=259186#action_259186
 ] 

Kristian Rosenvold commented on SUREFIRE-691:
---------------------------------------------

I'm not entirely sure I understand these patches still. The first patch 
contains a NoReportIT.java, while the second patch does not. And the test in 
the second patch seems to be referencing a diferctory "test-reports"; where is 
this ?

> Skip reports and menu link generation when it does not make sense (i.e: test 
> results are missing)
> -------------------------------------------------------------------------------------------------
>
>                 Key: SUREFIRE-691
>                 URL: http://jira.codehaus.org/browse/SUREFIRE-691
>             Project: Maven Surefire
>          Issue Type: Improvement
>          Components: Maven Surefire Report Plugin
>    Affects Versions: 2.8
>            Reporter: Marcin Kuthan
>            Priority: Minor
>         Attachments: screenshot.png, SUREFIRE-691.patch, SUREFIRE-691.patch
>
>
> Surefire report page and link in the menu are generated even if there is no 
> tests. It also happens for submodules when -Daggregate=true is given. 
> Another plugins which supports report aggregation (m-pmd-p, m-javadoc-p, 
> m-jxr-p) don't generate link in the menu and empty report page when report 
> does not make sense.

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