You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@continuum.apache.org by "Joel Turkel (JIRA)" <ji...@codehaus.org> on 2008/04/09 19:30:58 UTC
[jira] Issue Comment Edited: (CONTINUUM-1694) Adding Number of
Errors and Test Success Rate to Notification Email
[ http://jira.codehaus.org/browse/CONTINUUM-1694?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=130271#action_130271 ]
joel.turkel edited comment on CONTINUUM-1694 at 4/9/08 12:28 PM:
-----------------------------------------------------------------
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/errors section if there are test failures or test errors.
was (Author: joel.turkel):
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