Message:

  A new issue has been created in JIRA.

---------------------------------------------------------------------
View the issue:

  http://jira.codehaus.org/secure/ViewIssue.jspa?key=MPJUNITREPORT-2


Here is an overview of the issue:
---------------------------------------------------------------------
        Key: MPJUNITREPORT-2
    Summary: formatting/escaping of error messages
       Type: Bug

     Status: Unassigned
   Priority: Critical

 Original Estimate: Unknown
 Time Spent: Unknown
  Remaining: Unknown

    Project: maven-junitreport-plugin

   Assignee: 
   Reporter: fabrizio giustina

    Created: Tue, 13 Jan 2004 2:57 PM
    Updated: Tue, 13 Jan 2004 2:57 PM
Environment: maven rc1, maven-junit-report-plugin-1.3

Description:
error details and stacktraces in junit report can be unreadable. There are multiple 
problems in the formatting:

- error details in the original junit output are already xml-escaped. In junit.jsl are 
again html-escaped (${htmlescape.getText(current.attribute('message').value)}) so < 
%gt; will be escaped 2 times! Text should not be escaped again but put in a CDATA 
section (or unescaping xml before escaping html)

- newlines are not preserved. Reading stacktraces is really hard! Test should be 
wrapped in <pre>, not in <code>

- lots of whitespaces and newlines are added, also inside comment (usually around all 
[ ] < > / characters). This could also make the message loose all its meaning, say 
Error: expected value = "[2][2]" actual = "[2] [2 ]"
can become:
Error : expected value = " [ 2 ] [ 2 ] " actual = " [ 2 ] [ 2 ] ".
This is a common problem with jellyscript, but should be avoided wrapping text in 
cdata before the jsl transformation


see the jtidy maven site for an example of unreadable report:
http://jtidy.sourceforge.net/junit-report.html





---------------------------------------------------------------------
JIRA INFORMATION:
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

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to