Continuum

Remove TestResult/SuiteResult/TestCaseFailure from the model

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.1-beta-3, 1.1-beta-4
  • Fix Version/s: 1.1-beta-4
  • Component/s: None
  • Labels:
    None
  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

TestResult/SuiteResult/TestCaseFailure are populated in MavenTwoBuildExecutor and never used.
It consume lot of resources to find surefire result, parse them but we don't need them because surefire files are parsed directly by the SurefireReportAction

Activity

Hide
Brett Porter added a comment -

agree to remove them from the model

I thought they were added to make it possible to have historical reports - maybe we just need to copy the xml files alongside the build results?

Show
Brett Porter added a comment - agree to remove them from the model I thought they were added to make it possible to have historical reports - maybe we just need to copy the xml files alongside the build results?
Hide
Emmanuel Venisse added a comment -

We already copy xml files and we use them to generate the report.

Show
Emmanuel Venisse added a comment - We already copy xml files and we use them to generate the report.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: