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

Merging multiple reports

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • crap4j-plugin
    • None
    • Platform: All, OS: All

    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

          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.

          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.
          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.

          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.
          dlindner dlindner added a comment -

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

          dlindner dlindner added a comment - Created an attachment (id=562) Crap4J plugin with report merging ability
          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.

          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.
          oliva oliva added a comment -

          the provided plugin works fine for me.

          thanks a lot

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

          Changing resolution to FIXED

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

          Feature included in the 0.6 release.

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

          People

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

            Dates

              Created:
              Updated:
              Resolved: