Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-2995

Merging multiple reports

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Component/s: crap4j-plugin
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:

      Description

      ------- Additional comments from oliva Wed Feb 4 08:46:11 +0000 2009 -------

      hello daniel,

      another question regarding my tests: is it possible to merge multiple report.xml
      files within the crap4j/hudson plugin ?
      lets assume a fileset like **/target/crap4j/report.xml should find 2 reports:
      proj1/target/crap4j/report.xml and proj2/target/crap4j/report.xml. actually only
      one report is taken for the visualized report. is that what the plugin should do
      (take the first report that is found) or do i something wrong ?

      lg & thanks a lot,
      oliver

      ------- Additional comments from dlindner Thu Feb 5 20:30:06 +0000 2009 -------

      Hello Oliver,

      currently, the plugin only handles the first crap report file found by the
      pattern, that's true. Merging multiple reports would require the plugin to
      recalculate the overall numbers. This isn't too complicated, but maybe unstable
      if the two crap reports were taken with different parameter sets e.g. for the
      crap threshold.

      If you need this functionality, i would like you to open a new issue (request
      for enhancement) and gather thoughts there. Perhaps others chime in and want
      this feature, too. I'm willing to provide it, it may take some time, though.

        Attachments

          Activity

          Hide
          dlindner dlindner added a comment -

          I added rudimentary support for report merging. Whenever more than one report is
          found, all of the files get parsed (ensure clean patterns!) and merged together.

          Show
          dlindner dlindner added a comment - I added rudimentary support for report merging. Whenever more than one report is found, all of the files get parsed (ensure clean patterns!) and merged together.
          Hide
          dlindner dlindner added a comment -

          This is the issue for crap4j report merging. I will attach a test artifact soon.
          Please review this new feature. Thanks, Daniel.

          Show
          dlindner dlindner added a comment - This is the issue for crap4j report merging. I will attach a test artifact soon. Please review this new feature. Thanks, Daniel.
          Hide
          dlindner dlindner added a comment -

          Created an attachment (id=562)
          Crap4J plugin with report merging ability

          Show
          dlindner dlindner added a comment - Created an attachment (id=562) Crap4J plugin with report merging ability
          Hide
          dlindner dlindner added a comment -

          The attachment #562
          (https://hudson.dev.java.net/nonav/issues/showattachment.cgi/562/crap4j.hpi) is
          the newest version of the Crap4J plugin. It includes the ability to merge reports.
          Please review the new feature and provide feedback.
          Also mind that the memory consumption of all parsed reports will add up.

          Show
          dlindner dlindner added a comment - The attachment #562 ( https://hudson.dev.java.net/nonav/issues/showattachment.cgi/562/crap4j.hpi ) is the newest version of the Crap4J plugin. It includes the ability to merge reports. Please review the new feature and provide feedback. Also mind that the memory consumption of all parsed reports will add up.
          Hide
          oliva oliva added a comment -

          the provided plugin works fine for me.

          thanks a lot

          Show
          oliva oliva added a comment - the provided plugin works fine for me. thanks a lot
          Hide
          dlindner dlindner added a comment -

          Changing resolution to FIXED

          Show
          dlindner dlindner added a comment - Changing resolution to FIXED
          Hide
          dlindner dlindner added a comment -

          Feature included in the 0.6 release.

          Show
          dlindner dlindner added a comment - Feature included in the 0.6 release.

            People

            • Assignee:
              dlindner dlindner
              Reporter:
              dlindner dlindner
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: