Sonar Plugins

The name of the Emma .ec report file is hardcoded and must be "coverage-0.ec"

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: EMMA-1.2.1
  • Component/s: Emma
  • Labels:
    None
  • Number of attachments :
    3

Description

It would be better to automatically find a report file with the ".ec" extension point (whatever is its name) and to log a error if this report file can't be found.

  1. Java Source File
    EmmaProcessor.java
    18/Aug/11 4:46 AM
    5 kB
    Sébastien Alonzo
  2. Java Source File
    EmmaProcessorTest.java
    18/Aug/11 4:48 AM
    3 kB
    Sébastien Alonzo

Issue Links

is duplicated by

Improvement - An improvement or enhancement to an existing feature or task. SONARPLUGINS-1297 Re-using EMMA coverage data generated by TeamCity

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is related to

Bug - A problem which impairs or prevents the functions of the product. SONARPLUGINS-1090 Sonar Emma Plugin reporting 0.0% coverage

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Sébastien Alonzo added a comment -

Emma supports merging coverage sessions, therefore it could be interesting to look for all report files with ".ec" extension in the output folder.

I've already forked EmmaProcessor to implement such a feature and it's working pretty well. I'd be glad to contribute a patch if you're interested.

Show
Sébastien Alonzo added a comment - Emma supports merging coverage sessions, therefore it could be interesting to look for all report files with ".ec" extension in the output folder. I've already forked EmmaProcessor to implement such a feature and it's working pretty well. I'd be glad to contribute a patch if you're interested.
Hide
Permalink
Freddy Mallet added a comment -

With pleasure Sébastien ... feel free to attach your patch to this ticket. I can even grant you rights to be able to directly commit on the Sonar plugin forge but for this you must first create your Codehaus account (see http://xircles.codehaus.org/)

Show
Freddy Mallet added a comment - With pleasure Sébastien ... feel free to attach your patch to this ticket. I can even grant you rights to be able to directly commit on the Sonar plugin forge but for this you must first create your Codehaus account (see http://xircles.codehaus.org/ )
Hide
Permalink
Sébastien Alonzo added a comment - - edited

Added a patch to support multiple coverage files with ".ec" extension (see EmmaProcessor.java)

Show
Sébastien Alonzo added a comment - - edited Added a patch to support multiple coverage files with ".ec" extension (see EmmaProcessor.java)
Hide
Permalink
Sébastien Alonzo added a comment - - edited

Added TestCase for multiple coverage data files (see EmmaProcessorTest.java and multipleData.zip)

Show
Sébastien Alonzo added a comment - - edited Added TestCase for multiple coverage data files (see EmmaProcessorTest.java and multipleData.zip)
Hide
Permalink
Sébastien Alonzo added a comment -

I've posted the patch with an associated testcase.
I also created my Codehaus account and I'm willing to commit the changes if you would grant me access rights to the source repository.

Show
Sébastien Alonzo added a comment - I've posted the patch with an associated testcase. I also created my Codehaus account and I'm willing to commit the changes if you would grant me access rights to the source repository.
Hide
Permalink
Freddy Mallet added a comment -

Thanks Sébastien and welcome on board. You now have the developer role on the Sonar Plugins project so you should be able to checkout the Emma plugin source code to commit your changes : https://svn.codehaus.org/sonar-plugins/trunk/emma/.

Show
Freddy Mallet added a comment - Thanks Sébastien and welcome on board. You now have the developer role on the Sonar Plugins project so you should be able to checkout the Emma plugin source code to commit your changes : https://svn.codehaus.org/sonar-plugins/trunk/emma/ .
Hide
Permalink
Sébastien Alonzo added a comment -

The plugin will now look for any files ending with .ec (emma coverage data) and .em (emma metadata).
It's even possible to merge multiple files that way.

Show
Sébastien Alonzo added a comment - The plugin will now look for any files ending with .ec (emma coverage data) and .em (emma metadata). It's even possible to merge multiple files that way.
Hide
Permalink
Freddy Mallet added a comment -

Thanks SĂ©bastien for this patch. I'm releasing the version 1.2.1 of the Emma plugin.

Show
Freddy Mallet added a comment - Thanks SĂ©bastien for this patch. I'm releasing the version 1.2.1 of the Emma plugin.

People

  • Assignee:
    Sébastien Alonzo
    Reporter:
    Freddy Mallet
Vote (1)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: