Test Errors in Notification Mail

[murali mohan <mu...@gmail.com>]

I think it is very important to have the *number *of tests in error in the
notification sent after every build. I have added this functionality in my
project. So i can submit the changes. But a few things :

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?
We can also add a success rate of tests to the velocity template. Can we add
this also?
as far as I know, there is no way to display Tests in error and the
exception stacktrace in the mail. Should we add this? If yes, i can look at
this as the next step.

If everybody is fine about adding this, then I think i should follow the
steps:
1) create a Jira issue
2) submit the changes - a) ReportTestResult.java, b) common.vm and c) A unit
test for this

Pls let me know what you think.

Murali.