[ http://jira.codehaus.org/browse/SUREFIRE-691?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=259186#action_259186 ]
Kristian Rosenvold edited comment on SUREFIRE-691 at 3/7/11 3:06 PM: --------------------------------------------------------------------- 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 ? And please add license headers to the new files. was (Author: krosenvold): 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