SonarQube Plugins
  1. SonarQube Plugins
  2. SONARPLUGINS-1318

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. EmmaProcessor.java
        5 kB
        Sébastien Alonzo
      2. EmmaProcessorTest.java
        3 kB
        Sébastien Alonzo

        Issue Links

          Activity

          Hide
          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
          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
          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
          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
          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
          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
          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
          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
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: