[
http://jira.codehaus.org/browse/CONTINUUM-1694?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=130271#action_130271
]
Joel Turkel commented on CONTINUUM-1694:
----------------------------------------
With your fix it looks like the failure details aren't being reported properly
for test errors. I believe the fix is to swap lines 185 and 186 in
continuum-reports\src\main\java\org\apache\maven\continuum\reports\surefire\ReportTestSuite.java.
Also the velocity template should probably be updated to include test failures
if there are test failures or test errors.
> Adding Number of Errors and Test Success Rate to Notification Email
> -------------------------------------------------------------------
>
> Key: CONTINUUM-1694
> URL: http://jira.codehaus.org/browse/CONTINUUM-1694
> Project: Continuum
> Issue Type: Improvement
> Components: Notifier - Mail
> Affects Versions: 1.2
> Environment: Windows Vista
> Reporter: Murali Mohan
> Assignee: Olivier Lamy
> Priority: Minor
> Fix For: 1.2
>
> Attachments: CONTINUUM-1694.patch
>
>
> To have the *number* of tests in error and the success rate of tests in the
> notification mail sent after every build.
> Why is it required - Because, currently the mail gives a wrong sense of
> success rate in tests where as there may be a good number of tests in error.
> Even though, ppl can go and check the online report, having the summary in
> the mails is important.
>
> Was there any reason behind excluding test errors? - Not Sure
--
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
