Mojo EMMA Plugin

Split the report generation from the coverage measurement

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Blocker Blocker
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

This is a blocker for a full release.

The report generation should be implemented as a reportSet

The coverage measurement should remain implemented as a mojo

Not sure if the reportSet should always invoke the coverage measurement, or only invoke it if coverage results are not present.

Issue Links

supercedes

Improvement - An improvement or enhancement to an existing feature or task. MEMMA-4 [emma-maven-plugin] Allow generating reports outside the emma:emma goal

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Philippe Laflamme added a comment -

I feel that report generation should be conditional.

Run the coverage if any of the instrumented test classes or test resource file is more recent than coverage.em. Or if coverage.em does not exist.

I suppose converting coverage.[xml|html] should also be conditional to coverage.em's timestamp.

Show
Philippe Laflamme added a comment - I feel that report generation should be conditional. Run the coverage if any of the instrumented test classes or test resource file is more recent than coverage.em. Or if coverage.em does not exist. I suppose converting coverage. [xml|html] should also be conditional to coverage.em's timestamp.
Hide
Permalink
Chris Holman added a comment -

I think I have a simple solution for this.

Copy the EmmaReportMojo.java --> EmmaMojo.java
In EmmaReportMojo.java:
Rename the @goal from emma to report
Remove the @execute

Does this look reasonable?

Show
Chris Holman added a comment - I think I have a simple solution for this. Copy the EmmaReportMojo.java --> EmmaMojo.java In EmmaReportMojo.java: Rename the @goal from emma to report Remove the @execute Does this look reasonable?
Hide
Permalink
Steffen Stundzig added a comment -

Hi Chris,

your solution works perfect. I've patched the plugin in our local repository and it works. Hope the codehaus people will fix it also in the near future.

regards

Show
Steffen Stundzig added a comment - Hi Chris, your solution works perfect. I've patched the plugin in our local repository and it works. Hope the codehaus people will fix it also in the near future. regards

People

  • Assignee:
    Unassigned
    Reporter:
    Stephen Connolly
Vote (5)
Watch (5)

Dates

  • Created:
    Updated: