Details

    • Type: New Feature New Feature
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: 2.5
    • Labels:
      None
    • Number of attachments :
      4

      Description

      I'm intending to write a Merge task for this plugin, as the company I work for has a need for a grand, summarizing report for their entire project, not just one for each module.

      This warrants the question: is there any interest among other users and developers to have this bit contributed back into the plugin? And if so, does anyone have ideas on how to best go about doing such a thing? I'm currently trying to wrap my head around the MOJO API, and have some doubts as to how to go about certain things in the best way.

      1. coberturamerge.patch
        21 kB
        Henrik W Lund
      2. CoberturaMergeMojo.java
        1 kB
        Henrik W Lund
      3. example-config.xml
        0.7 kB
        Henrik W Lund
      4. MergeTask.java
        2 kB
        Henrik W Lund

        Issue Links

          Activity

          Hide
          Stéphane Nicoll added a comment -

          That one would be really great!

          Show
          Stéphane Nicoll added a comment - That one would be really great!
          Hide
          David Vicente added a comment -

          why you don't take a look of Dashboard-maven-plugin josted in codehaus ?

          http://mojo.codehaus.org/dashboard-maven-plugin/

          see "Sample reports" for single project and multi-moduls project.

          Show
          David Vicente added a comment - why you don't take a look of Dashboard-maven-plugin josted in codehaus ? http://mojo.codehaus.org/dashboard-maven-plugin/ see "Sample reports" for single project and multi-moduls project.
          Hide
          Bob Fields added a comment -

          It seems like the internal cobertura provided aggregation could be used instead. Is anybody using the cobertura-merge ant task to create the merged results for all projects, or does the patch accomplish the same thing? http://cobertura.sourceforge.net/anttaskreference.html. Is the merge task smart enough to understand that tests in one project module may provide additional coverage for classes in another dependent project module? Or does it simply merge all results into an existing single file?

          The dashboard plugin example shows 0% coverage for the Cobertura tests (a common problem related to a customized maven configuration). It also states that "Maven's internal support for report aggregation is rather poor and does have a number of limitations. This may be addressed by an application like Sonar." at http://mojo.codehaus.org/dashboard-maven-plugin/faq.html.

          Show
          Bob Fields added a comment - It seems like the internal cobertura provided aggregation could be used instead. Is anybody using the cobertura-merge ant task to create the merged results for all projects, or does the patch accomplish the same thing? http://cobertura.sourceforge.net/anttaskreference.html . Is the merge task smart enough to understand that tests in one project module may provide additional coverage for classes in another dependent project module? Or does it simply merge all results into an existing single file? The dashboard plugin example shows 0% coverage for the Cobertura tests (a common problem related to a customized maven configuration). It also states that "Maven's internal support for report aggregation is rather poor and does have a number of limitations. This may be addressed by an application like Sonar." at http://mojo.codehaus.org/dashboard-maven-plugin/faq.html .
          Hide
          Benson Margulies added a comment -

          Given the new aggregation support, is this still a wanted thing?

          Show
          Benson Margulies added a comment - Given the new aggregation support, is this still a wanted thing?
          Hide
          Benson Margulies added a comment -

          Addressed by -85.

          Show
          Benson Margulies added a comment - Addressed by -85.

            People

            • Assignee:
              Unassigned
              Reporter:
              Henrik W Lund
            • Votes:
              37 Vote for this issue
              Watchers:
              29 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: