[jira] [Commented] (SUREFIRE-2153) Replace SurefireReportGenerator with a new SurefireReportRenderer

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SUREFIRE-2153?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17718234#comment-17718234 ] 

ASF GitHub Bot commented on SUREFIRE-2153:
------------------------------------------

michael-o commented on PR #625:
URL: https://github.com/apache/maven-surefire/pull/625#issuecomment-1529805342

   @hboutemy Long overdue, last people of Doxia 2.0.0 preparation.




> Replace SurefireReportGenerator with a new SurefireReportRenderer
> -----------------------------------------------------------------
>
>                 Key: SUREFIRE-2153
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-2153
>             Project: Maven Surefire
>          Issue Type: Improvement
>          Components: Maven Surefire Report Plugin
>    Affects Versions: 3.0.0-M9
>            Reporter: Michael Osipov
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: 3.1.0
>
>
> The problem with {{SurefireReportGenerator}} is that is has been written 10+ years ago. It does not use {{AbstractMavenReportRenderer}} which causes a lot of code duplication. It shall be superseded with {{SurefireReportRenderer}} which does extend from the abstract renderer.
> The new renderer will also modernize the way data is displayed:
> * {{SurefireReportParser}} is supposed to collect data, not to format it. This is a task of the view layer
> * Separate percent and number formatters
> * Hide full table row instead of leaving ugly condensed one because of hidden div
> * Remove redudant div around verbatim block
> * Simplify renderer due to the foundatation on abstact renderer
> * Simplify tests



--
This message was sent by Atlassian Jira
(v8.20.10#820010)